-
-
Notifications
You must be signed in to change notification settings - Fork 16.7k
64tass: init at 1.60.3243 #425210
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
base: master
Are you sure you want to change the base?
64tass: init at 1.60.3243 #425210
Conversation
pkgs/by-name/64/64tass/package.nix
Outdated
meta = { | ||
homepage = "https://tass64.sourceforge.net/"; | ||
description = "multi pass optimizing macro assembler for the 65xx series of processors"; | ||
license = lib.licenses.gpl2; |
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.
licenses.gpl#
are deprecated, choose between gpl#Only
or gpl#Plus
(the -or-later
one)
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'm not a license expert, can you advise on which one to choose?
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.
Look at the source.
- If the code files contain GNU GPL header (those licenses have specific headers), which has lines
under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.
, it's GPL-XXX-or-later. - Otherwise, if the build manifest (Pyproject, Cargo, Cabal, you name it) or another source of metadata (like those
metadata.xml
files) specifiesGPL-XXX-or-later
. - Otherwise, if metadata specifies
-only
, or if there's neither metadata nor headers at all, it's-only
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 believe I fixed it, there's quite a few licenses involved in the project, and I believe I did it correctly.
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.
What you said would be good for some nixpkgs docs
e8919d6
to
b92107e
Compare
Co-authored-by: Acid Bong <acidbong@tilde.club>
e6c16e9
to
2e5f32f
Compare
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.