Change in vdsm[ovirt-3.5]: spec: Only build x86_64 and ppc64 architectures

[Vinzenz Feenstra]

Vinzenz Feenstra has uploaded a new change for review.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

spec: Only build x86_64 and ppc64 architectures

We're not supporting any other architectures than x86_64 and ppc64
at the moment. This patch restricts the rpm builds to those
architectures.

Change-Id: Iab30dd4eba6fd068e9605312bfc1c90b03b01564
Signed-off-by: Vinzenz Feenstra 
Reviewed-on: http://gerrit.ovirt.org/32125
Reviewed-by: Dan Kenigsberg 
Tested-by: Michal Skrivanek 
(cherry picked from commit 565bb6cba89af6278e3d4e5ae03884d36a2d803e)
---
M vdsm.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/52/32152/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index ecac369..a25389e 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -61,6 +61,7 @@
 Source0:        %{vdsm_name}-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

+ExclusiveArch: x86_64 ppc64
 BuildRequires: gcc
 BuildRequires: python
 BuildRequires: python-devel

[Vinzenz Feenstra]

Vinzenz Feenstra has posted comments on this change.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

Patch Set 1: Verified+1

Yoav Kleinberger has posted comments on this change.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

Patch Set 1: Code-Review+1

I'm currently working on extracting arch-dependent aspects of vdsm to a different package.

http://gerrit.ovirt.org/gitweb?p=safelease.git;a=summary
http://gerrit.ovirt.org/gitweb?p=vdsm-arch-dependencies.git;a=summary

If we go this way, then we probably don't need these projects, and it will also take less time to get it going.

I'm for it.

Anyone else?

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

Patch Set 1: Code-Review+1

Yoav - I don't understand what you are saying.

On the long run, we want Vdsm to be noarch. As soon you can deliver this - the better. On the mean while, we have some issues with i686 and arm, which we'd rather not solve, and hence this patch.

Yaniv Bronhaim has posted comments on this change.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

Patch Set 1: Code-Review+2

Yaniv Bronhaim has submitted this change and it was merged.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

spec: Only build x86_64 and ppc64 architectures

We're not supporting any other architectures than x86_64 and ppc64
at the moment. This patch restricts the rpm builds to those
architectures.

Change-Id: Iab30dd4eba6fd068e9605312bfc1c90b03b01564
Signed-off-by: Vinzenz Feenstra 
Reviewed-on: http://gerrit.ovirt.org/32125
Reviewed-by: Dan Kenigsberg 
Tested-by: Michal Skrivanek 
(cherry picked from commit 565bb6cba89af6278e3d4e5ae03884d36a2d803e)
Reviewed-on: http://gerrit.ovirt.org/32152
Reviewed-by: Yoav Kleinberger 
Reviewed-by: Yaniv Bronhaim 
---
M vdsm.spec.in
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Vinzenz Feenstra: Verified
  Yoav Kleinberger: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, but someone else must approve

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: spec: Only build x86_64 and ppc64 architectures
......................................................................

Patch Set 2:

Build Successful

http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms_merged/123/ : SUCCESS