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 package management #772

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Conversation

bclement-ocp
Copy link
Collaborator

This patch adds support for developing and building alt-ergo using nix.

Usage:

  • nix-shell creates a new development shell that can be used to develop alt-ergo in a standard way using dune

  • nix-run -f . alt-ergo to build and run alt-ergo using nix

Dependencies are managed using niv.

@Halbaroth Halbaroth self-assigned this Aug 10, 2023
@Halbaroth Halbaroth added this to the 2.6.0 milestone Aug 10, 2023
Copy link
Collaborator

@Halbaroth Halbaroth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

We could add a bit of documentation about installation with nix?

This patch adds support for developing and building alt-ergo using nix.

Usage:
 - `nix-shell` creates a new development shell that can be used to
   develop alt-ergo in a standard way using `dune`

 - `nix run -f . alt-ergo` to build and run alt-ergo using nix

Dependencies are managed using [niv](nmattia/niv).
@bclement-ocp
Copy link
Collaborator Author

I don't know if there is anything specific to advertise, default.nix and shell.nix are standard entry points for nix projects.

Further, this is more for development purposes, for users Alt-Ergo is already packaged in nixpkgs. I have added a short paragraph to CONTRIBUTING.md.

@bclement-ocp bclement-ocp enabled auto-merge (squash) August 10, 2023 12:18
@bclement-ocp bclement-ocp merged commit 01b55a6 into OCamlPro:next Aug 10, 2023
8 checks passed
@bclement-ocp bclement-ocp deleted the bclement/niv branch January 3, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants