Skip to content

fix: #8034 chown parse trailing colon as implicit group-is-user#12392

Open
MarcusGrass wants to merge 8 commits into
uutils:mainfrom
MarcusGrass:mg/8034-chown-trailing-colon
Open

fix: #8034 chown parse trailing colon as implicit group-is-user#12392
MarcusGrass wants to merge 8 commits into
uutils:mainfrom
MarcusGrass:mg/8034-chown-trailing-colon

Conversation

@MarcusGrass
Copy link
Copy Markdown
Contributor

fixes #8034

Resurrects #8156

I'll have another stab at the CI issues

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/date/resolution (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/misc/tty-eof (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/pr/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/tail-n0f (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cut/cut-huge-range is now passing!

@MarcusGrass MarcusGrass force-pushed the mg/8034-chown-trailing-colon branch 2 times, most recently from baf3718 to 2c71aeb Compare May 20, 2026 15:48
@MarcusGrass MarcusGrass force-pushed the mg/8034-chown-trailing-colon branch from 2c71aeb to 2a6dc4e Compare May 20, 2026 19:41
@MarcusGrass
Copy link
Copy Markdown
Contributor Author

I'd consider the branch resurrected, things have changed in the underlying system and the current errors are just test-procedure that needs to be fixed up, something with the CI on macos, and a test-condition that doesn't hold for OpenBSD

@MarcusGrass
Copy link
Copy Markdown
Contributor Author

I don't like this openbsd hack but it seems that the directory has a sticky wheel group on it, not sure of easy ways to solve that at the moment. Improvement suggestions welcome.
Last one is an issue on the mac-runner where it runs as the user runner but no group exists for that user so setting runner:runner fails because of it

@MarcusGrass MarcusGrass force-pushed the mg/8034-chown-trailing-colon branch from fa2b9bb to fe73a74 Compare May 21, 2026 19:42
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.

chown compatibility: chown root: abc doesn't change group

1 participant