Conversation
Duncaen
left a comment
There was a problem hiding this comment.
Changes need to be squashed into one commit and follow the commit title format: https://github.com/void-linux/void-packages/blob/096b05ebc9b25224ae0fd14b3bc83a0dc6c7f9f4/CONTRIBUTING.md#committing-your-changes.
| vlicense LICENSE | ||
|
|
||
| for shell in bash fish zsh; do | ||
| ${DESTDIR}/usr/bin/pixi completion -s ${shell} > pixi.${shell} |
There was a problem hiding this comment.
The binary can't be executed when cross compiling.
There was a problem hiding this comment.
How should I write for cross compiling?
There was a problem hiding this comment.
add build_helper="qemu"
and do vtargetrun "${DESTDIR}/usr/bin/pixi" ... see other templates like srcpkgs/ty/template.
srcpkgs/pixi/template
Outdated
| revision=1 | ||
| build_style=cargo | ||
| make_install_args="--path crates/pixi" | ||
| hostmakedepends="cargo-auditable cmake nushell pkg-config" |
There was a problem hiding this comment.
cargo-auditable should already be installed by default through build_style=cargo. Is nushell actually required to build it?
There was a problem hiding this comment.
Upon examining the codebase, nushell is not required for the build.
Removed 'cargo-auditable' and 'nushell' from hostmakedepends.
Testing the changes
New package
Local build testing