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
meli: init at pre-alpha-0.3.2 #69169
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.
Builds and runs with nix-review on nixos
Did you mean to request me to review this? |
@GrahamcOfBorg build meli |
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.
Doesn't seem to build on aarch so maybe platforms
should be changed
@alexarice Should now build and run on aarch64. |
@GrahamcOfBorg build meli |
@alexarice did you test the program locally? |
I think so but I'll do it again now |
Build fails with:
|
@0x4A6F please ff-pull from https://github.com/matthiasbeyer/nixpkgs/tree/master-meli - I fixed the hashes for the derivation. Though, my build still fails, with:
|
The source branch aarch64 vanished. @matthiasbeyer Your commit is similar to "meli: init at pre-alpha-0.3.2", sans cargoPatches. We might wait for the next release and thus get rid of cargoPatches and have aarch64 support? |
Closing in favor of #76519. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @