[binutils-gdb] gdb/copyright.py: Do not forget to remind about MULTIPLE_COPYRIGHT_HEADERS

[Joel Brobecker <brobecke at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=0f0c98a8a1ca2fa2e73170849b1db4876c3ea954

commit 0f0c98a8a1ca2fa2e73170849b1db4876c3ea954
Author: Joel Brobecker <brobecker@adacore.com>
Date:   Mon Jan 1 08:30:28 2018 +0400

    gdb/copyright.py: Do not forget to remind about MULTIPLE_COPYRIGHT_HEADERS
    
    There is a small logical error in the part of the script that
    dumps the list of files in BY_HAND + MULTIPLE_COPYRIGHT_HEADERS
    but only checkis the contents of BY_HAND. The issue becomes
    apparent as soon as BY_HAND is empty. Prevent this from happening
    by treating the two lists separately, as this allows us to provide
    a more informative message in the case of MULTIPLE_COPYRIGHT_HEADERS.
    
    gdb/ChangeLog:
    
            * copyright.py (main): Dump the contents of
            MULTIPLE_COPYRIGHT_HEADERS (separately) from BY_HAND,
            even if BY_HAND is empty.

Diff:
---
 gdb/ChangeLog    |  6 ++++++
 gdb/copyright.py | 11 ++++++++++-
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 929c513..27f7c7f 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,11 @@
 2018-01-01  Joel Brobecker  <brobecker@adacore.com>
 
+	* copyright.py (main): Dump the contents of
+	MULTIPLE_COPYRIGHT_HEADERS (separately) from BY_HAND,
+	even if BY_HAND is empty.
+
+2018-01-01  Joel Brobecker  <brobecker@adacore.com>
+
 	* top.c (print_gdb_version): Update Copyright year in version
 	message.
 
diff --git a/gdb/copyright.py b/gdb/copyright.py
index dca2d56..bb21b21 100644
--- a/gdb/copyright.py
+++ b/gdb/copyright.py
@@ -150,11 +150,20 @@ def main ():
     update_files (update_list)
 
     # Remind the user that some files need to be updated by HAND...
+
+    if MULTIPLE_COPYRIGHT_HEADERS:
+        print
+        print("\033[31m"
+              "REMINDER: Multiple copyright headers must be updated by hand:"
+              "\033[0m")
+        for filename in MULTIPLE_COPYRIGHT_HEADERS:
+            print "  ", filename
+
     if BY_HAND:
         print
         print "\033[31mREMINDER: The following files must be updated by hand." \
               "\033[0m"
-        for filename in BY_HAND + MULTIPLE_COPYRIGHT_HEADERS:
+        for filename in BY_HAND:
             print "  ", filename
 
 ############################################################################