public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r13-5187] contrib: Partial fix for failed update-copyright --this year [PR108413]
@ 2023-01-16 10:04 Jakub Jelinek
  0 siblings, 0 replies; only message in thread
From: Jakub Jelinek @ 2023-01-16 10:04 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:d1c6a352cad473edbacecad5ceb49eaaa4e67a52

commit r13-5187-gd1c6a352cad473edbacecad5ceb49eaaa4e67a52
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Mon Jan 16 11:03:30 2023 +0100

    contrib: Partial fix for failed update-copyright --this year [PR108413]
    
    As mentioned on IRC or in PR108413, the last update-copyright.py --this year
    failed and that is why we are in a strange state where some copyrights have
    been updated and others have not.
    The full list of errors I got was I think:
    gcc/m2/mc-boot/GmcOptions.c: unrecognised copyright: comment (f, (const char *) "Copyright (C) ''2021'' Free Software Foundation, Inc.", 53);
    gcc/m2/mc-boot/GmcOptions.c: unrecognised copyright: comment (f, (const char *) "Copyright (C) ''2021'' Free Software Foundation, Inc.", 53);
    gcc/testsuite/gm2/switches/pedantic-params/pass/Strings.mod: unrecognised copyright holder: Faculty of Information Technology,
    gcc/testsuite/gm2/switches/pedantic-params/pass/Strings2.mod: unrecognised copyright holder: Faculty of Information Technology,
    libphobos/libdruntime/__builtins.di: unrecognised copyright: * Copyright: Copyright Digital Mars 2022
    libstdc++-v3/src/c++17/fast_float/fast_float.h: unrecognised copyright holder: The fast_float authors
    libstdc++-v3/include/c_compatibility/stdatomic.h: unrecognised copyright holder: The GCC developers
    
    The following patch deals with the gcc/testsuite/gm2 ones and
    with the fast_float.h one, ok for trunk?
    
    Not really sure what we should do in the GmcOptions.c case
    (perhaps obfuscate it in the source somehow by splitting
    the string literals into different substrings
    Perhaps "Copy" "right (" "C) ''..." would do it?  Or do we want
    to bump there each year (manually or by the script)?
    E.g. in gcc.cc we have
          printf ("Copyright %s 2023 Free Software Foundation, Inc.\n",
                  _("(C)"));
    which also prints (C) nicer in Unicode if possible and is updated
    by hand each year.
    
    I have no idea about the libphobos case, we have tons of
    libphobos/src/std/format/spec.d:Copyright: Copyright The D Language Foundation 2000-2013.
    libphobos/src/std/random.d:Copyright: Copyright Andrei Alexandrescu 2008 - 2009, Joseph Rushton Wakeling 2012.
    etc. lines and those aren't reported as errors.
    
    And the last one is that I think for The GCC developers we should treat it
    similarly like FSF and bump copyright on it.
    Would
            canon_gcc = 'The GCC developers'
            self.add_package_author ('The GCC developers', canon_gcc)
            self.add_package_author ('The GCC Developers', canon_gcc)
    or something similar do the trick?
    
    2023-01-16  Jakub Jelinek  <jakub@redhat.com>
    
            PR other/108413
            * update-copyright.py (TestsuiteFilter): Add .mod and .rs extensions.
            (GCCCopyright): Add 'The fast_float authors' as external author.

Diff:
---
 contrib/update-copyright.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/contrib/update-copyright.py b/contrib/update-copyright.py
index 06e6fb61757..529032c677b 100755
--- a/contrib/update-copyright.py
+++ b/contrib/update-copyright.py
@@ -596,6 +596,8 @@ class TestsuiteFilter (GenericFilter):
                 '.go',
                 '.inc',
                 '.java',
+                '.mod',
+                '.rs'
                 ])
 
     def skip_file (self, dir, filename):
@@ -733,6 +735,7 @@ class GCCCopyright (Copyright):
         self.add_external_author ('Stephen L. Moshier')
         self.add_external_author ('Sun Microsystems, Inc. All rights reserved.')
         self.add_external_author ('The D Language Foundation, All Rights Reserved')
+        self.add_external_author ('The fast_float authors')
         self.add_external_author ('The Go Authors.  All rights reserved.')
         self.add_external_author ('The Go Authors. All rights reserved.')
         self.add_external_author ('The Go Authors.')

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-01-16 10:04 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-16 10:04 [gcc r13-5187] contrib: Partial fix for failed update-copyright --this year [PR108413] Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).