Skip to content

fix(build): raise minimum cfg_aliases to 0.2.2 - #2811

Closed
jzuber4 wants to merge 1 commit into
nix-rust:masterfrom
jzuber4:fix-minver-cfg-aliases
Closed

fix(build): raise minimum cfg_aliases to 0.2.2#2811
jzuber4 wants to merge 1 commit into
nix-rust:masterfrom
jzuber4:fix-minver-cfg-aliases

Conversation

@jzuber4

@jzuber4 jzuber4 commented Aug 13, 2026

Copy link
Copy Markdown

What does this PR do

Raises the minimum cfg_aliases build-dependency from 0.2.1 to 0.2.2.

nix's build.rs invokes the cfg_aliases! macro. Under
cargo -Zdirect-minimal-versions, cargo pins the declared floor (0.2.1),
whose macro expansion contains a trailing semicolon in expression position.
Current Rust promotes that from a warning to a hard error, so the build script
fails to compile and the minver CI job is red. (This is currently failing on
master, independent of any individual PR.)

cfg_aliases 0.2.2 restructured the macro to remove the trailing semicolon, so
raising the floor makes the minimum-versions build compile again. cfg_aliases
is a build-dependency only — it is not part of nix's public API — is edition
2018 with no dependencies and no declared MSRV, so this has no effect on nix's
consumers or on nix's MSRV.

Checklist:

  • I have read CONTRIBUTING.md
  • I have written necessary tests and rustdoc comments
  • A change log has been added if this PR modifies nix's API

nix's build script invokes the `cfg_aliases!` macro. Under
`cargo -Zdirect-minimal-versions`, cargo pins the declared floor
(0.2.1), whose macro expansion ends in a trailing semicolon in
expression position. Current Rust treats that as a hard error, so
the build script fails to compile and the `minver` CI job breaks.

cfg_aliases 0.2.2 restructured the macro to drop the trailing
semicolon. Raising the floor to 0.2.2 makes the minimum-versions
build compile. cfg_aliases is a build-dependency only (not part of
nix's public API), edition 2018, with no dependencies and no MSRV
bump, so nix's consumers and MSRV are unaffected.
@jzuber4

jzuber4 commented Aug 13, 2026

Copy link
Copy Markdown
Author

Superseded by #2812, which keeps this cfg_aliases floor bump and adds the MSRV 1.69 → 1.70 bump so that both the minver and msrv CI jobs pass together (the msrv job is gated on minver, so neither shows green alone).

@jzuber4 jzuber4 closed this Aug 13, 2026
@xtqqczze

Copy link
Copy Markdown
Contributor

Duplicate of #2808

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants