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

chroot-fixups: prevent %install of llvm from taking ages to complete #180

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

kdudka
Copy link
Member

@kdudka kdudka commented Jul 16, 2024

The brp-llvm-compile-lto-elf script does not work well in our scanning environment and causes the %install section of llvm-16.0.6-4.el9 to take 7 days to complete.

Depends-on: #179

@kdudka kdudka requested a review from lzaoral July 16, 2024 13:01
@kdudka kdudka self-assigned this Jul 16, 2024
@kdudka kdudka changed the title chroot-fixups: prevent %install of llvm from taking ages to complete chroot-fixups: prevent %install of llvm from taking ages to complete Jul 16, 2024
scripts/chroot-fixups/rpm-build-scripts.sh Show resolved Hide resolved
fi
# skip RPM scripts running in %install that are not needed and break scans
for script in /usr/lib/rpm/{redhat/brp-llvm-compile-lto-elf,brp-strip-static-archive}; do
if [ -f $script ]; then

Check warning

Code scanning / shellcheck

SC2086

Double quote to prevent globbing and word splitting.
# skip RPM scripts running in %install that are not needed and break scans
for script in /usr/lib/rpm/{redhat/brp-llvm-compile-lto-elf,brp-strip-static-archive}; do
if [ -f $script ]; then
ln -fsvT /bin/true $script

Check warning

Code scanning / shellcheck

SC2086

Double quote to prevent globbing and word splitting.
The `brp-llvm-compile-lto-elf` script does not work well in our scanning
environment and causes the %install section of `llvm-16.0.6-4.el9` to
take 7 days to complete.

Depends-on: csutils#179
Closes: https://github.com/kdudka/csmock/pull/new/chroot-fixup-llvm
@kdudka
Copy link
Member Author

kdudka commented Jul 16, 2024

@lzaoral Thanks for review!

@kdudka kdudka merged commit b3a2279 into csutils:main Jul 16, 2024
33 checks passed
@kdudka kdudka deleted the chroot-fixup-llvm branch July 16, 2024 14:10
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.

None yet

2 participants