[abendrothj]

Based on #15029; only the last commit is new here.

On Linux, a directory operand was changed through a descriptor opened for reading, and when that open failed the change was skipped without a message.

Without -R the directory is now changed like any other file: by name, or through an O_PATH descriptor with --from. Under -R it's still left alone with exit 1, as with GNU 9.12. Test: test_chown_unreadable_dir_operand.

Closes #15114

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[codspeed]

Merging this PR will improve performance by 4.17%

2 warnings

⚠️ 65 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

⚠️ 1 benchmark with a significant change ran on different hardware

Its base and head results were measured on machines with a different CPU, core count or memory, so the change might not come from the code. Run base and head on the same runner type.

Learn more about Macro Runners

See affected benchmarks

⚡ 1 improved benchmark
✅ 114 untouched benchmarks
⏩ 360 skipped benchmarks¹

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
⚡	du_wide_tree[(5000, 500)]	15.9 ms	15.3 ms	+4.17%

Tip

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

---

_{Comparing abendrothj:chown-unreadable-dir-operand (a3ed451) with main (bbef561)}

Footnotes

1. 360 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. ↩

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[sylvestre]

sorry but it needs to be rebased