Cleanup: Replace license headers with SPDX identifiers#406
Cleanup: Replace license headers with SPDX identifiers#406kaechele wants to merge 2 commits intoOpenTTD:masterfrom
Conversation
Signed-off-by: Felix Kaechele <felix@kaechele.ca>
Rito13
left a comment
Rito13
left a comment
There was a problem hiding this comment.
NML is part of OpenTTD project, please make sure that your PR does not violate the CONTRIBUTING.md. It would be a shame, if it did.
nml/_lz77.c
Outdated
| * with NML; if not, write to the Free Software Foundation, Inc., | ||
| * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. | ||
| */ | ||
| // SPDX-License-Identifier: GPL-2.0-or-later |
There was a problem hiding this comment.
Pushed a fix. Thanks for catching that.
| You should have received a copy of the GNU General Public License along | ||
| with NML; if not, write to the Free Software Foundation, Inc., | ||
| 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.""" | ||
| # SPDX-License-Identifier: GPL-2.0-or-later |
There was a problem hiding this comment.
Doesn't that violate the license?
attach the following notices to the program. It is safest
to attach them to the start of each source file to most effectively
convey the exclusion of warranty; and each file should have at least
the "copyright" line and a pointer to where the full notice is found.<one line to give the program's name and a brief idea of what it does.> Copyright (C) <year> <name of author> This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version. This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with this program; if not, see <https://www.gnu.org/licenses/>.
NML contains the first two lines in the third one:
NML is free software; you can redistribute it and/or modify
There was a problem hiding this comment.
It does not violate the license. Your quote above is missing some context, namely a line that's a few lines up:
END OF TERMS AND CONDITIONS
The part you quoted is not part of the license, so not following it to the letter does not violate said license.
There was a problem hiding this comment.
Sorry, I just don't see the point of using SPDX-License-Identifier.
With the old solution the main part license was accessible through the python embedded help method, which is very helpful in interactive python. With your suggestion it just states which license is used.
Also for python I would opt for using """ ... """ instead of # ... as it makes it more clear that it is a description and not simple comment.
There was a problem hiding this comment.
Ultimately, it's a matter of preference. I'm happy to provide a PR that just changes the address, if that is what maintainers prefer.
Signed-off-by: Felix Kaechele <felix@kaechele.ca>
|
Thanks for your review and comments, @Rito13. Just to add, I can attest that this PR is in compliance with the contribution guidelines. I appreciate the spirit of the AI rule and also personally agree with it. In full transparency: The Copilot Code Review was obviously AI generated, but this must be some new default setting that I didn't call for. I'll see to fix that. |
License headers contain the old FSF address, replace them with SPDX headers.
The SPDX License Identifier headers provide a compact and unambiguous way to identify the license applicable to each file.
I also updated the LICENSE file with the latest version available from https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt