Skip to content

Commit

Permalink
chroot-fixups: make our scripts work again in pre-UsrMove chroots
Browse files Browse the repository at this point in the history
When we build csmock for RHEL-9, rpmbuild translates `/bin/bash`
to `/usr/bin/bash` in our scripts.  When we copy the scripts into
an old chroot, such as `rhel-6-x86_64`, the translated path does
not exist and the script fails with:
```
>>> 2024-05-20 12:04:36	"/usr/bin/mock" "-r" "rhel-6-x86_64" "--plugin-option=tmpfs:keep_mounted=True" "--config-opts=print_main_output=True" "--quiet" "--chroot" "/usr/share/csmock/scripts/run-shellcheck.sh /builddir/build/BUILDROOT > /builddir/shellcheck-capture.err"
/bin/sh: /usr/share/csmock/scripts/run-shellcheck.sh: /usr/bin/bash: bad interpreter: No such file or directory
```
  • Loading branch information
kdudka committed May 20, 2024
1 parent 6b088e3 commit 86d12cd
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
2 changes: 1 addition & 1 deletion make-srpm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,7 @@ This package contains the unicontrol plug-in for csmock.
%{_mandir}/man1/csmock.1*
%{_datadir}/csmock/cwe-map.csv
%{_datadir}/csmock/scripts/enable-keep-going.sh
%{_datadir}/csmock/scripts/chroot-fixups
%attr(755,root,root) %{_datadir}/csmock/scripts/chroot-fixups
%{_datadir}/csmock/scripts/patch-rawbuild.sh
%{python3_sitelib}/csmock/__init__.py*
%{python3_sitelib}/csmock/common
Expand Down
12 changes: 12 additions & 0 deletions scripts/chroot-fixups/00-pre-usr-move-shells.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# intentionally no shebang so that rpmbuild does not break this script

# exit successfully if this is a post-UsrMove chroot
test -L /bin && exit 0

# if rpmbuild from the host env translated /bin/bash to /usr/bin/bash in our
# scripts, create a reverse symlink to make them work in the chroot env again
for sh in sh bash; do
dst=/usr/bin/${sh}
test -e ${dst} && continue
test -x /bin/${sh} && ln -sv ../../bin/${sh} $dst
done

0 comments on commit 86d12cd

Please sign in to comment.