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
avocode: init at 2.26.0 #40425
avocode: init at 2.26.0 #40425
Conversation
comment = "The bridge between designers and developers"; | ||
}; | ||
|
||
dontBuild= 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.
space
buildInputs = [unzip]; | ||
nativeBuildInputs = [ makeWrapper ]; | ||
|
||
buildCommand = '' |
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.
Why use buildCommand
instead of the standard phases? You already set dontBuild = true
, so you may as well describe an installPhase
and a postFixup
.
@FRidh Thanks for feedback. Updated the PR. |
|
||
buildInputs = [ unzip ]; | ||
|
||
unpackCmd = '' |
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 for this because the builder already does that (though you might need fetchzip
instead of fetchurl
)
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.
fetchzip
and the normal unpackPhase
expect the unzip
command to produce a single directory. That is not the case for the sources of avocode.
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 see. That isn't clear from the code, so include a comment.
Motivation for this change
Add avocode package
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)