fix(nl): allow repeated flags to match GNU nl behavior (fixes #9132) by naoNao89 · Pull Request #9140 · uutils/coreutils

naoNao89 · 2025-11-04T07:35:46Z

Fixes #9132. uu-nl now accepts repeated flags like GNU nl. Added args_override_self(true)to all 10 value-accepting arguments.

github-actions · 2025-11-04T07:57:03Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

tests/by-util/test_nl.rs

src/uu/nl/src/nl.rs

src/uu/nl/src/helper.rs

github-actions · 2025-11-05T17:31:30Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

tests/by-util/test_nl.rs

github-actions · 2025-11-07T13:13:51Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

codspeed-hq · 2025-11-08T21:42:50Z

CodSpeed Performance Report

Merging #9140 will not alter performance

_{Comparing naoNao89:fix/nl-repeated-flags-9132 (abcfbe4) with main (5318c9c)}

Summary

✅ 123 untouched
⏩ 5 skipped¹

5 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 · 2025-11-08T21:52:09Z

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

tests/by-util/test_nl.rs

github-actions · 2025-11-10T19:48:30Z

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

tests/by-util/test_nl.rs

github-actions · 2025-11-11T13:21:30Z

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

tests/by-util/test_nl.rs

github-actions · 2025-11-11T18:08:07Z

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

tests/by-util/test_nl.rs

github-actions · 2025-11-12T12:59:45Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

tests/by-util/test_nl.rs

• Enable args_override_self(true) for repeated flags • Remove unnecessary ArgAction::Append calls • Fix tests to verify last value wins, not just succeeds • Update test output validation for repeated flags Fixes uutils#9132

github-actions · 2025-11-12T15:02:50Z

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

cakebaker · 2025-11-12T16:05:09Z

Thanks :)

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

src/uu/nl/src/nl.rs Show resolved Hide resolved

cakebaker reviewed Nov 5, 2025

View reviewed changes

src/uu/nl/src/helper.rs Show resolved Hide resolved

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from 5238335 to a2db784 Compare November 5, 2025 17:10

cakebaker reviewed Nov 6, 2025

View reviewed changes

tests/by-util/test_nl.rs Show resolved Hide resolved

naoNao89 closed this by deleting the head repository Nov 6, 2025

naoNao89 reopened this Nov 7, 2025

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from a2db784 to 8f1bd52 Compare November 8, 2025 21:30

cakebaker reviewed Nov 10, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 10, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 10, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

cakebaker reviewed Nov 10, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from 8f1bd52 to 2d56770 Compare November 10, 2025 19:28

cakebaker reviewed Nov 11, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from 2d56770 to a4819ba Compare November 11, 2025 12:59

cakebaker reviewed Nov 11, 2025

View reviewed changes

tests/by-util/test_nl.rs Show resolved Hide resolved

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch 2 times, most recently from 1ed8792 to aff651d Compare November 11, 2025 17:47

cakebaker reviewed Nov 12, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from aff651d to 01cfb02 Compare November 12, 2025 12:40

cakebaker reviewed Nov 12, 2025

View reviewed changes

tests/by-util/test_nl.rs Outdated Show resolved Hide resolved

fix(nl): allow repeated flags to match GNU behavior

abcfbe4

• Enable args_override_self(true) for repeated flags • Remove unnecessary ArgAction::Append calls • Fix tests to verify last value wins, not just succeeds • Update test output validation for repeated flags Fixes uutils#9132

naoNao89 force-pushed the fix/nl-repeated-flags-9132 branch from 01cfb02 to abcfbe4 Compare November 12, 2025 14:06

cakebaker merged commit d432131 into uutils:main Nov 12, 2025
126 checks passed

naoNao89 deleted the fix/nl-repeated-flags-9132 branch November 12, 2025 17:06

moonfruit mentioned this pull request Dec 15, 2025

uutils-selected 0.5.0 moonfruit/homebrew-tap#387

Closed

Uh oh!

Conversation

naoNao89 commented Nov 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Nov 4, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 5, 2025

Uh oh!

Uh oh!

github-actions bot commented Nov 7, 2025

Uh oh!

codspeed-hq bot commented Nov 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CodSpeed Performance Report

Merging #9140 will not alter performance

Summary

Footnotes

Uh oh!

github-actions bot commented Nov 8, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Nov 10, 2025

Uh oh!

Uh oh!

github-actions bot commented Nov 11, 2025

Uh oh!

Uh oh!

github-actions bot commented Nov 11, 2025

Uh oh!

Uh oh!

github-actions bot commented Nov 12, 2025

Uh oh!

Uh oh!

github-actions bot commented Nov 12, 2025

Uh oh!

Uh oh!

cakebaker commented Nov 12, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

naoNao89 commented Nov 4, 2025 •

edited

Loading

codspeed-hq bot commented Nov 8, 2025 •

edited

Loading