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 licenses #45282
Add licenses #45282
Conversation
@@ -152,6 +152,7 @@ let | |||
meta = { | |||
homepage = https://www.perl.org/; | |||
description = "The standard implementation of the Perl 5 programmming language"; | |||
license = licenses.gpl1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per https://dev.perl.org/licenses/ you can choose gpl v1 or any later version. It is really gpl + clarifications by the author, however, so maybe consider using the Artistic License which makes those clarifications explicit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I make it just make it [ gpl1Plus artistic1 ]
? (Perl 6 would be artistic2
?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just use Artistic, because it expresses most clearly the intent of the author, but I'm happy to leave that to your judgment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To clarify, I'd be happy to leave it as is, too, I just wanted to mention this as a potential consideration, not a blocker.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see your point here. We can change it to Artistic.
Failure on aarch64-linux (full log) Attempted: pkgsCross.armv7l-hf-multiplatform.systemd Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.systemd Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: pkgsCross.aarch64-multiplatform.systemd Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: pkgsCross.armv7l-hf-multiplatform.systemd Partial log (click to expand)
|
Motivation for this change
Another round for #43716
Things done
Added license tags to:
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)