Yaniv Dary has posted comments on this change. Change subject: packaging: spec: added jboss and postgresql deps ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/23251/1/packaging/ovirt-engine-reports.spec.in File packaging/ovirt-engine-reports.spec.in: Line 45: Requires: jpackage-utils Line 46: Requires: ovirt-engine >= 3.4.0 Line 47: Requires: ovirt-engine-dbscripts >= 3.4.0 Line 48: Requires: ovirt-engine-dwh >= 3.4.0 Line 49: Requires: jasperreports-server >= 5.5.0 > We're going to remove the requires in -3, and we may need to add other chan Because server customizations are matched to version. Line 50: Requires: m2crypto Line 51: Line 52: Requires: jboss-as >= 7.1.1 Line 53: Requires: postgresql-jdbc 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 > the above is provided by engine dependency anyway... any reason to add it? I would remove this. 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
