diff options
| author | Rohan Garg <rohan.garg@collabora.com> | 2021-08-26 15:00:40 +0200 |
|---|---|---|
| committer | Marge Bot <eric+marge@anholt.net> | 2021-08-30 14:52:08 +0000 |
| commit | a6a89aaa2f2943532d99d9bc7b80106a1740f237 (patch) | |
| tree | 74d913129f668e96efd60c23d4d7e6f71ed00406 /.gitlab-ci | |
| parent | 9df9fe7dfaee84a7152e3d0c0e7bc62db1e5d889 (diff) | |
ci: Fix a minor issue in prepare-artifacts.sh script
bash is whitespace sensitive.
https://github.com/koalaman/shellcheck/wiki/SC1020
This was noticed from the logs of a CI job
https://gitlab.freedesktop.org/mesa/mesa/-/jobs/13158779
There was no fallout from this bug as not having this variable defined
leads to the right behavior, and defining it to 1 leads to a error
which consequently also leads to the right behaviour.
Signed-off-by: Rohan Garg <rohan.garg@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12562>
Diffstat (limited to '.gitlab-ci')
| -rwxr-xr-x | .gitlab-ci/prepare-artifacts.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/.gitlab-ci/prepare-artifacts.sh b/.gitlab-ci/prepare-artifacts.sh index 393329effc4..bbc37ef1419 100755 --- a/.gitlab-ci/prepare-artifacts.sh +++ b/.gitlab-ci/prepare-artifacts.sh @@ -18,7 +18,7 @@ if [ -n "$CROSS" ]; then else STRIP="strip" fi -if [ -z "$ARTIFACTS_DEBUG_SYMBOLS"]; then +if [ -z "$ARTIFACTS_DEBUG_SYMBOLS" ]; then find install -name \*.so -exec $STRIP {} \; fi |