Yaniv Dary has posted comments on this change. Change subject: packaging: setup: do not modify jasperserver files ......................................................................
Patch Set 3: (1 comment) http://gerrit.ovirt.org/#/c/23317/3/packaging/legacy-setup/ovirt-engine-reports-setup.py File packaging/legacy-setup/ovirt-engine-reports-setup.py: Line 241 Line 242 Line 243 Line 244 Line 245 > anyway, I remove postgresql* to be on safe side, although we modify the dat We also use this driver for report's data source, so we need to make sure it's not in the classpath. I'll talk about this with Barak tomorrow because this was changed like this as a security bug fix -- To view, visit http://gerrit.ovirt.org/23317 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7eaa6f20d444b82161571d2abc2d7b9f78faa626 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Yaniv Dary <[email protected]> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
