Skip to content
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

Build all dependencies for umu clients #143

Merged
merged 4 commits into from
Jul 11, 2024

Conversation

R1kaB3rN
Copy link
Member

@R1kaB3rN R1kaB3rN commented Jul 10, 2024

This pull request updates the Makefile to properly build all umu's Python dependencies for umu clients. Currently, python-xlib would be built, however the six module wouldn't which may cause Lutris users to crash due to the missing dependency.

To build all dependencies for clients like Lutris, we need to use a tool resolve all of the dependencies such as pip or uv and have them built in the Makefile.

R1kaB3rN added 2 commits July 10, 2024 14:22
- The Makefile currently builds python-xlib and the client is responsible for putting the module within their runtime directory. However, python-xlib depends on six.py and this will not get built for umu clients. To properly build dependencies, we need to use pip to resolve all of the dependencies and have them put in the object directory during the build process.
@R1kaB3rN R1kaB3rN marked this pull request as ready for review July 10, 2024 21:55
@R1kaB3rN R1kaB3rN force-pushed the umu-clients-build-deps branch from c8a5ea9 to 12d9d36 Compare July 10, 2024 22:01
@R1kaB3rN R1kaB3rN merged commit f7164c3 into Open-Wine-Components:main Jul 11, 2024
6 checks passed
@R1kaB3rN R1kaB3rN deleted the umu-clients-build-deps branch July 11, 2024 17:01
@R1kaB3rN R1kaB3rN mentioned this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant