-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
No make unittests #11301
No make unittests #11301
Conversation
Now that we can run all tests with Meson, we want developers making code changes to use it. (Only the manual needs to be built with the build system, and that will change shortly.) This reverts commit b0bc2a9.
d52ccf6
to
b8b679c
Compare
@@ -222,7 +222,7 @@ | |||
# for which we don't apply the full build matrix such as cross or static. | |||
inherit (nixpkgsFor.${system}.native) | |||
changelog-d; | |||
default = self.packages.${system}.nix; | |||
default = self.packages.${system}.nix-ng; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good because it preserves the property that nix build .
only works if the tests pass.
14d0ae1
to
ec4d347
Compare
This ensures just `nix build`-ing the flake doesn't forget to run all tests. One can still specifiy specific attributes to just build one thing. Co-authored-by: Robert Hensing <[email protected]>
7964c83
to
b41cc1a
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2024-08-14-nix-team-meeting-minutes-169/50633/1 |
Ehm, how am I supposed to run unit tests now? Please revert, removing make functionality seems premature. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2024-08-19-nix-team-meeting-minutes-170/50942/1 |
Motivation
Now that we can run all tests with Meson, we want developers making code changes to use it.
(Only the manual needs to be built with the build system, and that will change shortly.)
This reverts commit b0bc2a9.
Context
#2503
Priorities and Process
Add 👍 to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.