Skip to content

Conversation

yuankunzhang
Copy link
Contributor

Closes #8395.

@yuankunzhang yuankunzhang force-pushed the mkfifo-support-symbolic-mode branch 2 times, most recently from 809d8a0 to b845ae5 Compare August 6, 2025 13:51
Copy link

github-actions bot commented Aug 6, 2025

GNU testsuite comparison:

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

@yuankunzhang yuankunzhang force-pushed the mkfifo-support-symbolic-mode branch from b845ae5 to 5825e5e Compare August 6, 2025 14:19
Copy link
Collaborator

@drinkcat drinkcat left a comment

Choose a reason for hiding this comment

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

Thanks! This is good.

But... if you're looking for a small refactoring project... grep for parse_numeric in the codebase. We now have.... 4 copies of very similar code in chmod, install, mkdir and mkfifo.

Copy link

github-actions bot commented Aug 6, 2025

GNU testsuite comparison:

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

@drinkcat drinkcat merged commit 1c6be20 into uutils:main Aug 6, 2025
90 checks passed
@JupiterSky11
Copy link

Dang, so fast. I and all the other users appreciate y'all's efforts. :)

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.

mkfifo -m o+w fails.
3 participants