Bug#834370: lintian: c/debhelper.pm: Correct false positive in matching targets with extra whitespace
Niels Thykier:
> Chris Lamb:
>> [...]
> 
> Hi,
> 
> Thanks for working on this. :)
> 
> This makes me wonder if:
> 
>   # Skip these
>   override_dh_foo override_dh_bar:
> 
> work as intended[1]. From a quick glance, it looks like our code only
> validates the first override in a single line.
> 
> Thanks,
> ~Niels
> 
> [1] Examples include:
> https://anonscm.debian.org/git/debhelper/debhelper.git/tree/debian/rules
> 
Obviously, I would forget to mention that we also have:
 override_dh_foo-arch (or override_dh_foo-indep)
Which preferably should be matched against dh_foo rather than
dh_foo-{arch,indep}.
(Admittedly, this would generally go in the false-negative bucket, but
while we are fixing this check, we might as well support it).
Thanks,
~Niels
Reply to: