Dan Kenigsberg has posted comments on this change. Change subject: Vdsm supports only x86, ppc and ppc64 ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/62465/2/vdsm.spec.in File vdsm.spec.in: Line 57: Url: http://www.ovirt.org/develop/developer-guide/vdsm/vdsm/ Line 58: Source0: %{vdsm_name}-%{version}.tar.gz Line 59: BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) Line 60: Line 61: ExclusiveArch: x86_64 ppc ppc64 > I don't recall why, but before https://gerrit.ovirt.org/40738 we had a diff answering myself: Fedora requires building on all its supported archs. http://gerrit.ovirt.org/33368 introduced this. If we want to break this rule, we should "blame" the other packages missing on non-intel archs in the commit message. Line 62: Line 63: %{!?_licensedir:%global license %%doc} Line 64: Line 65: BuildRequires: cyrus-sasl-lib -- To view, visit https://gerrit.ovirt.org/62465 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic6f7a7d9aac6033b60fe592749bab2613b62466a Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com> Gerrit-Reviewer: Jenkins CI 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/admin/lists/vdsm-patches@lists.fedorahosted.org