Put python3 from the system before python3 from dev-utils/bin in the PATH
Needs ReviewPublic

Authored by albertvaka on Mar 7 2019, 10:06 AM.

Details

Reviewers
vonreth
Group Reviewers
Craft
Summary

For me, when craft tries to update python, it fails because the binary is
in use by craft itself. This patch aims to fix that, by using the python3
binary specified in CraftSettings.ini

Diff Detail

Repository
R138 Craft
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9274
Build 9292: arc lint + arc unit
albertvaka requested review of this revision.Mar 7 2019, 10:06 AM
albertvaka created this revision.

I'm not sure this is the good solution, or if something is broken in my setup and I'm the only one experiencing this problem.

kfunk added a subscriber: kfunk.Mar 7 2019, 10:23 AM

FTR: I think I experienced the same or a similar issue.

I had it too.
Your issue is on Windows? Actually craft should use the one from the settings directly and not python from the path.

Yes, the issue happens on Windows. And I think it always happens, because the line before the one I added reads: "prefix the path with dev-utils/bin"... and Craft's python3.exe is installed right there.

Any update on this? Is it good to merge?

Sorry, forgot about it.
Which package is actually broken? python?
When was your craft install?