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

Pin nixpkgs #88

Merged
merged 5 commits into from
Oct 24, 2023
Merged

Pin nixpkgs #88

merged 5 commits into from
Oct 24, 2023

Conversation

thiagokokada
Copy link
Owner

@thiagokokada thiagokokada commented Oct 24, 2023

This PR will pin nixpkgs in the generated default.nix files by extracting the repository information from nix-index-database's Flake metadata.

This should ensure that whatever package we find using nix-index will match the output files and make it just work(TM), avoiding issues where the user has a different <nixpkgs> set in their channels and the shell failing with errors because it can't find the specified package.

There are also some fixes, especially in the Flake/nix-ld codepaths (I think this mean they're not being used, since nobody reported issues and this was broken for a long time).

Fix #87.

This PR should fix #87 by ensuring that what database the
nix-index-database is pointing up is the nixpkgs version we will end up
using in the default.nix files.

For now this only changes the non-Flake version. The Flake ones will
come in another PR.
Since there is so many changes in nix-alien right now, this commit will
bump the UUID used as a base for hashing the names, so we can ensure
everyone will use the new changes.
@thiagokokada thiagokokada merged commit a72ce7c into master Oct 24, 2023
2 checks passed
@thiagokokada thiagokokada deleted the pin-nixpkgs branch October 24, 2023 17:31
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.

error somewhere in nix code when trying to run nix-alien
1 participant