Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Make the integration work "out of the box" #21

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

Closed
exodrifter opened this issue Oct 5, 2019 · 2 comments
Closed

Make the integration work "out of the box" #21

exodrifter opened this issue Oct 5, 2019 · 2 comments

Comments

@exodrifter
Copy link
Owner

exodrifter commented Oct 5, 2019

Right now, the Python standard library is not included in the build. The reason for this is because the IronPython library only exposes a method for loading python libraries from the disk (as far as I know). We can accomplish this by putting the Python library in the StreamingAssets folder, but this is a security problem since the library can be trivially edited by any user to do whatever they might want. This is the main reason for why the included python library in unity-python doesn't work in builds right out of the box.

Several issues are affected by this problem:

I did find an approach that looks appealing recently... It might be possible to compile the Python libraries into DLLs.

If this works the way I think it does, then I can make an interface for installing Python libraries which fetches the package from pip, compiles it into a DLL, and then adds it to your project. This would fix all of the aforementioned issues.

However, it would be best to keep in mind that Python 2 is being deprecated on January 1st, 2020 with pip to follow suit shortly after. I have never tested the library as it stands today with Python 3 (and I don't have any reason to believe it will work). So, perhaps this work will have to wait until the library is ported to work with Python 3.

@Lingfeng158
Copy link

Thank you very much! Your work is much needed! May I know the ETA of this integration update?

@exodrifter
Copy link
Owner Author

@Lingfeng158 I work on this in my free time so I'm unable to give an ETA. I've been thinking about this issue for quite a while, thank you for expressing interest. ❤️

Repository owner locked and limited conversation to collaborators Jan 20, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants