Page MenuHome

CloudAddon: Build process
ClosedPublic

Authored by Jeroen Bakker (jbakker) on May 27 2019, 9:32 AM.

Details

Summary

setup.py used systemcalls for package management pip. This call is
platform dependent as on ubuntu distros this needs to be pip3. On
these platforms pip points to the python2 version.

By direct calling the pip module from within the running python process
we know for sure we are triggering the correct one.

Diff Detail

Event Timeline

Looks good to me.

We always work within a virtualenv, in which case python, pip etc. all point to the correct versions. Your approach avoids starting a subprocess and is automatically working on all environments, so good one!

This revision is now accepted and ready to land.May 27 2019, 11:55 PM
This revision was automatically updated to reflect the committed changes.