On 27/06/12 04:13, Alex Jia wrote:
On 06/18/2012 06:01 PM, Alex Jia wrote:
This is a user case if users build virt-v2v on x86_64 platform
and only want to try V2V section then they must see P2V rpm
build error, it's not necessary for V2V users.

In addition, if some automation scripts call autobuild.sh on
x86_64 platform then the programming also will exit due to
the buiding error.


Hi Matt, any advise about this? thanks. I guess you have other good idea ;)

Sorry, Alex. I missed this first time round.

ACK and pushed.

Matt


Signed-off-by: Alex Jia<[email protected]>
---
  autobuild.sh |   15 +++++++++++----
  1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/autobuild.sh b/autobuild.sh
index 21886a7..7d07cbd 100755
--- a/autobuild.sh
+++ b/autobuild.sh
@@ -50,6 +50,10 @@ fi
  rm -f $NAME-*.tar.gz
  ./Build dist

+if [ -z "$ARCH" ]; then
+    ARCH=`uname -p`
+fi
+
  if [ -f /usr/bin/rpmbuild ]; then
    if [ -n "$AUTOBUILD_COUNTER" ]; then
      EXTRA_RELEASE=".auto$AUTOBUILD_COUNTER"
@@ -63,10 +67,13 @@ if [ -f /usr/bin/rpmbuild ]; then
             --define "extra_release $EXTRA_RELEASE" \
             --clean virt-v2v.spec

-  rpmbuild --nodeps -ba --target i686 \
-           --define "_sourcedir `pwd`" \
-           --define "extra_release $EXTRA_RELEASE" \
-           --clean rubygem-virt-p2v.spec
+  # virt-p2v only can be built on i686 platform now
+  if [ "$ARCH" = "i686" ]; then
+      rpmbuild --nodeps -ba --target $ARCH \
+               --define "_sourcedir `pwd`" \
+               --define "extra_release $EXTRA_RELEASE" \
+               --clean rubygem-virt-p2v.spec
+  fi
  fi

  exit 0

_______________________________________________
Libguestfs mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libguestfs


--
Matthew Booth, RHCA, RHCSS
Red Hat Engineering, Virtualisation Team

GPG ID:  D33C3490
GPG FPR: 3733 612D 2D05 5458 8A8A 1600 3441 EA19 D33C 3490


_______________________________________________
Libguestfs mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libguestfs

Reply via email to