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

[lintian] 01/01: c/lintian: Deprecate --fail-on-warnings



This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository lintian.

commit c7d658fd7c45c605cc7c6e6b95393117b2f4d1f2
Author: Niels Thykier <niels@thykier.net>
Date:   Sat Dec 3 19:10:04 2016 +0000

    c/lintian: Deprecate --fail-on-warnings
    
    This is the foreshadowing of overhauling the exit code handling
    (among other aiming to fix #709932)
    
    Signed-off-by: Niels Thykier <niels@thykier.net>
---
 commands/lintian   | 10 ++++++++++
 debian/changelog   |  4 ++++
 man/lintian.pod.in |  2 ++
 3 files changed, 16 insertions(+)

diff --git a/commands/lintian b/commands/lintian
index c7b0389..fbbd676 100755
--- a/commands/lintian
+++ b/commands/lintian
@@ -229,6 +229,7 @@ Behaviour options:
     -E, --display-experimental display "X:" tags (normally suppressed)
     --no-display-experimental suppress "X:" tags
     --fail-on-warnings        return a non-zero exit status if warnings found
+                              (Deprecated)
     -i, --info                give detailed info about tags
     -I, --display-info        display "I:" tags (normally suppressed)
     -L, --display-level       display tags with the specified level
@@ -651,6 +652,10 @@ sub main {
 
     configure_output();
 
+    if ($opt{'fail-on-warnings'}) {
+        warning('--fail-on-warnings is deprecated');
+    }
+
     # check for arguments
     if (    $action =~ /^(?:check|unpack)$/
         and $#ARGV == -1
@@ -1298,6 +1303,11 @@ sub parse_config_file {
                       " - Using the first value!\n";
                     next;
                 }
+                if ($var eq 'fail-on-warnings') {
+                    print STDERR "The config option ${var} is deprecated\n";
+                    print STDERR
+                      " - Found in $opt{'LINTIAN_CFG'} (line: $.)\n";
+                }
                 $conf_opt{$var} = 1;
                 $found = 1;
                 # Translate boolean strings to "0" or "1"; ignore
diff --git a/debian/changelog b/debian/changelog
index 174aeea..c3912f4 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -34,6 +34,10 @@ lintian (2.5.50) UNRELEASED; urgency=medium
   * collection/objdump-info.desc:
     + [NT] Bump version due to RUNPATH collection.
 
+  * commands/lintian:
+    + [NT] Deprecate --fail-on-warnings, which will be removed in
+      Debian/buster.
+
   * data/binaries/embedded-libs:
     + [BR] Allow openssl1.0 as source package.  (Closes: #843406).
   * data/debhelper/dh_commands-manual:
diff --git a/man/lintian.pod.in b/man/lintian.pod.in
index 3a22ab8..dc848ff 100644
--- a/man/lintian.pod.in
+++ b/man/lintian.pod.in
@@ -223,6 +223,8 @@ warnings or errors are found.
 This option overrides the B<fail-on-warnings> variable in the
 configuration file.
 
+This option is deprecated and will be removed in the near future.
+
 =item B<-i>, B<--info>
 
 Print explanatory information about each problem discovered in

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: