-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
espresso: init at 2.4 #272597
espresso: init at 2.4 #272597
Conversation
50adead
to
fe00295
Compare
Added sourceProvenance. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3203 |
pkgs/by-name/es/espresso/package.nix
Outdated
owner = "chipsalliance"; | ||
repo = pname; | ||
rev = "v${version}"; | ||
sha256 = "sha256-z5By57VbmIt4sgRgvECnLbZklnDDWUA6fyvWVyXUzsI="; |
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.
sha256 = "sha256-z5By57VbmIt4sgRgvECnLbZklnDDWUA6fyvWVyXUzsI="; | |
hash = "sha256-z5By57VbmIt4sgRgvECnLbZklnDDWUA6fyvWVyXUzsI="; |
pkgs/by-name/es/espresso/package.nix
Outdated
version = "2.4"; | ||
src = fetchFromGitHub { | ||
owner = "chipsalliance"; | ||
repo = pname; |
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.
repo = pname; | |
repo = "espresso"; |
pkgs/by-name/es/espresso/package.nix
Outdated
sha256 = "sha256-z5By57VbmIt4sgRgvECnLbZklnDDWUA6fyvWVyXUzsI="; | ||
}; | ||
|
||
nativeBuildInputs = [ cmake ninja ]; |
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.
nativeBuildInputs = [ cmake ninja ]; | |
nativeBuildInputs = [ cmake ]; |
ninja
is only faster on incremental builds.
pkgs/by-name/es/espresso/package.nix
Outdated
|
||
nativeBuildInputs = [ cmake ninja ]; | ||
|
||
enableParallelBuilding = true; |
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.
enableParallelBuilding = true; |
cmake
already sets this by default.
pkgs/by-name/es/espresso/package.nix
Outdated
passthru.updateScript = nix-update-script { }; | ||
|
||
meta = with lib;{ | ||
description = "Espresso is a Multi-valued PLA minimization."; |
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.
description = "Espresso is a Multi-valued PLA minimization."; | |
description = "Multi-valued PLA minimization"; |
pkgs/by-name/es/espresso/package.nix
Outdated
maintainers = with maintainers;[ pineapplehunter ]; | ||
mainProgram = "espresso"; | ||
platforms = lib.platforms.all; | ||
sourceProvenance = with lib.sourceTypes;[ fromSource ]; |
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.
No need to mark sourceProvenance explicitly for source builds.
sourceProvenance = with lib.sourceTypes;[ fromSource ]; |
fe00295
to
7ebba97
Compare
@NickCao Thanks for the review! |
Result of 2 packages built:
|
Description of changes
The license is set ro unfree, since the source on github does not provide a licence. See: chipsalliance/espresso#4
cc: @sequencer
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.