Alon Bar-Lev has posted comments on this change. Change subject: spec: add build patches ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/23303/1/jasperreports-server-5.5.0-additional-config.patch File jasperreports-server-5.5.0-additional-config.patch: Line 9: Line 10: --- a/jasperreports-server-cp-5.5.0-bin/buildomatic/bin/js-import-export.sh 2014-01-16 01:41:57.058840500 +0200 Line 11: +++ b/jasperreports-server-cp-5.5.0-bin/buildomatic/bin/js-import-export.sh 2014-01-16 01:42:13.853845937 +0200 Line 12: @@ -34,7 +34,7 @@ fi Line 13: > Why are there trailing white spaces here? because this is unified diff format. Line 14: # Additional config folder. This will be used to Line 15: # get js.jdbc.properties from buildomatic setup Line 16: -export ADDITIONAL_CONFIG_DIR=$BASEDIR/build_conf/default Line 17: +export ADDITIONAL_CONFIG_DIR=${ADDITIONAL_CONFIG_DIR:-$BASEDIR/build_conf/default} Line 10: --- a/jasperreports-server-cp-5.5.0-bin/buildomatic/bin/js-import-export.sh 2014-01-16 01:41:57.058840500 +0200 Line 11: +++ b/jasperreports-server-cp-5.5.0-bin/buildomatic/bin/js-import-export.sh 2014-01-16 01:42:13.853845937 +0200 Line 12: @@ -34,7 +34,7 @@ fi Line 13: Line 14: # Additional config folder. This will be used to but this should not be... :) Line 15: # get js.jdbc.properties from buildomatic setup Line 16: -export ADDITIONAL_CONFIG_DIR=$BASEDIR/build_conf/default Line 17: +export ADDITIONAL_CONFIG_DIR=${ADDITIONAL_CONFIG_DIR:-$BASEDIR/build_conf/default} Line 18: -- To view, visit http://gerrit.ovirt.org/23303 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia1619b341638f3644892f0905c9c6372500c06b8 Gerrit-PatchSet: 1 Gerrit-Project: jasperreports-server-rpm Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
