Skip to content

fix(deps): refactor rustix/nix package configuration - #14337

Merged
cakebaker merged 2 commits into
uutils:mainfrom
xtqqczze:deps/rustix
Sep 2, 2026
Merged

fix(deps): refactor rustix/nix package configuration#14337
cakebaker merged 2 commits into
uutils:mainfrom
xtqqczze:deps/rustix

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Refactor rustix and nix dependency configuration to use explicit, target-specific features.

Benefits

  • Avoid enabling unused features on platforms where they aren't needed.
  • Keep platform-specific dependencies scoped to the relevant targets.
  • Make the Cargo configuration easier to understand and maintain.
  • Reduce unnecessary dependency features and cross-platform coupling.

@codspeed-hq

codspeed-hq Bot commented Aug 31, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 361 untouched benchmarks
⏩ 50 skipped benchmarks1


Comparing xtqqczze:deps/rustix (f0222eb) with main (dfb953d)

Open in CodSpeed

Footnotes

  1. 50 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout-group (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/date/date-locale-hour (passes in this run but fails in the 'main' branch)

@xtqqczze

This comment was marked as outdated.

@xtqqczze

This comment was marked as outdated.

@xtqqczze

This comment was marked as outdated.

@xtqqczze
xtqqczze force-pushed the deps/rustix branch 2 times, most recently from 9587621 to 318f3c4 Compare September 1, 2026 02:30
@xtqqczze xtqqczze changed the title fix(deps): refactor rustix package configuration fix(deps): refactor rustix/nix package configuration Sep 1, 2026
@xtqqczze
xtqqczze force-pushed the deps/rustix branch 5 times, most recently from e91a006 to a9cc1ee Compare September 1, 2026 11:13
@xtqqczze
xtqqczze marked this pull request as ready for review September 1, 2026 16:14
@cakebaker
cakebaker merged commit f8569ef into uutils:main Sep 2, 2026
177 checks passed
@cakebaker

Copy link
Copy Markdown
Contributor

Thanks!

@xtqqczze
xtqqczze deleted the deps/rustix branch September 2, 2026 10:56
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