Doron Fediuck has submitted this change and it was merged. Change subject: tools: engine-notifier accept special chars ......................................................................
tools: engine-notifier accept special chars Due to the way configuration was loaded in the bash script special characters would create some unwanted behaviors. Now the configuration file is read and the variables are added using "declare" statement It is expected that the config file is of the form key=value Change-Id: I5e6419a225bd02edaf1db38a2025dee74761ddf2 Bug-Url: https://bugzilla.redhat.com/883301 Signed-off-by: Noam Slomianko <[email protected]> --- M backend/manager/tools/src/main/shell/engine-notifier.sh 1 file changed, 15 insertions(+), 2 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10704 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e6419a225bd02edaf1db38a2025dee74761ddf2 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Noam Slomianko <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: ofri masad <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
