Skip to content

Conversation

@ChrisDryden
Copy link
Collaborator

This has been in the works for a few days where I have been adding the functionality that is expected by the gnu integration tests for cp, the main areas that were missing were related to skipping copying the selinux xattr's.

Theres a bit of overlap between this PR and this one: #10083 since they both cover how to skip those types of ENOSUP errors.

The only other change is that I added is that one of the utilities was needed to run this test on the SELINUX build.

@github-actions
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@sylvestre
Copy link
Contributor

some jobs are failing

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 13, 2026

Merging this PR will degrade performance by 5.25%

⚡ 1 improved benchmark
❌ 2 regressed benchmarks
✅ 281 untouched benchmarks
⏩ 38 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation numfmt_padding[(10000, 50)] 49.6 ms 51.1 ms -3.02%
Simulation mv_force_overwrite 147.5 ms 141.9 ms +3.92%
Simulation split_number_chunks 255 µs 269.1 µs -5.25%

Comparing ChrisDryden:fix-cp-a-selinux-test (4560b1f) with main (7b0e7e4)

Open in CodSpeed

Footnotes

  1. 38 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.

@github-actions
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tty/tty-eof (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@sylvestre
Copy link
Contributor

sorry, it conflicts :(

@ChrisDryden ChrisDryden force-pushed the fix-cp-a-selinux-test branch 2 times, most recently from 2f7060a to d351855 Compare January 21, 2026 03:44
@github-actions
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@sylvestre
Copy link
Contributor

@ChrisDryden seems that it needs some love :)

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/tail/retry. tests/tail/retry is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/follow-name (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

@ChrisDryden ChrisDryden force-pushed the fix-cp-a-selinux-test branch from 39b1753 to 4560b1f Compare February 10, 2026 03:50
@github-actions
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!

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