Skip to content

tests/cp/preserve-gid: Partial fix#8380

Merged
RenjiSann merged 2 commits intouutils:mainfrom
drinkcat:cp-preserve-gid
Jul 24, 2025
Merged

tests/cp/preserve-gid: Partial fix#8380
RenjiSann merged 2 commits intouutils:mainfrom
drinkcat:cp-preserve-gid

Conversation

@drinkcat
Copy link
Copy Markdown
Collaborator

Partial fix for tests/cp/preserve-gid, and also a small refactoring of util/build-gnu.sh.

I'll file a bug for the next issue.


util/build-gnu.sh: Replace $abs_path_dir_

path_prepend_ sets $abs_path_dir_, but the sed command above removes
it. Set it manually instead.

Fixes part of tests/cp/preserve-gid.

util/build.sh: Patch tests/local.mk instead of Makefile

Patching tests/local.mk instead of Makefile makes it a bit easier
to see what has been modified with a git diff, and Makefile
is generated anyway.

drinkcat added 2 commits July 24, 2025 15:19
Patching tests/local.mk instead of Makefile makes it a bit easier
to see what has been modified with a `git diff`, and Makefile
is generated anyway.
path_prepend_ sets $abs_path_dir_, but the sed command above removes
it. Set it manually instead.

Fixes part of tests/cp/preserve-gid.
@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

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

@RenjiSann RenjiSann merged commit 07b68ee into uutils:main Jul 24, 2025
193 of 194 checks passed
@drinkcat drinkcat deleted the cp-preserve-gid branch August 8, 2025 10:19
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