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.

Sign up for GitHub

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

Jump to bottom

fix flake reference for nix-systems #24

Merged
merged 1 commit into from
Jun 25, 2024
Merged

fix flake reference for nix-systems #24

merged 1 commit into from
Jun 25, 2024

Conversation

eclairevoyant
Copy link
Contributor

@eclairevoyant eclairevoyant commented Jun 25, 2024 •
edited
Loading

@LovingMelody
Copy link
Owner

Thank you for catching this, not sure why nix flake update changed my nix path to the store, I'll merge the PR once I confirm it passes.

@LovingMelody LovingMelody merged commit 1b1d304 into LovingMelody:main Jun 25, 2024
2 of 5 checks passed
@LovingMelody
Copy link
Owner

Thanks for your contribution

@eclairevoyant
Copy link
Contributor Author

It's because the flake used systems in outputs:

nix-citizen/flake.nix

Line 22 in 1b1d304

{ self, systems, ... }@inputs:

and as it wasn't listed in the inputs, it pulled from the flake registry, which could literally be anything (including a store path)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Build failure: flake.nix
2 participants
@eclairevoyant @LovingMelody