Skip to content

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

Merged
kdudka merged 1 commit intocsutils:mainfrom
kdudka:chroot-fixup-llvm
Jul 16, 2024
Merged

chroot-fixups: prevent %install of llvm from taking ages to complete#180
kdudka merged 1 commit intocsutils:mainfrom
kdudka:chroot-fixup-llvm

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 force-pushed the chroot-fixup-llvm branch from 01e3417 to 52fc540 Compare July 16, 2024 13:01
@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
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 kdudka force-pushed the chroot-fixup-llvm branch from a713f74 to b3a2279 Compare July 16, 2024 13:44
@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
@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.

2 participants