You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
if grep -q -E '^etc/profile\.d/.*\.sh(\.new)?'"$varlogpkg";then
whileread -r script;do
if [ -f"${MY_CHRDIR}"/"$script" ];then
log_info -a " Running profile script: /$script"
."${MY_CHRDIR}"/"$script"
elif [ -f"${MY_CHRDIR}"/"$script".new ];then
log_info -a " Running profile script: /$script.new"
."${MY_CHRDIR}"/"$script".new
fi
done<<(grep '^etc/profile\.d/.*\.sh'"$varlogpkg"| sed 's/.new$//')
fi
return
}
If the profile script has conditionals in it based on the existence of some files or directories, this will not work. For example, the qt6 profile script has this code:
This means that nothing will be done in the case of using a clean env and building with a chroot since qt6 is not installed on the host too. This means that any packages using qt6 as a dependency and depending on these paths being setup will not work.
The text was updated successfully, but these errors were encountered:
aclemons
changed the title
Profile script sourcing when building with chroot enable incorrect
Profile script sourcing when building with chroot enabled incorrect
Apr 20, 2024
Overview
When building with a chroot enabled, slackrepo sources each profile script from dependent packages as it does when building without. See code here:
slackrepo/libexec/functions.d/installfunctions.sh
Lines 382 to 403 in 625dbef
If the profile script has conditionals in it based on the existence of some files or directories, this will not work. For example, the qt6 profile script has this code:
https://github.com/SlackBuildsOrg/slackbuilds/blob/23949cf1f662dce285cda0762ba26a016129265f/libraries/qt6/profile.d/qt6.sh#L6-L15
This means that nothing will be done in the case of using a clean env and building with a chroot since qt6 is not installed on the host too. This means that any packages using qt6 as a dependency and depending on these paths being setup will not work.
The text was updated successfully, but these errors were encountered: