Yaniv Bronhaim has posted comments on this change. Change subject: spec: Move vdsm package contents to vdsm-deamon ......................................................................
Patch Set 7: (1 comment) https://gerrit.ovirt.org/#/c/42491/7/vdsm.spec.in File vdsm.spec.in: Line 255: Requires: genisoimage Line 256: Requires: libselinux-python Line 257: Line 258: # Vdsm required sub packages Line 259: Requires: %{name}-daemon = %{version}-%{release} > Yea, I also dislike this dependencies here, I'll keep only vdsm-daemon. vdsm-daemon must require vdsm.rpm as well. yes. only the subpackages that have arch dependency will be built in the specific arch. the rest will be built whereever the build system is using Line 260: Requires: %{name}-hook-vmfex-dev = %{version}-%{release} Line 261: Requires: %{name}-infra = %{version}-%{release} Line 262: Requires: %{name}-jsonrpc = %{version}-%{release} Line 263: Requires: %{name}-python = %{version}-%{release} -- To view, visit https://gerrit.ovirt.org/42491 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I58b0b0e7461f9ac10a189dd8ca6715d02183ad21 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com> Gerrit-Reviewer: Nelly Credi <ncr...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches