diff options
| author | Niels de Vos <ndevos@redhat.com> | 2013-10-09 12:08:49 +0200 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-10-09 08:10:12 -0700 |
| commit | ce9a704361c0a3638e46911eb5275c02e6a5d9ab (patch) | |
| tree | afc5d126d0404225bc3c0bd7887ebb29b862629c | |
| parent | e51ca3c1c991416895e1e8693f7c3e6332d57464 (diff) | |
tests/basic/rpm.t: fix script error introduced by commit a66bf679
The if-clause will always return false, causing each regression test to
build the rpms. The following error is listed on the Jenkins Console
Output of the regression test:
./tests/basic/rpm.t: line 42: [: argument expected
BUG: 904005
Change-Id: I343c80d8c57fa0ec8b8a531bb5f5c86209d88d7e
CC: Harshavardhana <harsha@harshavardhana.net>
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/6057
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Reviewed-by: Harshavardhana <harsha@harshavardhana.net>
Tested-by: Harshavardhana <harsha@harshavardhana.net>
| -rwxr-xr-x | tests/basic/rpm.t | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/basic/rpm.t b/tests/basic/rpm.t index a914d536c29..a577726a82b 100755 --- a/tests/basic/rpm.t +++ b/tests/basic/rpm.t @@ -39,7 +39,7 @@ fi BUILD_FILES=$(git diff --name-status ${GIT_PARENT} | grep -Ev '^M.*\.(c|h|py)' | awk {'print $2'}) SELFTEST=$(grep -e 'tests/basic/rpm.t' <<< "${BUILD_FILES}") BUILD_FILES=$(grep -Ev '^tests/' <<< "${BUILD_FILES}") -if [ -z "${BUILD_FILES}" -a -z "${SELFTEST}"] +if [ -z "${BUILD_FILES}" -a -z "${SELFTEST}" ] then # nothing affecting packaging changed, no need to retest rpmbuild SKIP_TESTS |