[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 55eecc314775c88e18a771e9d9f9b7118ac5dc5c



The following commit has been merged in the master branch:
commit 55eecc314775c88e18a771e9d9f9b7118ac5dc5c
Author: Russ Allbery <rra@debian.org>
Date:   Sat Jun 28 19:46:23 2008 -0700

    Warn about DEB_BUILD_OPTS in debian/rules
    
    * checks/rules{,.desc}:
      + [RA] Warn about use of DEB_BUILD_OPTS.  Patch from Raphael Geissert.

diff --git a/checks/rules b/checks/rules
index 238b64d..300d0b9 100644
--- a/checks/rules
+++ b/checks/rules
@@ -113,6 +113,9 @@ while (<RULES>) {
 	/^\t\s*(?:\$[\(\{]MAKE[\}\)]|make)\s(?:.*\s)?-\w*i.*(?:dist)?clean/) {
         tag "debian-rules-ignores-make-clean-error", "line $.";
     }
+    if (/$[\(\{]DEB_BUILD_OPTS[\)\}]/) {
+        tag "debian-rules-uses-DEB_BUILD_OPTS", "line $.";
+    }
 
     # Listing a rule as a dependency of .PHONY is sufficient to make it
     # present for the purposes of GNU make and therefore the Policy
diff --git a/checks/rules.desc b/checks/rules.desc
index 5259dad..f2fd28d 100644
--- a/checks/rules.desc
+++ b/checks/rules.desc
@@ -64,6 +64,12 @@ Info: A rule in the <tt>debian/rules</tt> file for this package calls the
  so that other error messages from the clean or distclean rule will still
  be caught (or just remove the "-" if the package uses a static makefile).
 
+Tag: debian-rules-uses-DEB_BUILD_OPTS
+Type: warning
+Info: The standard environment variable for build options is
+ DEB_BUILD_OPTIONS.  Usually, referring to DEB_BUILD_OPTS is a mistake and
+ DEB_BUILD_OPTIONS was intended instead.
+
 Tag: debian-rules-sets-DH_COMPAT
 Type: warning
 Ref: debhelper(7)
diff --git a/debian/changelog b/debian/changelog
index 91f5973..b151539 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -19,6 +19,8 @@ lintian (1.24.2) unstable; urgency=low
       quilt patches when using the 3.0 (quilt) package format.  Fix build
       dependency checking and check all dpatch files.  Patch from Raphaël
       Hertzog.  (Closes: #484549)
+  * checks/rules{,.desc}:
+    + [RA] Warn about use of DEB_BUILD_OPTS.  Patch from Raphael Geissert.
   * checks/scripts:
     + [FL] Apply File::Spec->canonpath() to rpaths to improve
       matching with directories.

-- 
Debian package checker


Reply to: