Skip to content

ls: exit with code 2 on write error - #14111

Merged
sylvestre merged 1 commit into
uutils:mainfrom
Devel08:ls-error-code-dev-null
Aug 24, 2026
Merged

ls: exit with code 2 on write error#14111
sylvestre merged 1 commit into
uutils:mainfrom
Devel08:ls-error-code-dev-null

Conversation

@Devel08

@Devel08 Devel08 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

GNU ls errors with code 2 on write error
see #9782 (comment)

$ ls --version; ls > /dev/full; printf $?
ls (GNU coreutils) 9.11
Copyright (C) 2026 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Richard M. Stallman and David MacKenzie.
ls: write error: No space left on device
2

ls: add test_write_error
@Devel08
Devel08 force-pushed the ls-error-code-dev-null branch from 01c32e8 to 305a8a2 Compare August 24, 2026 09:40
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/date/date-locale-hour (passes in this run but fails in the 'main' branch)
Note: The gnu test tests/basenc/bounded-memory is now being skipped but was previously passing.
Note: The gnu test tests/tail/tail-n0f is now being skipped but was previously passing.

@codspeed-hq

codspeed-hq Bot commented Aug 24, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 3.35%

⚡ 1 improved benchmark
✅ 356 untouched benchmarks
⏩ 50 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation du_deep_tree[(100, 3)] 2.3 ms 2.2 ms +3.35%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing Devel08:ls-error-code-dev-null (305a8a2) with main (3d14980)

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.

@sylvestre
sylvestre merged commit a9b64fc into uutils:main Aug 24, 2026
169 checks passed
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