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

doc: Add NixOS to INSTALL.md #920

Merged
merged 2 commits into from
May 21, 2024
Merged

Conversation

lorenzleutgeb
Copy link
Contributor

@lorenzleutgeb lorenzleutgeb commented Sep 27, 2023

I am currently working on a NixOS module for BenchExec, stay tuned.

See: NixOS/nixpkgs#263375

@PhilippWendler
Copy link
Member

Nice! I won't be able to help with the NixOS side, but if you have any other questions, I would be glad to assist.

@lorenzleutgeb
Copy link
Contributor Author

The systemd unit file you and the docs on the cgroups setup you provide are very helpful already, thank you! If I run into issues, I'll contact you!

I am currently working on a NixOS module for BenchExec, stay tuned.
@lorenzleutgeb lorenzleutgeb marked this pull request as ready for review May 18, 2024 15:21
@lorenzleutgeb
Copy link
Contributor Author

lorenzleutgeb commented May 18, 2024

Downstream pull request NixOS/nixpkgs#263375 was merged. This means that the options mentioned in this PR are available in the "unstable" rolling release of NixOS shortly. Also, it was merged just in time for the NixOS 24.05 branch-off (see Release Schedule), which means that BenchExec and friends will appear in the upcoming stable release NixOS 24.05.

The current NixOS stable release 23.11 will reach its EOL in 2024-06, and all earlier stable releases are EOL already. Because of the relatively period left to 23.11 EOL I will not make any effort backporting BenchExec support to 23.11.

I don't see a big issue in the fact that the changes in this PR mention 24.05, because it will be released within a few weeks, and this PR still needs review.

Rendered view of the latest state of this PR: https://github.com/lorenzleutgeb/benchexec/blob/patch-1/doc/INSTALL.md#nixos

@PhilippWendler
Copy link
Member

Just to make sure, this is ready to merge from your point of view? Then I will do so.

@lorenzleutgeb
Copy link
Contributor Author

Just resolved the conflict with #1038 but good to merge from my side, yes!

@PhilippWendler PhilippWendler merged commit c393b74 into sosy-lab:main May 21, 2024
6 of 7 checks passed
@lorenzleutgeb
Copy link
Contributor Author

lorenzleutgeb commented May 21, 2024

Cool, thanks for this smooth merge! If you get reports about issues with the packages in Nixpkgs or NixOS modules (also for cpu-energy-meter and pqos_wrapper) feel free to ping me. I cannot guarantee any service level but am planning to maintain those on a best-effort basis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants