26. kesäkuuta 2024 2.20.19 GMT+03:00 "Miro Hrončok" <mhron...@redhat.com> kirjoitti: >On 25. 06. 24 22:50, Miroslav Suchý wrote: >> Dne 25. 06. 24 v 1:09 odp. Miro Hrončok napsal(a): >>> >>> Could you make the comment something like this? >>> >>> # Automatically converted from old format: GPLv2 >>> # TODO check if there are other licenses to be listed >>> License: GPL-2.0-only >> >> We (the Change owners) discussed this on a meeting today. And we agreed on >> output: >> >> # Automatically converted from old format: GPLv2 >> # TODO convert to correct SPDX identifier >> # See https://docs.fedoraproject.org/en-US/legal/update-existing-packages/ >> License: LicenseRef-Callaway-GPLv2 >> >> This is valid SPDX identifier. But not on the list of Fedora's allowed >> licenses, so any QA tool will remind you to check the license. >> >> What do you think? > >I don't understand what is the benefit of doing this at all. Sorry. >
At this point, I would just take GPL variants off the list of trivially convertible identifiers. If it takes a longish mailing list thread and a meeting, and the end result *still* contains a TODO comment asking for manual action, the conversion is not "trivial". Just file the Bugzillas and let maintainers eventually sort them out like the other tricky cases. If we go the LicenseRef route there is no need to restrict to GPL only. You can just convert every identifier in every package that does not have "SPDX conversion done" tag yet. I think that approach was discussed and rejected when SPDX migration was initially discussed? -- _______________________________________________ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
