Fix set -eu and pipefail failures
Fixes problems found by set -eu and pipefail, including: -Many unset variables -Commands that can fail under normal circumstances, which breaks with set -e. This change swallows those expected errors to allow our existing error code to handle them. -The dkms element was not finding Fedora kernel versions correctly. This may be an issue for other distros too, but since Fedora was working fine without this functionality I only changed it to print a warning message rather than failing the build when it happens. -The ramdisk init script will not be set -eu because if it fails the result is a kernel panic, which can be tricky to debug. However, in testing with set -e a few failing commands were found and have been fixed in this patch. Change-Id: I44cf98dfc80cfcaec54b88cc83be80a3dbf2cec3
This commit is contained in:
parent
97b8ffcc46
commit
4c977948fe
|
@ -59,7 +59,7 @@ fi
|
||||||
# parallelized later
|
# parallelized later
|
||||||
# Note: -maxdepth 1 ensures only files in the target directory (but not
|
# Note: -maxdepth 1 ensures only files in the target directory (but not
|
||||||
# subdirectories) are run, which is the way run-parts behaves.
|
# subdirectories) are run, which is the way run-parts behaves.
|
||||||
targets=$(find $target_dir -maxdepth 1 -xtype f -executable -printf '%f\n' | grep -E "$allowed_regex" | LANG=C sort -n)
|
targets=$(find $target_dir -maxdepth 1 -xtype f -executable -printf '%f\n' | grep -E "$allowed_regex" | LANG=C sort -n || echo "")
|
||||||
|
|
||||||
if [ "$show_list" == "1" ] ; then
|
if [ "$show_list" == "1" ] ; then
|
||||||
for target in $targets ; do
|
for target in $targets ; do
|
||||||
|
|
Loading…
Reference in New Issue