Skip to content

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Oct 19, 2025

Fixes #7996 .

Separated from #8937 because of unknown test failure for Android.

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/invalid-opt (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/runcon/runcon-no-reorder (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/runcon/runcon-no-reorder (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@sylvestre
Copy link
Contributor

sorry but i still don't understand why you have to do that ?

@oech3
Copy link
Contributor Author

oech3 commented Oct 20, 2025

Because this project is impossible to build on Linux without libselinux if chcons are included.

@oech3
Copy link
Contributor Author

oech3 commented Oct 20, 2025

Skipping SELINUX_PROGS is same behavior with cargo build --release --features unix.
If missing progs by default is not acceptible, document and small option are needed to avoid build failure by default on many system.

@oech3 oech3 changed the title Omit SELINUX_PROGS on non SELinux env build: Omit SELINUX_PROGS to avoid build failure without SELinux Oct 20, 2025
@sylvestre
Copy link
Contributor

Sorry but I still don't get why you need to touch the gnu testsuite. I want to run runcon against them

@github-actions
Copy link

GNU testsuite comparison:

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

@oech3 oech3 requested a review from sylvestre October 20, 2025 23:55
@oech3

This comment was marked as outdated.

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

1 similar comment
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 8, 2025

CodSpeed Performance Report

Merging #8955 will not alter performance

Comparing oech3:noselinuxprogs (dfa30bc) with main (73d1bce)

Summary

✅ 123 untouched
⏩ 5 skipped1

Footnotes

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

github-actions bot commented Nov 8, 2025

GNU testsuite comparison:

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

@oech3 oech3 changed the title build: Omit SELINUX_PROGS to avoid build failure without SELinux build: Omit SELINUX_PROGS, fix build failure without SELinux Nov 8, 2025
@github-actions
Copy link

github-actions bot commented Nov 8, 2025

GNU testsuite comparison:

GNU test failed: tests/id/context. tests/id/context is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/id/no-context. tests/id/no-context is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/mkdir/selinux. tests/mkdir/selinux is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/runcon/runcon-no-reorder. tests/runcon/runcon-no-reorder is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)
Note: The gnu test tests/chcon/chcon is now being skipped but was previously passing.
Note: The gnu test tests/cp/no-ctx is now being skipped but was previously passing.
Note: The gnu test tests/install/install-C-selinux is now being skipped but was previously passing.
Note: The gnu test tests/install/install-Z-selinux is now being skipped but was previously passing.
Note: The gnu test tests/ls/selinux is now being skipped but was previously passing.
Note: The gnu test tests/misc/selinux is now being skipped but was previously passing.
Note: The gnu test tests/mkdir/restorecon is now being skipped but was previously passing.

@oech3
Copy link
Contributor Author

oech3 commented Nov 8, 2025

coreutils 9.9 will stop building and testing runcon and chcon by default. I changed this PR to backporting the behavior instead of hacking to force-enabling tests by force-building them. We tests them iff SELINUX_ENABLED=1. So

Warning: Changes from 'main': PASS -9 /
    FAIL +1 / ERROR +0 / SKIP +8
Error: PASS count is reduced from
        'main': PASS -9

should be fine. @sylvestre @cakebaker Would you review this?

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@oech3 oech3 changed the title build: Omit SELINUX_PROGS, fix build failure without SELinux GNUmakefile: fix build failure without libseinux Nov 12, 2025
@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/invalid-opt. tests/misc/invalid-opt is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/invalid-opt. tests/misc/invalid-opt is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@oech3
Copy link
Contributor Author

oech3 commented Nov 14, 2025

I'll split this to 3 PRs more robust.

@oech3 oech3 closed this Nov 14, 2025
@oech3 oech3 deleted the noselinuxprogs branch November 15, 2025 13:05
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.

Build failure without libselinux (missing ifeq in GNUmakefile)

2 participants