GNU bug report logs - #72809
ms-windows: Copyright embedded into executable is outdated

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 72809 in the body.
You can then email your comments to 72809 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Cecilio Pardo <cpardo <at> imayhem.com>
To: bug-gnu-emacs <at> gnu.org
Subject: ms-windows: Copyright embedded into executable is outdated
Date: Sun, 25 Aug 2024 21:38:44 +0200

The copyright text included in nt/emacs.rc.in is not updated:

  VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"

Message #10 received at 72809-done <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Cecilio Pardo <cpardo <at> imayhem.com>, 72809-done <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Mon, 26 Aug 2024 00:50:48 +0000

Version: 30.1

Cecilio Pardo <cpardo <at> imayhem.com> writes:

> The copyright text included in nt/emacs.rc.in is not updated:
>
>    VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"

Thanks, fixed on emacs-30 (commit aa6ed7f67ec).

Message #13 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: cpardo <at> imayhem.com, 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Thu, 29 Aug 2024 15:03:49 +0300

> Resent-To: bug-gnu-emacs <at> gnu.org
> From: Stefan Kangas <stefankangas <at> gmail.com>
> Date: Mon, 26 Aug 2024 00:50:48 +0000
> 
> Version: 30.1
> 
> Cecilio Pardo <cpardo <at> imayhem.com> writes:
> 
> > The copyright text included in nt/emacs.rc.in is not updated:
> >
> >    VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"
> 
> Thanks, fixed on emacs-30 (commit aa6ed7f67ec).

Should we perhaps fix the script not to miss these files?

Message #16 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: stefankangas <at> gmail.com
Cc: 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Sat, 14 Sep 2024 10:30:17 +0300

> Cc: cpardo <at> imayhem.com, 72809 <at> debbugs.gnu.org
> Date: Thu, 29 Aug 2024 15:03:49 +0300
> From: Eli Zaretskii <eliz <at> gnu.org>
> 
> > Resent-To: bug-gnu-emacs <at> gnu.org
> > From: Stefan Kangas <stefankangas <at> gmail.com>
> > Date: Mon, 26 Aug 2024 00:50:48 +0000
> > 
> > Version: 30.1
> > 
> > Cecilio Pardo <cpardo <at> imayhem.com> writes:
> > 
> > > The copyright text included in nt/emacs.rc.in is not updated:
> > >
> > >    VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"
> > 
> > Thanks, fixed on emacs-30 (commit aa6ed7f67ec).
> 
> Should we perhaps fix the script not to miss these files?

Stefan, what are your thoughts about improving the script?  Do you
understand why these files were missed?

Message #19 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Sat, 14 Sep 2024 06:28:15 -0700

Eli Zaretskii <eliz <at> gnu.org> writes:

> Stefan, what are your thoughts about improving the script?  Do you
> understand why these files were missed?

The simple reason is that the Gnulib `build-aux/update-copyright` script
don't support them, so we need to remember to update them manually.

As for why this is, the cases are different.

1. There's more than one copyright year in these files:

    configure.ac
    doc/lispref/tips.texi
    etc/images/gud/README

   However, the Gnulib script only updates the first copyright year.

2. For these files:

    nt/emacs.rc.in
    nt/emacsclient.rc.in

   the Gnulib script does not support making this update:

   -	    VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"
   +	    VALUE "LegalCopyright", "Copyright (C) 2001-2024\0"

3. This file:

    test/lisp/which-key-tests.el

   was added with the wrong copyright in 2024.

There are even more edge cases, see for example commit 33d159c36f42.

I'm not sure if it's worth updating our script to support all this.
It's not too hard to remember to grep for "202[0-9]" and updating them
manually.  Writing code to do this has a maintenance cost too, and also
runs a risk of missing some years.  Files change, new files are added,
etc.  So there's no way around grepping to make sure, I think.

Thus, perhaps we should just do this:

diff --git a/admin/update-copyright b/admin/update-copyright
index b9007100b35..23b78443ca4 100755
--- a/admin/update-copyright
+++ b/admin/update-copyright
@@ -77,3 +77,6 @@ updatable_files=
   -print) &&

 build-aux/update-copyright $updatable_files
+
+printf "!! Please remember to grep for and manually update any"
+printf "!! copyright years that were not updated by this script."

Message #22 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Sat, 14 Sep 2024 17:02:47 +0300

> From: Stefan Kangas <stefankangas <at> gmail.com>
> Date: Sat, 14 Sep 2024 06:28:15 -0700
> Cc: 72809 <at> debbugs.gnu.org
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > Stefan, what are your thoughts about improving the script?  Do you
> > understand why these files were missed?
> 
> The simple reason is that the Gnulib `build-aux/update-copyright` script
> don't support them, so we need to remember to update them manually.
> 
> As for why this is, the cases are different.
> 
> 1. There's more than one copyright year in these files:
> 
>     configure.ac
>     doc/lispref/tips.texi
>     etc/images/gud/README
> 
>    However, the Gnulib script only updates the first copyright year.
> 
> 2. For these files:
> 
>     nt/emacs.rc.in
>     nt/emacsclient.rc.in
> 
>    the Gnulib script does not support making this update:
> 
>    -	    VALUE "LegalCopyright", "Copyright (C) 2001-2021\0"
>    +	    VALUE "LegalCopyright", "Copyright (C) 2001-2024\0"
> 
> 3. This file:
> 
>     test/lisp/which-key-tests.el
> 
>    was added with the wrong copyright in 2024.
> 
> There are even more edge cases, see for example commit 33d159c36f42.
> 
> I'm not sure if it's worth updating our script to support all this.
> It's not too hard to remember to grep for "202[0-9]" and updating them
> manually.  Writing code to do this has a maintenance cost too, and also
> runs a risk of missing some years.  Files change, new files are added,
> etc.  So there's no way around grepping to make sure, I think.
> 
> Thus, perhaps we should just do this:
> 
> diff --git a/admin/update-copyright b/admin/update-copyright
> index b9007100b35..23b78443ca4 100755
> --- a/admin/update-copyright
> +++ b/admin/update-copyright
> @@ -77,3 +77,6 @@ updatable_files=
>    -print) &&
> 
>  build-aux/update-copyright $updatable_files
> +
> +printf "!! Please remember to grep for and manually update any"
> +printf "!! copyright years that were not updated by this script."

That's okay, but could you also update admin/notes/years?  That file
seems to be severely outdated (e.g., it mentions files we no longer
have), and I think it would be good to add what you say here to the
text, including the list of files that need manual updates.

Thanks.

Message #25 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Sat, 14 Sep 2024 10:41:21 -0500

Eli Zaretskii <eliz <at> gnu.org> writes:

>> diff --git a/admin/update-copyright b/admin/update-copyright
>> index b9007100b35..23b78443ca4 100755
>> --- a/admin/update-copyright
>> +++ b/admin/update-copyright
>> @@ -77,3 +77,6 @@ updatable_files=
>>    -print) &&
>>
>>  build-aux/update-copyright $updatable_files
>> +
>> +printf "!! Please remember to grep for and manually update any"
>> +printf "!! copyright years that were not updated by this script."
>
> That's okay, but could you also update admin/notes/years?  That file
> seems to be severely outdated (e.g., it mentions files we no longer
> have), and I think it would be good to add what you say here to the
> text, including the list of files that need manual updates.

Done.

The only non-existent file I could find in notes/years was that XPM
image files no longer have a copyright year since commit b5b59d1b8af.
If you spot any others, please amend what I pushed.

Message #28 received at 72809 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 72809 <at> debbugs.gnu.org
Subject: Re: bug#72809: ms-windows: Copyright embedded into executable is
 outdated
Date: Sat, 14 Sep 2024 19:04:48 +0300

> From: Stefan Kangas <stefankangas <at> gmail.com>
> Date: Sat, 14 Sep 2024 10:41:21 -0500
> Cc: 72809 <at> debbugs.gnu.org
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > That's okay, but could you also update admin/notes/years?  That file
> > seems to be severely outdated (e.g., it mentions files we no longer
> > have), and I think it would be good to add what you say here to the
> > text, including the list of files that need manual updates.
> 
> Done.
> 
> The only non-existent file I could find in notes/years was that XPM
> image files no longer have a copyright year since commit b5b59d1b8af.
> If you spot any others, please amend what I pushed.

Thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.