fix: #8034 chown parse trailing colon as implicit group-is-user#12392
Open
MarcusGrass wants to merge 8 commits into
Open
fix: #8034 chown parse trailing colon as implicit group-is-user#12392MarcusGrass wants to merge 8 commits into
MarcusGrass wants to merge 8 commits into
Conversation
|
GNU testsuite comparison: |
baf3718 to
2c71aeb
Compare
…ill display different errors Invalid group for root on mac, failed to change on ex. linux
2c71aeb to
2a6dc4e
Compare
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 |
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. |
fa2b9bb to
fe73a74
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #8034
Resurrects #8156
I'll have another stab at the CI issues