Sandro Bonazzola has posted comments on this change.

Change subject: packaging: spec: fix missing dep on JDK 1.7
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/33161/1/ovirt-engine.spec.in
File ovirt-engine.spec.in:

Line 177: 
Line 178: BuildArch:    noarch
Line 179: 
Line 180: BuildRequires:        java-devel
Line 181: BuildRequires:        java7-devel
> not sure you need to have both
you were both right, 1.6 is not needed, 1.7 is required.
Line 182: BuildRequires:        jpackage-utils
Line 183: BuildRequires:        make
Line 184: BuildRequires:        python
Line 185: %if %{ovirt_require_maven}


-- 
To view, visit http://gerrit.ovirt.org/33161
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia71fb7f6c21f3f7ce26a1a0cc1ab8608dfa224c3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Lev Veyde <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Simone Tiraboschi <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to