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.49%

❌ 4 regressed benchmarks
✅ 137 untouched benchmarks
⏩ 38 skipped benchmarks1

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

Performance Changes

Benchmark BASE HEAD Efficiency
split_bytes 515.3 µs 545.2 µs -5.49%
split_number_chunks 289.6 µs 302.9 µs -4.4%
sort_ascii_utf8_locale 20.4 ms 21.4 ms -4.88%
sort_numeric_utf8_locale 22.4 ms 23.2 ms -3.3%

Comparing ChrisDryden:fix-cp-a-selinux-test (060903b) with main (0b0ba60)

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!

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