summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorLyndon Brown <jnqnfe@gmail.com>2020-03-28 02:21:36 +0000
committerLyndon Brown <jnqnfe@gmail.com>2020-04-23 11:52:04 +0100
commitf73797158e631d9fa2d4c9994e2feff542a7501b (patch)
treec977bc05f20d3ace1df3166fd904ac288f2c1611 /scripts
parentae9143141d460ffbbe6084209977653cdc375bdb (diff)
downloadvyos-live-build-f73797158e631d9fa2d4c9994e2feff542a7501b.tar.gz
vyos-live-build-f73797158e631d9fa2d4c9994e2feff542a7501b.zip
bootloaders: improve bootloader list handling
two parts of the code worked with both comma and space separated lists, while two others only worked with comma separated. swapping out commas with spaces when we setup the var in Set_config_defaults() means that individual scripts no longer need to worry about it and everything supports both; and that we can avoid the IFS/OLDIFS mess. Gbp-Dch: Short
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/build/binary_hdd2
-rwxr-xr-xscripts/build/binary_iso3
-rwxr-xr-xscripts/build/source_debian2
3 files changed, 2 insertions, 5 deletions
diff --git a/scripts/build/binary_hdd b/scripts/build/binary_hdd
index b81c301da..437913929 100755
--- a/scripts/build/binary_hdd
+++ b/scripts/build/binary_hdd
@@ -53,7 +53,7 @@ Check_package chroot /sbin/parted parted
Check_package host /sbin/fdisk fdisk
Check_package host /sbin/losetup mount
-for BOOTLOADER in $(echo "${LB_BOOTLOADERS}" | tr "," "\n"); do
+for BOOTLOADER in ${LB_BOOTLOADERS}; do
case "${BOOTLOADER}" in
grub-legacy)
Check_package chroot /usr/sbin/grub grub
diff --git a/scripts/build/binary_iso b/scripts/build/binary_iso
index eef752f80..9f1b11a86 100755
--- a/scripts/build/binary_iso
+++ b/scripts/build/binary_iso
@@ -104,8 +104,6 @@ fi
# Set an explicit modification date
XORRISO_OPTIONS="${XORRISO_OPTIONS} --modification-date=$(date --utc --date="@${SOURCE_DATE_EPOCH:-$(date +%s)}" +%Y%m%d%H%m%S00)"
-OLDIFS="$IFS"
-IFS=","
BOOTLOADER_NUMBER=0
for BOOTLOADER in ${LB_BOOTLOADERS}
do
@@ -156,7 +154,6 @@ do
esac
done
-IFS="$OLDIFS"
#if [ "${LB_DEBIAN_INSTALLER}" != "live" ]
#then
diff --git a/scripts/build/source_debian b/scripts/build/source_debian
index b9899b2bc..5e37edbca 100755
--- a/scripts/build/source_debian
+++ b/scripts/build/source_debian
@@ -54,7 +54,7 @@ Chroot chroot "sh source-selection.sh"
rm -f chroot/source-selection.sh
mv chroot/source-selection.txt ./
-for BOOTLOADER in $(echo "${LB_BOOTLOADERS}" | tr "," "\n"); do
+for BOOTLOADER in ${LB_BOOTLOADERS}; do
echo "${BOOTLOADER}" >> source-selection.txt
done