skia icon indicating copy to clipboard operation
skia copied to clipboard

Should we use python2 in the build instructions?

Open dacap opened this issue 5 years ago • 6 comments

Should we use python2 in the build instructions and workflow file?

Related to: https://github.com/aseprite/aseprite/pull/2090

dacap avatar May 25 '20 19:05 dacap

Not sure about this one, on Windows if I use python2 I got this error:

C:\deps\skia>python2 tools/git-sync-deps
'python2' is not recognized as an internal or external command,
operable program or batch file.

While python works correctly.

dacap avatar May 29 '20 03:05 dacap

so that i was using the wrong version of python? im using ubuntu when im typing python it's using python2 you have to explicitly type python3 to use python3 and i thought it was using python2..

alvinseptiano avatar Jun 04 '20 08:06 alvinseptiano

Skia uses/needs python 2. In my previous comment I said that we must use python (it should be pointing to python 2), but we cannot use python2 in the command line because it's not included in the Skia's depot_tools. Just a detail.

dacap avatar Jun 04 '20 14:06 dacap

The aseprite-m81 branch of skia in this repo seems to build fine for me using python 3.9.

ryandesign avatar May 19 '21 07:05 ryandesign

The aseprite-m81 branch of skia in this repo seems to build fine for me using python 3.9.

I was mistaken; #22 is an example of failure when using python 3.

ryandesign avatar Apr 25 '22 22:04 ryandesign

I think the biggest problem is that depot_tools is a "living thing" where Google updates for new Skia & Chrome, versions. I think they switched to Python 3 recently (but I might be wrong). Probably the best thing to do would be to update Skia to the latest version so it can use the latest depot_tools.

dacap avatar Apr 26 '22 13:04 dacap