-
Notifications
You must be signed in to change notification settings - Fork 288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow alternative GPL-2.0/LGPL-2.0/LGPL-2.1 wording #2204
Comments
I just noticed there is similar problem with GPL-2.0-only and GPL-2.0-or-later where there is extra GNU GENERAL PUBLIC LICENSE at https://www.gnu.org/licenses/old-licenses/gpl-2.0.txt when compared to https://www.gnu.org/licenses/old-licenses/gpl-2.0-standalone.html. |
LGPL-2.0-only and LGPL-2.0-or-later are affected too. Please see https://www.gnu.org/licenses/old-licenses/lgpl-2.0-standalone.html and https://www.gnu.org/licenses/old-licenses/lgpl-2.0.txt. |
@mtelka - this is actually accommodated in markup, just via the alternative tag instead of the optional tag, see https://github.com/spdx/license-list-XML/blob/main/src/GPL-2.0-or-later.xml I checked this for I did make a PR to add the other FSF url, see #2244 - as I think having that is probably helpful. Could you make a PR to add the links for the other licenses? We only need to do so for the non-deprecated variants, that is, those ending with Thanks! |
Hm, yes, you are right. It is there in the Would you mind to change Thank you. |
the colored text on the html page is really there for human-readable convenience, with the actual markup in the XML file being more definitive. This is especially true for replaceable text where the html page can only show one set of text, not all the options. There are some reasons to use the tag instead of the tag for a situation like this where specific text could be there or not. Usually, that might be in the case where optional text is less likely to be there than nothing and having it "show" on the html page might be confusing. Obviously, there are no hard and fast rules on this and it's not a common scenario. That being said, I'm not sure if this case matters much how we code it, but I'd still like others to weigh in. @richardfontana @swinslow - I'm thinking you both might have some thoughts on this? |
I just noticed that similar issue is with ending comma here: https://github.com/spdx/license-list-XML/blob/main/src/GPL-2.0-or-later.xml#L25. It is marked in the xml file, but this marking is not visible in the rendered html file. Yes, html pages are for human-readable convenience, but since we are humans (at least I am :-)) I'd prefer to see this on the html page too. In this case it is hard to say what variant is less likely to be there, but if it is show in blue it cannot be confusing. For replaceable text it is not confusing, because it is clearly shown in red. So it is clear there could be variants. Again, not confusing. |
The https://github.com/spdx/license-list-XML/blob/main/src/LGPL-2.1.xml file is missing optional or alt "GNU LESSER GENERAL PUBLIC LICENSE" and it leads to https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt not matching LGPL-2.1. These should probably also include the optional "GNU LESSER GENERAL PUBLIC LICENSE" but currently don't include it: |
this issue has been resolved via another similar issue and PR |
The LGPL-2.1 wording available at LGPL-2.1-only and LGPL-2.1-or-later is based on officially published LGPL-2.1 by FSF. Unfortunately, there is another slightly different LGPL-2.1 text available at gnu.org.
The difference is this:
A lot of software distributed on internet and licensed under LGPL-2.1 includes the plain text version of the license and so they contain extra GNU LESSER GENERAL PUBLIC LICENSE words in their license text. This causes problem for automatic license matching because SPDX does not show the GNU LESSER GENERAL PUBLIC LICENSE words there as omitable (in blue).
Could you please add GNU LESSER GENERAL PUBLIC LICENSE before the TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION and mark it as omitable (blue)?
Thank you.
The text was updated successfully, but these errors were encountered: