Sandro Bonazzola has posted comments on this change.

Change subject: packaging: using patches to customize war instead of overwrites
......................................................................


Patch Set 3: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/24275/3/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
File 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py:

Line 803:             )
Line 804:         ):
Line 805:             rc, stdout, stderr = self.execute(
Line 806:                 args=(
Line 807:                     '/usr/bin/patch',
you'll need to require patch in spec file and you should use  
self.command.detect and self.command.get for acquiring patch full path.
Line 808:                     '-p2',
Line 809:                     '-i',
Line 810:                     p,
Line 811:                 ),


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If0becc8ef9206fa077547682fd4c299fa5491b15
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <yd...@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-Reviewer: Yedidyah Bar David <d...@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