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(dracut): do not add all lib subdirs to LD_LIBRARY_PATH with --sysroot (bsc#1230354) #372

Merged
merged 3 commits into from
Sep 12, 2024

Conversation

aafeijoo-suse
Copy link
Collaborator

On systems with too many library subdirectories, the current approach causes Argument list too long errors, and the initrd also fails to build.

This patch does not fix the original issue by itself (i.e., libsystemd libs not found), so all the dracut modules that require libraries stored in subdirectories must explicitly install them using inst_libdir_file, which already successfully handles the --sysroot prefix.

xurubezi and others added 3 commits September 11, 2024 14:35
…sysroot`

On systems with too many library subdirectories, the current approach causes
`Argument list too long` errors, and the initrd also fails to build.

This patch does not fix the original issue by itself (i.e., libsystemd libs not
found), so all the dracut modules that require libraries stored in
subdirectories must explicitly install them using `inst_libdir_file`, which
already successfully handles the `--sysroot` prefix.

Fixes dracut-ng/dracut-ng@e0b8768

Co-authored-by: Antonio Alvarez Feijoo <[email protected]>

bsc#1230354

(cherry picked from commit dracut-ng/dracut-ng@d0c8232)
Copy link
Collaborator

@tblume tblume left a comment

Choose a reason for hiding this comment

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

LGTM

@aafeijoo-suse aafeijoo-suse merged commit acb9ef9 into openSUSE:SUSE/059 Sep 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants