-
Notifications
You must be signed in to change notification settings - Fork 9
Multi-platform support #2
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
Conversation
Revisited PySide2 shipping and added build script. Windows build is still included for the time being.
Moved publish2 hooks from tk-config-unreal to the framework with supporting files.
Heads up that this PR has now further changes from our latest round of work: publish2 tk-maya hooks are now shipped with this framework rather than with tk-config-unreal to allow easy updates of the config. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work, just some comments and a suggestion for validating the existence of an attribute using hasattr.
"python", | ||
"vendors", | ||
"py%d" % python_major, | ||
pname, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to double check, currently there's not python/vendors folder and base_path is the root path of the framework?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, there was only the Windows binaries, that I removed for the CR.
"site-packages" | ||
) | ||
|
||
os.environ["VIRTUAL_ENV"] = base_path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A friendly request: can you explain a little bit the strategy with virtualenv?. It's not usual in toolkit play with virtualenv specially when distributing components
hooks/tk-multi-publish2/tk-maya/unreal/unreal_setup_turntable.py
Outdated
Show resolved
Hide resolved
hooks/tk-multi-publish2/tk-maya/unreal/unreal_setup_turntable.py
Outdated
Show resolved
Hide resolved
Closing in favor of the PR against gpltech branch |
* For #13673, support for Maya 2025 Removed all six dependencies. Added support for PySide6.
Revisited PySide2 shipping and added build script.
Added Azure pipeline script to build releases for Windows, Osx and Linux.
Note: Windows build is still included for the time being.