Yaniv Dary has posted comments on this change.

Change subject: spec: add build patches
......................................................................


Patch Set 1:

(1 comment)

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?
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}


-- 
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

Reply via email to