Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: do not build unneeded utilities and man for SELinux libraries #1073

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

dsseng
Copy link
Member

@dsseng dsseng commented Nov 4, 2024

Utilities end up on rootfs increasing size and binary count

Fixes #1067

Signed-off-by: Dmitry Sharshakov dmitry.sharshakov@siderolabs.com

--- a/Makefile
+++ b/Makefile
@@ -4,26 +4,21 @@ export DISABLE_CIL

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we don't want to maintain patches, we can also let it build, but then do rm -rf /rootfs/usr/....

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They aren't likely to change that often. Maybe I should send a patch upstream so there's an option like SUBDIRS in libselinux

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstreamed a couple of change for SUBDIRS in every lib.

@smira
Copy link
Member

smira commented Nov 11, 2024

@dsseng should we rebase/merge this?

@dsseng
Copy link
Member Author

dsseng commented Nov 11, 2024

@dsseng should we rebase/merge this?

Should we keep the patch or delete built stuff? See my response. After closing that thread it's good to go

@smira
Copy link
Member

smira commented Nov 11, 2024

@dsseng should we rebase/merge this?

Should we keep the patch or delete built stuff? See my response. After closing that thread it's good to go

up to you, it's your call

Utilities end up on rootfs increasing size and binary count

Fixes siderolabs#1067

Signed-off-by: Dmitry Sharshakov <dmitry.sharshakov@siderolabs.com>
@dsseng
Copy link
Member Author

dsseng commented Nov 12, 2024

/m

@talos-bot talos-bot merged commit 567a14a into siderolabs:main Nov 12, 2024
14 checks passed
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.

Minimize SELinux libraries, drop unneeded binaries
4 participants