Juan Hernandez has posted comments on this change. Change subject: packaging: spec: added jboss and postgresql deps ......................................................................
Patch Set 1: Code-Review+1 (1 comment) http://gerrit.ovirt.org/#/c/23251/1/packaging/ovirt-engine-reports.spec.in File packaging/ovirt-engine-reports.spec.in: Line 51: Line 52: Requires: jboss-as >= 7.1.1 Line 53: Requires: postgresql-jdbc Line 54: Requires: postgresql-server >= 8.4.7 Line 55: Requires: postgresql-contrib >= 8.4.7 I'm not sure if these are actually required. I mean, does any part of the reports or the setup actually need these packages? The postgresql-contrib package used to be required (by ovirt-engine) because we used PostgreSQL functions to generate UUIDs. But we no longer use those functions. Line 56: Line 57: # We need some packages installed in order to workaround Line 58: # bug 862355: Line 59: Requires: liberation-mono-fonts -- To view, visit http://gerrit.ovirt.org/23251 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I35a623c47225f7cb2c56b2e780cde29f4b09a88f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Juan Hernandez <[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
