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

tnisan at redhat.com tnisan at redhat.com
Tue May 3 12:20:38 UTC 2016


Tal Nisan 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/1332463
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
  Jenkins CI: Passed CI tests
  Yedidyah Bar David: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If647f1ed1fa9e0499e79e4b2f5badcea45f27e9c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan 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