-
Notifications
You must be signed in to change notification settings - Fork 161
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
Add License
field to PackageInfo.g format
#997
Comments
Shall we try this for GAP 4.9.0? Then packages updating for GAP 4.9.1 would be able to make use of this. |
A pull request for this would be welcome at any time. |
So, which of my several proposals do you want to implement? For your plan: Perhaps also notify package authors to add the field, and which values are permissible? This notification could be sent as soon as the master branch implements this, no need to wait for 4.9 (in particular since adding this field will not break compatibility with old GAP versions). |
@fingolfin I like the 3rd, both with the file and SPDX id. Agree that we don't actually have to wait for the beta to notify package authors. |
I support the idea of adding these field(s). |
While this would be nice to have, I don't think it's mandatory for 4.9.0, hence I just removed the milestone. Perhaps we can try to get this into 4.10.0. |
I really want to get this done in GAP 4.11. I suggest that we use approach 2., a License field using an SPDX identifier (see https://spdx.org). We can then still add a |
@fingolfin this can be closed - all boxes now ticked (one of them transferred to gap-system/GitHubPagesForGAP#17) |
(I thought I'd seen this discussed before, but didn't see an issue, so I am opening a new one. If there is already an issue, feel free to close this one with a pointer to the existing issue. Thanks)
We should add a
License
field to GAP packages. It would initially be optional, but on the long run mandatory. Several variants for the content of that file come to mind:License
field with the license name (SPDX license expression), and aLicenseFile
field with file path.UPDATE: for now we have settled on variant 2, and here is a "roadmap" suggested by @alex-konovalov
ValidatePackageInfo
(see Teach ValidatePackageInfo about the optional License field #3286)optionally: update GitHubPagesForGAP to render info on License, if present(transferred to Extract license fromPackageInfo.g
, and show it to user (if present) GitHubPagesForGAP#17)The text was updated successfully, but these errors were encountered: