[Patchew-devel] [PATCH 2/5] deploy: install Python 3 pip

[Fam Zheng]

On Fri, 03/16 17:44, Paolo Bonzini wrote:
> On 16/03/2018 03:52, Fam Zheng wrote:
> > On Tue, 03/13 18:45, Paolo Bonzini wrote:
> >> We do not really need the Python 2.x pip for anything.  In the end
> >> Python 3.x pip is probably already there because Fedora has it as
> >> a dependency of python3, but let's just be clean and install it.
> >>
> >> Signed-off-by: Paolo Bonzini <pbonzini at redhat.com>
> >> ---
> >>  scripts/playbooks/tasks/docker-deploy.yml | 6 +++---
> >>  1 file changed, 3 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/scripts/playbooks/tasks/docker-deploy.yml b/scripts/playbooks/tasks/docker-deploy.yml
> >> index 21c8a8d..ec04306 100644
> >> --- a/scripts/playbooks/tasks/docker-deploy.yml
> >> +++ b/scripts/playbooks/tasks/docker-deploy.yml
> >> @@ -1,10 +1,10 @@
> >>  ---
> >> -- name: Install pip
> >> -  package:
> >> -    name: python2-pip
> >>  - name: Install Python 3
> >>    package:
> >>      name: python34
> >> +- name: Install pip
> >> +  package:
> >> +    name: python3-pip
> >>  - name: Install docker
> >>    package:
> >>      name: docker
> >> -- 
> >> 2.14.3
> >>
> > 
> > I've reverted this one. It breaks deploying to RHEL 7.
> 
> Would the right fix be simply to remove the task, and not install either
> python2-pip or python3-pip?

Better to use a fallback method. Will send a patch.

Fam