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

Package the Nix CLI with Meson #11060

Merged
merged 1 commit into from
Jul 9, 2024
Merged

Conversation

Ericson2314
Copy link
Member

@Ericson2314 Ericson2314 commented Jul 8, 2024

Motivation

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@github-actions github-actions bot added documentation new-cli Relating to the "nix" command labels Jul 8, 2024
@Ericson2314 Ericson2314 force-pushed the meson-nix branch 2 times, most recently from 52383a8 to 40cf8a0 Compare July 8, 2024 16:13
@Ericson2314 Ericson2314 merged commit 3755493 into NixOS:master Jul 9, 2024
11 checks passed
@Ericson2314 Ericson2314 deleted the meson-nix branch July 9, 2024 13:49
@Ericson2314
Copy link
Member Author

I am merging this because we would like the Mesonification to be done by the next release.

@echo >> [email protected]
@echo ')"' >> [email protected]
@mv [email protected] $@
$(d)/profile.md: $(d)/profiles.md.gen.hh
Copy link
Member

@edolstra edolstra Jul 11, 2024

Choose a reason for hiding this comment

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

@Ericson2314 This looks wrong? I assume this rule should be the other way around. (But also the rule should be superfluous because we have a pattern rule for this.)

Copy link
Member Author

Choose a reason for hiding this comment

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

@edolstra I think it is correct note profile vs profiles (the s). One of them includes the other. It could be .md.gen.hh on both sides, however.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation new-cli Relating to the "nix" command
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants