David Caro has uploaded a new change for review.
Change subject: Fixed only one rpmbuild option being used
......................................................................
Fixed only one rpmbuild option being used
Change-Id: Ia337c3c12b5dabea18a98b250ac0806b687b4e87
Signed-off-by: David Caro <dcaro...@redhat.com>
---
M jobs/confs/shell-scripts/build_local_srcrpm.sh
M jobs/confs/shell-scripts/build_mock_srcrpm.sh
M jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh
M jobs/confs/shell-scripts/mock_build_onlyrpm.sh
4 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/jenkins refs/changes/46/40646/1
diff --git a/jobs/confs/shell-scripts/build_local_srcrpm.sh
b/jobs/confs/shell-scripts/build_local_srcrpm.sh
index 8ea4d63..3697f13 100644
--- a/jobs/confs/shell-scripts/build_local_srcrpm.sh
+++ b/jobs/confs/shell-scripts/build_local_srcrpm.sh
@@ -49,7 +49,7 @@
## build src.rpm
rpmbuild_options=("-D" "release_suffix ${{suffix}}")
-for option in $extra_rpmbuild_options; do
+for option in "${{extra_rpmbuild_options[@]}}"; do
rpmbuild_options+=("-D" "${{option//=/ }}")
done
rpmbuild \
diff --git a/jobs/confs/shell-scripts/build_mock_srcrpm.sh
b/jobs/confs/shell-scripts/build_mock_srcrpm.sh
index 0cdb5c9..0e53979 100644
--- a/jobs/confs/shell-scripts/build_mock_srcrpm.sh
+++ b/jobs/confs/shell-scripts/build_mock_srcrpm.sh
@@ -71,7 +71,7 @@
### Generate the mock configuration
rpmbuild_options=("-D" "release_suffix ${{suffix}}")
mock_build_options=("--define" "release_suffix ${{suffix}}")
-for option in $extra_rpmbuild_options; do
+for option in "${{extra_rpmbuild_options[@]}}"; do
rpmbuild_options+=("-D" "${{option//=/ }}")
mock_build_options+=("--define" "${{option//=/ }}")
done
diff --git a/jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh
b/jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh
index 7ec13d1..a9ec500 100644
--- a/jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh
+++ b/jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh
@@ -48,7 +48,7 @@
### Generate the mock configuration
rpmbuild_options=("-D" "release_suffix ${{suffix}}")
mock_build_options=("--define" "release_suffix ${{suffix}}")
-for option in $extra_rpmbuild_options; do
+for option in "${{extra_rpmbuild_options[@]}}"; do
rpmbuild_options+=("-D" "${{option//=/ }}")
mock_build_options+=("--define" "${{option//=/ }}")
done
diff --git a/jobs/confs/shell-scripts/mock_build_onlyrpm.sh
b/jobs/confs/shell-scripts/mock_build_onlyrpm.sh
index 26248c2..fd7c58a 100644
--- a/jobs/confs/shell-scripts/mock_build_onlyrpm.sh
+++ b/jobs/confs/shell-scripts/mock_build_onlyrpm.sh
@@ -47,7 +47,7 @@
rpmbuild_options+=("-D" "release_suffix ${{suffix}}")
mock_build_options+=("--define" "release_suffix ${{suffix}}")
fi
-for option in $extra_rpmbuild_options; do
+for option in "${{extra_rpmbuild_options[@]}}"; do
rpmbuild_options+=("-D" "${{option//=/ }}")
mock_build_options+=("--define" "${{option//=/ }}")
done
--
To view, visit https://gerrit.ovirt.org/40646
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia337c3c12b5dabea18a98b250ac0806b687b4e87
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaro...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
