-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] Change python3 to python #7
Comments
Hmm, we have Chromium builds on Windows, and what worked was running both scripts with
So Windows can run I'm a wary that the change might backfire with existing users, e.g. Brave. To make it backward compatible we'd probably need to clone both scripts and have a |
hm weird! So when I installed Python officially there was only TL;DR: I have a workaround for now, so happy to close, just confusing and I thought python is what tools use nowadays. What convention does Chromium use upstream? Looks like |
AFAIK, Chromium migrated to However: I just realized that I think I can fix your problem with just this: #8 |
I can confirm that this works for us. Appreciate! |
If you install Python on a Windows machine, it usually is only available as
python
and not aspython3
. This will then e.g. break the following:reclient-configs/configure_reclient.py
Line 144 in fe37dd8
We can probably expect nowadays machines have Python3 as
python
. Would you be up for a contribution to changepython3
->python
?Happy for more thoughts on this, thanks!
The text was updated successfully, but these errors were encountered: