diff mbox series

[ovs-dev] travis: Use pip2 instead of pip for OSX build.

Message ID 1512737590-2092-1-git-send-email-i.maximets@samsung.com
State Accepted
Headers show
Series [ovs-dev] travis: Use pip2 instead of pip for OSX build. | expand

Commit Message

Ilya Maximets Dec. 8, 2017, 12:53 p.m. UTC
xcode8.3 is a new default image for OS X on Travis-CI, but
it does not have 'pip':

    pip install --user six
    ./.travis/osx-prepare.sh: line 3: pip: command not found

'pip2' or 'pip3' should be used explicitly instead:

    https://github.com/travis-ci/travis-ci/issues/8829

Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
---
 .travis/osx-prepare.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Alin-Gabriel Serdean Dec. 11, 2017, 2:23 p.m. UTC | #1
> -----Original Message-----
> From: ovs-dev-bounces@openvswitch.org [mailto:ovs-dev-
> bounces@openvswitch.org] On Behalf Of Ilya Maximets
> Sent: Friday, December 8, 2017 2:53 PM
> To: ovs-dev@openvswitch.org
> Cc: Ilya Maximets <i.maximets@samsung.com>; Lance Richardson
> <lrichard@redhat.com>; Heetae Ahn <heetae82.ahn@samsung.com>
> Subject: [ovs-dev] [PATCH] travis: Use pip2 instead of pip for OSX build.
> 
> xcode8.3 is a new default image for OS X on Travis-CI, but it does not
have
> 'pip':
> 
>     pip install --user six
>     ./.travis/osx-prepare.sh: line 3: pip: command not found
> 
> 'pip2' or 'pip3' should be used explicitly instead:
> 
>     https://github.com/travis-ci/travis-ci/issues/8829
> 
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>

I haven't tested it, but I saw other people complaining about the same issue
on travis-ci.

Acked-by: Alin Gabriel Serdean <aserdean@ovn.org>
Alin-Gabriel Serdean Dec. 12, 2017, 12:02 a.m. UTC | #2
I applied this on master.

> -----Original Message-----
> From: ovs-dev-bounces@openvswitch.org [mailto:ovs-dev-
> bounces@openvswitch.org] On Behalf Of aserdean@ovn.org
> Sent: Monday, December 11, 2017 4:23 PM
> To: 'Ilya Maximets' <i.maximets@samsung.com>; ovs-dev@openvswitch.org
> Cc: 'Lance Richardson' <lrichard@redhat.com>; 'Heetae Ahn'
> <heetae82.ahn@samsung.com>
> Subject: Re: [ovs-dev] [PATCH] travis: Use pip2 instead of pip for OSX
build.
> 
> > -----Original Message-----
> > From: ovs-dev-bounces@openvswitch.org [mailto:ovs-dev-
> > bounces@openvswitch.org] On Behalf Of Ilya Maximets
> > Sent: Friday, December 8, 2017 2:53 PM
> > To: ovs-dev@openvswitch.org
> > Cc: Ilya Maximets <i.maximets@samsung.com>; Lance Richardson
> > <lrichard@redhat.com>; Heetae Ahn <heetae82.ahn@samsung.com>
> > Subject: [ovs-dev] [PATCH] travis: Use pip2 instead of pip for OSX
build.
> >
> > xcode8.3 is a new default image for OS X on Travis-CI, but it does not
> have
> > 'pip':
> >
> >     pip install --user six
> >     ./.travis/osx-prepare.sh: line 3: pip: command not found
> >
> > 'pip2' or 'pip3' should be used explicitly instead:
> >
> >     https://github.com/travis-ci/travis-ci/issues/8829
> >
> > Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> 
> I haven't tested it, but I saw other people complaining about the same
issue
> on travis-ci.
> 
> Acked-by: Alin Gabriel Serdean <aserdean@ovn.org>
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff mbox series

Patch

diff --git a/.travis/osx-prepare.sh b/.travis/osx-prepare.sh
index 1f144fb..4725fd8 100755
--- a/.travis/osx-prepare.sh
+++ b/.travis/osx-prepare.sh
@@ -1,7 +1,7 @@ 
 #!/bin/bash
 set -ev
-pip install --user six
-pip install --user --upgrade docutils
+pip2 install --user six
+pip2 install --user --upgrade docutils
 
 brew update || true
 brew uninstall libtool && brew install libtool || true