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

Fix Discord invites for paths with spaces #188

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Flower35
Copy link
Contributor

@Flower35 Flower35 commented Jul 9, 2024

Reworked the platform-dependent executable paths code.

Testing required for cases when the game is installed in different directories, with Discord Activity enabled:

  • Windows
    • simple paths (like C:\Games\sm64coopdx\)
    • paths with spaces and unicode characters (like C:\users\Użytkownik\Desktop\My Games\Super Mario 64 Coop DX\)
  • Linux
    • simple paths (like /home/user/Games/sm64coopdx/)
    • paths with spaces (like /home/user/Desktop/My Games/Super Mario 64 Coop DX/)
  • macOS

I can only confirm that this compiles for Linux and Windows, but I cannot test on macOS and I cannot test the Discord Invites feature at the moment.

@Isaac0-dev
Copy link
Collaborator

I can only confirm that this compiles for Linux and Windows, but I cannot test on macOS and I cannot test the Discord Invites feature at the moment.

Can anyone check this?
We need to know:

  • Do Discord invites work on this PR, specifically when coop is in a folder which has spaces in its name
  • Does this work wtih macOS

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.

2 participants