Skip to content

Adding Note for LM22 and how to add wine64 to the PATH variable. #34

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
wants to merge 1 commit into from

Conversation

Kwyrky
Copy link

@Kwyrky Kwyrky commented Aug 1, 2024

The script mentioned on this page https://docs.monogame.net/articles/getting_started/1_setting_up_your_development_environment_unix.html at the bottom did not work because it could not find the wine64 binary. To make it work the correct directory of the wine64 binary needs to be added either to the PATH variable or a symlink needs to be added to a directory which is already in the current PATH.

@harry-cpp
Copy link
Member

The script should be updated to try and look for wine64 in a separate folder instead of modifying the docs.

Closing this and opening up #35 so we can keep track of this issue.

Thanks for reporting the issue.

@harry-cpp harry-cpp closed this Aug 1, 2024
@AristurtleDev
Copy link
Contributor

The script should be updated to try and look for wine64 in a separate folder instead of modifying the docs.

@harry-cpp the setup script checks for the existence of wine64 by using the command itself. If it’s installed correctly, which means it’s added to $PATH, or in a directory that’s in $PATH, then that verifies it:

If the script is updated to check if it’s installed in different locations, that will work, but then later mgcb will fail when it attempts to compile shaders because the command isn’t in $PATH.

Users should ensure the wine64 command is valid in $PATH, the script shouldn’t be updated to check various locations. We can’t account for every possible location a user may install it.

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.

3 participants