Russ Allbery wrote:
> Luk Claes <luk@debian.org> writes:
>
>> tags 330025 +patch
>> thanks
>
>> Hi
>
>> Easy patch included for consideration.
>
> Thanks! At this point I'm tempted to take this, although I think that
> CDBS still uses DH_COMPAT rather than a debian/compat file (but I'm not
> positive, and regardless this sort of check wouldn't catch it).
>
> However, in a quick check, I can't find any information that substantiates
> the original reporter's assertion that it's deprecated. The description
> in debhelper's man page is:
>
> DH_COMPAT
> Temporarily specifies what compatibility level debhelper should run
> at, overriding any value in debian/compat.
>
> which doesn't sound deprecated. Maybe it's the "temporarily" that should
> be read as implying that it shouldn't be used as a normal part of
> debian/rules?
>
> A warning about setting DH_COMPAT in debian/rules is going to catch a
> *lot* of packages, so I want to be sure that it's really the right thing
> to warn about before we do it.
According to joeyh it's appropriate to warn for use of DH_COMPAT at the top of
the file (before any Makefile target).
Applying:
- if (m/^\s*DH_COMPAT(?:\s|:)/) {
- tag "declares-deprecated-DH_COMPAT");
+ if (m/^\s*DH_COMPAT(?=\s|=)/ && keys(%seen) == 0) {
+ tag "declares-deprecated-DH_COMPAT";
in the patch solves that AFAICS :-)
Though the description should also be updated: first try...
N: As of debhelper version 4, the DH_COMPAT environment variable is
N: only to be used for temporarily overriding debian/compat, so any command
N: in debian/rules that sets it on top of debian/rules should be deleted.
N:
N: Please refer to the debhelper(7) manual page for more information.
Cheers
Luk
--
Luk Claes - http://people.debian.org/~luk - GPG key 1024D/9B7C328D
Fingerprint: D5AF 25FB 316B 53BB 08E7 F999 E544 DE07 9B7C 328D
Attachment:
signature.asc
Description: OpenPGP digital signature