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
build-idris-package: add ipkgName #43735
Conversation
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.
Nice! Maybe in the future some more logic should be implemented to handle multiple packages/ipkgs per source, I'm not 100% sure how it's all supposed to work just yet (and test ipkgs for that matter, I think there's a new tests =
attribute in ipkg files which replaces test.ipkg's)
Some idris packages currently use rm something.ipkg
in the postUnpack
phase to not build certain ipkgs, can you adjust those to use ipkgName
instead?
Alright, if we want to adjust existing packages, a better/cleaner way would be to set |
Yeah that sounds better |
726bb41
to
260938c
Compare
@infinisil alright, added |
260938c
to
7e0f7a0
Compare
@GrahamcOfBorg build idrisPackages.lightyear idrisPackages.http |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: idrisPackages.lightyear, idrisPackages.http Partial log (click to expand)
|
@GrahamcOfBorg idrisPackages.bi idrisPackages.electron idrisPackages.hamt |
Success on x86_64-darwin (full log) Attempted: idrisPackages.lightyear, idrisPackages.http Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: idrisPackages.lightyear, idrisPackages.http Partial log (click to expand)
|
Not sure why it didn't build those, let's try again @GrahamcOfBorg idrisPackages.bi idrisPackages.electron idrisPackages.hamt |
@infinisil I did a test build with all the packages included, and they managed to find the respective .ipkg files |
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.
Ah nice, thanks
Motivation for this change
Add the following attributes to
build-idris-package
:ipkgName
: specify base name of the ipkg file explicitly, since*.ipkg
fails if there are more than 1 ipkg files in the source root directory, defaults to*
as beforeextraInstallPhase
: extra commands for theinstallPhase
to be able to install executables, sinceidris --install
only install libsThings done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @infinisil