Skip to content

Removing usage_vs_refs from comparison gnu test suite#10896

Open
ChrisDryden wants to merge 1 commit into
mainfrom
usage_vs_refs_removal
Open

Removing usage_vs_refs from comparison gnu test suite#10896
ChrisDryden wants to merge 1 commit into
mainfrom
usage_vs_refs_removal

Conversation

@ChrisDryden

Copy link
Copy Markdown
Collaborator

This test checks that all of the options are included in the GNU documentation, our implementation has a bunch of options that are not included and the test does not provide much value for us for regression purposes

@oech3

oech3 commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

I'd like to keep them as failing test at CI log.
If they produce big logs, you can symlink false to test files instead of sed.

@sylvestre sylvestre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM — this test can never pass for uutils since we intentionally have different options (e.g. clap -V). Good riddance.

@sylvestre sylvestre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM — this test can never pass for uutils since we intentionally have different options (e.g. clap -V). Good riddance.

@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/pr/bounded-memory (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/tail-n0f (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)
Note: The gnu test tests/cp/link-heap is now being skipped but was previously passing.
Congrats! The gnu test tests/expand/bounded-memory is now passing!

@sylvestre sylvestre force-pushed the usage_vs_refs_removal branch from 6ed4539 to 609d4ca Compare June 7, 2026 09:03
@sylvestre sylvestre force-pushed the usage_vs_refs_removal branch from 609d4ca to 15ac7ac Compare June 8, 2026 20:08
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.

3 participants