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

add nix derivation #39

Merged
merged 1 commit into from
Jan 16, 2021
Merged

add nix derivation #39

merged 1 commit into from
Jan 16, 2021

Conversation

wucke13
Copy link
Contributor

@wucke13 wucke13 commented Jan 16, 2021

This allows nix users to easily run the scripts on systems with nix

This is only a quick hack. Ideally, we would make this more idiomatic: All supplied binaries go in a bin folder, this folder is added to the $PATH in the beginning of both script. A env var could be used to allow both nix and the user to prefer system binaries instead of those shipped with flash-multi.

Any feedback welcome :)

This allows nix users to easily run the scripts on systems with nix
@benlye benlye merged commit de836e1 into benlye:master Jan 16, 2021
@benlye
Copy link
Owner

benlye commented Jan 16, 2021

I've never heard of nix before but it looks neat. Happy to merge this as-is and improve it later :-)

@wucke13
Copy link
Contributor Author

wucke13 commented Jan 16, 2021

That is excellent news. Well, I can recommend you take a look at it, it many of todays problems in software distribution and dependency management. However, it does so at the cost of most innovative technologies, it's niche.

@wucke13 wucke13 deleted the dev/wucke13/add-nix branch January 16, 2021 16:30
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