Alon Bar-Lev has posted comments on this change.

Change subject: packaging: pull vdsm-jsonrpc-java-1.0.2
......................................................................


Patch Set 1: Verified-1

(1 comment)

this cannot be merged until saggi pushes his patches and jenkins builds 1.0.2 
and this published somehow into 3.5 as a release, I have the tarball, but you 
guys disabled the manual tarball build... so the source of binary build is not 
the release (source tarball).

again, this is misunderstanding of what open source is, and the meaning of 
release of sources.

apart from having all ready I cannot do much about it.

downstream is ready and will work with this patch.

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

Line 547: Requires:     %{name}-setup-plugin-ovirt-engine = 
%{version}-%{release}
Line 548: Requires:     openssh-server
Line 549: Requires:     ovirt-host-deploy >= 1.3.0
Line 550: Requires:     ovirt-host-deploy-offline >= 1.3.0
Line 551: Requires:     vdsm-jsonrpc-java >= 1.0.2
sandro, why all-in-one needs explicit dependency on that?
Line 552: 
Line 553: %description setup-plugin-allinone
Line 554: All-in-one plugin for %{ovirt_product_name_short}'s setup
Line 555: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icd7e932319fa5b07de2c05023d4fb1d5db97bad7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: David Caro <[email protected]>
Gerrit-Reviewer: Eyal Edri <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to