pysciter icon indicating copy to clipboard operation
pysciter copied to clipboard

Suggestion not to download the whole SDK but only the required library

Open 8ctopus opened this issue 4 years ago • 3 comments

I've noticed that only the sciter library is actually required for pysciter to work so it would be possible to use a small script to download the necessary files. Here's an example Let me know if you're interested, I'll happily make it.

8ctopus avatar Nov 14 '21 13:11 8ctopus

I ended up doing anyway https://github.com/8ctopus/pysciter-example

8ctopus avatar Nov 14 '21 13:11 8ctopus

Yeah. In fact, that is exactly what we do in CI:

https://github.com/sciter-sdk/pysciter/blob/f5f00112d635f9d1ee98661eeebb2afcaa56c1f4/.travis.yml#L58-L60

pravic avatar Nov 15 '21 07:11 pravic

Thanks for sharing, I see now.

IMO it would suggest remove the requirement to download the whole SDK from the README and add it to the path. Maybe just me, but I don't like big downloads and adding things to PATH. The script installation would provide an alternative.

Also I don't understand what you mean by

...install Sciter shared library to your LIBRARY_PATH.

What's your opinion?

8ctopus avatar Nov 15 '21 08:11 8ctopus