这是indexloc提供的服务,不要输入任何密码
Skip to content

add nix flake #57

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

Merged
merged 1 commit into from
Mar 16, 2025
Merged

add nix flake #57

merged 1 commit into from
Mar 16, 2025

Conversation

itepastra
Copy link
Contributor

Add the approxmc package build from ganak to a flake in this repository.
Depends on

@msoos
Copy link
Collaborator

msoos commented Mar 16, 2025

Wow, you have done so much work! Thank you so much. I am now adjusting all of these to point to meelgroup and the relevant branches.

@itepastra
Copy link
Contributor Author

don't forget to regenerate the lockfiles as well using nix flake update after modifying the branch names, if you add --commit-lock-file it will automatically create a commit as well

@msoos
Copy link
Collaborator

msoos commented Mar 16, 2025

Ohhh so that's how it's done! I have always been just checking the error and copy-pasting the hash it calculated :D Thanks for that pro tip! Sorry, my nix/flake knowledge is pretty bad. Thanks, this helps a lot!

@itepastra
Copy link
Contributor Author

In the old version with the src = fetchFromGitHub that was indeed how you'd do that. But that's a thing flakes were made to improve the experience of in the first place since now the flake always has the source since it's in the same git repo. The dependencies just fetch a specific git commit and nix flake update updates those to the newest version.

@itepastra
Copy link
Contributor Author

itepastra commented Mar 16, 2025

I may decide in the future to use inputs.{name}.follows = {name} for more parts so running nix flake update on for example ganak will also update all the transitive dependencies instead of only the direct ones and the versions will be the same within the build itself

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.

2 participants