Anatoly Litovsky has posted comments on this change.

Change subject: Adding ovirt node and ovirt node iso projects
......................................................................


Patch Set 9:

(2 comments)

I dont think bash understand {{ after function declaration at least my dont. 
Some find and replace gone wrong ?

http://gerrit.ovirt.org/#/c/36058/9/jobs/confs/shell-scripts/build-local-ovirt-node.sh
File jobs/confs/shell-scripts/build-local-ovirt-node.sh:

Line 4: 
Line 5: DISTRO="{distro}"
Line 6: 
Line 7: 
Line 8: # the die on error function
I dont think bash understand {{ after function declaration
at least my dont.
Some find and replace gone wrong ?
Line 9: function die {{
Line 10:     echo "$1"
Line 11:     exit 1
Line 12: }}


http://gerrit.ovirt.org/#/c/36058/9/jobs/confs/shell-scripts/ovirt_node_cleanup.sh
File jobs/confs/shell-scripts/ovirt_node_cleanup.sh:

Line 4: DISTRO="{distro}"
Line 5: CACHE="$PWD"/build
Line 6: 
Line 7: # the die on error function
Line 8: function die {{
I dont think bash understand {{ after function declaration at least my dont. 
Some find and replace gone wrong ?
Line 9:     echo "$1"
Line 10:     exit 1
Line 11: }}
Line 12: 


-- 
To view, visit http://gerrit.ovirt.org/36058
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If543fe28f200e8254d4d96fa6789db3ced44f2b1
Gerrit-PatchSet: 9
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Anatoly Litovsky <tlito...@redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlito...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabi...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to