Change in ovirt-engine[master]: packaging: engine-backup: Fix apache reconfiguration

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Apr 18 11:10:36 UTC 2016


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: engine-backup: Fix apache reconfiguration
......................................................................


packaging: engine-backup: Fix apache reconfiguration

We want to remove this line from postinstall:

OVESETUP_APACHE/configured=bool:True

Escape '/'.

Without this patch, instead of being deleted, '/' was interepreted as
the end of the pattern, 'c' means 'replace', so it was replaced with:

onfigured=bool:True$/d

which was ignored by otopi, effectively being harmless.

Change-Id: If647f1ed1fa9e0499e79e4b2f5badcea45f27e9c
Bug-Url: https://bugzilla.redhat.com/1318580
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M packaging/bin/engine-backup.sh.in
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Yedidyah Bar David: Verified



-- 
To view, visit https://gerrit.ovirt.org/56272
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If647f1ed1fa9e0499e79e4b2f5badcea45f27e9c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list