Merge "Ensure scripts are set -u"
This commit is contained in:
commit
273d364fc4
@ -72,7 +72,7 @@ for i in $(find elements -type f); do
|
|||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# Check that all scripts are set -e
|
# Check that all scripts are set -eu
|
||||||
# NOTE(bnemec): This doesn't verify that the set call occurs high
|
# NOTE(bnemec): This doesn't verify that the set call occurs high
|
||||||
# enough in the file to be useful, but hopefully nobody will be
|
# enough in the file to be useful, but hopefully nobody will be
|
||||||
# sticking set calls at the end of their file to trick us. And if
|
# sticking set calls at the end of their file to trick us. And if
|
||||||
@ -88,6 +88,12 @@ for i in $(find elements -type f); do
|
|||||||
rc=1
|
rc=1
|
||||||
fi
|
fi
|
||||||
fi
|
fi
|
||||||
|
if ! excluded setu; then
|
||||||
|
if [ -z "$(grep "^set -[^ ]*u" $i)" ]; then
|
||||||
|
echo "ERROR: $i is not set -u"
|
||||||
|
rc=1
|
||||||
|
fi
|
||||||
|
fi
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user