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

Nix flakes #247

Merged
merged 1 commit into from
Jul 2, 2024
Merged

Nix flakes #247

merged 1 commit into from
Jul 2, 2024

Conversation

InfiniteCoder01
Copy link
Contributor

Adds flake for nix package manager

@InfiniteCoder01
Copy link
Contributor Author

Note: flake.lock should be present in the repository for better reproduceability

@InfiniteCoder01 InfiniteCoder01 force-pushed the main branch 2 times, most recently from 67a6e7d to 97fce40 Compare June 20, 2024 15:02
@InfiniteCoder01
Copy link
Contributor Author

Please, review

@MihailRis
Copy link
Owner

  • What is it for (in the project context) and is this worth further support?
  • How does it work with dependencies (especially - entt)?

@InfiniteCoder01
Copy link
Contributor Author

  1. Nix - cross-platform package management solution (and only way to manage packages on NixOS). Works by specifying list of dependencies and tools required to maintain a piece of software, locking them for reproducibility.
  2. All dependencies (including enTT, which is already in) are specified in flake.nix file. Dependencies that are not packaged with nix packages (that's rare) can be imported as other flakes straight from github. If there is no existing nix solution for a certain dependency, it's possible to package it manually, as Nix is a functional programming language.

@MihailRis MihailRis merged commit 9472383 into MihailRis:main Jul 2, 2024
4 checks passed
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.
2 participants
@InfiniteCoder01 @MihailRis