Skip to content
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

feat(cli): Allow --override-input to refer to flake name without flake/ prefix of devour_flake #74

Merged
merged 7 commits into from
Jul 9, 2024

Conversation

shivaraj-bh
Copy link
Contributor

resolves #71

@shivaraj-bh shivaraj-bh force-pushed the preprocess-extra-args branch from 55b9c8a to 8133612 Compare July 9, 2024 12:15
@shivaraj-bh shivaraj-bh changed the title feat(api): Preprocess extra nix build args feat(cli): Preprocess extra nix build args Jul 9, 2024
srid added 5 commits July 9, 2024 10:11
The rest of the code, outside of devour_flake.rs, does not need to know about this logic.
Make it happen in the same place, so code remains readable over time.
Copy link
Owner

@srid srid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did some refactor; see commit history for explanation.

Tested it by running ~/code/nixci/result/bin/nixci build -- --override-input ema github:srid/ema on Emanote, which generates:

🐚 nix build '/nix/store/66a9w28f0yrgdy5zgwf8wnxp0n22iahc-source#default' -L --no-link --print-out-paths --override-input flake . --override-input systems github:nix-systems/empty --override-input flake/ema github:srid/ema️

@srid srid changed the title feat(cli): Preprocess extra nix build args feat(cli): Allow --override-input to refer to flake name without flake/ prefix of devour_flake Jul 9, 2024
@srid srid merged commit c17f42f into srid:master Jul 9, 2024
4 checks passed
@shivaraj-bh shivaraj-bh deleted the preprocess-extra-args branch July 9, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preprocess --override-inputs arguments
2 participants