Mailing-List: contact commits-help@subversion.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@subversion.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1800547 - /subversion/trunk/tools/dist/release.py
Date: Sun, 02 Jul 2017 05:59:04 -0000
To: commits@subversion.apache.org
From: danielsh@apache.org
Message-Id: <20170702055906.E7DA03A00E6@svn01-us-west.apache.org>
archived-at: Sun, 02 Jul 2017 05:59:09 -0000

Author: danielsh
Date: Sun Jul  2 05:59:04 2017
New Revision: 1800547

URL: http://svn.apache.org/viewvc?rev=1800547&view=rev
Log:
release.py: Revert r934990.

* tools/dist/release.py
  (get_siginfo): Follow best practice: Don't ignore arbitrary warnings.

Modified:
    subversion/trunk/tools/dist/release.py

Modified: subversion/trunk/tools/dist/release.py
URL: http://svn.apache.org/viewvc/subversion/trunk/tools/dist/release.py?rev=1800547&r1=1800546&r2=1800547&view=diff
==============================================================================
--- subversion/trunk/tools/dist/release.py (original)
+++ subversion/trunk/tools/dist/release.py Sun Jul  2 05:59:04 2017
@@ -875,8 +875,14 @@ def get_siginfo(args, quiet=False):
             stderr=subprocess.STDOUT,
         )
 
-        gpg_output = "\n".join([ l for l in gpg_output.splitlines()
-                                                     if l[0:7] != 'Warning' ])
+        # This code was added in r934990, but there was no comment (nor log
+        # message text) explaining its purpose.  I've commented it out since
+        # ignoring arbitrary warnings in a verification codepath is Bad.  If
+        # you run into warnings on your machine, feel free to uncomment it,
+        # but when you do so please make it match specific warnings only.
+        #
+        #gpg_output = "\n".join([ l for l in gpg_output.splitlines()
+        #                                             if l[0:7] != 'Warning' ])
 
         match = fp_pattern.match(gpg_output)
         format_expandos = dict(