New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 820717 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 777069
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

gsutil's vpython does not use python 2.7 by default

Reported by ziraker...@gmail.com, Mar 10 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.146 Safari/537.36

Steps to reproduce the problem:
1. Symlink /bin/python to something other than python2 (e.g. python3 or old version of python2)
2. Compile chromium from scratch

What is the expected behavior?
All Chromium tools should run in environments not dependent on the host's /bin/python symlink, but on its own version of python 2.7.

Alternatively, the python found most recently in $PATH should be used to enable virtualenvs.

What went wrong?
In the case of python 3 hosts (e.g. arch),
gsutil imports urllib2 which is not available, and the build will exit with syntax errors.

Additionally, when gsutil's dependencies are being met and an unexpected python version is used, it'll attempt to download a file like "infra/python/wheels/cffi/linux-amd64_cp32_abi3" (notice cp3, that's the *host* python version) which is not registered. Example log of my own: https://gist.github.com/Zirak/2c20fd63c4d72bef65da53e1ac23c8ae This has been happening in the wild as well: https://aur.archlinux.org/packages/v8/?comments=all , ryanvade's comment in 2018-01-03 04:27.

Did this work before? N/A 

Chrome version: 65.0.3325.146  Channel: stable
OS Version: 
Flash Version: 

I've patched depot_tools to include the necessary python version, but it may be an indicator of a deeper bug in vpython: Even when configuring a virtualenv of your own, the host python is grabbed and not the virtualenv's.
 
Cc: sindhu.chelamcherla@chromium.org
Components: Platform>Apps>Default>Hterm
Labels: Triaged-ET NeedsTriage-M65 TE-NeedsTriageHelp
This seems to out of scope of TE. Hence adding TE-NeedsTriageHelp label. Could someone from dev team please have a look at this issue. 

Tentatively adding suggested component Platform>Apps>Default>Hterm .

Thanks!

Comment 3 by vapier@chromium.org, Mar 12 2018

Components: -Platform>Apps>Default>Hterm Infra>SDK
Labels: -TE-NeedsTriageHelp -Via-Wizard-Other -NeedsTriage-M65
no relationship to hterm here.  this is on the Chromium infra side.
The issue seems to be out of TE-scope as it is related to infra. Hence, requesting someone from Infra>SDK team to please help us in triaging this further.

Thanks...!!
Mergedinto: 777069
Status: Duplicate (was: Unconfirmed)
Thanks for raising this issue. It is already being tracked in a different bug.

Marking as a dupe.

Sign in to add a comment