-
-
Notifications
You must be signed in to change notification settings - Fork 480
fix: add default path for crossover on macOS #3728
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
base: main
Are you sure you want to change the base?
Conversation
If user disabled spotlight, `mdfind` will not work. Fixes Heroic-Games-Launcher#3025
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
crossoverMacPath.add(crossoverAppPath) | ||
} | ||
|
||
// search for crossover installed around the system | ||
await execAsync( |
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.
Have you tested to see if it won't duplicate the entry on the wine list?
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.
I tested it on my machine with Spotlight disabled; it has only one entry of crossover.
I cannot test for duplicates as I do not have Spotlight (mdfind
) enabled.
However, since crossoverMacPath
is a Set
, it should ignore duplicate entries.
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.
Yeah, good point.
I will try to test on a macOS this week just to be sure.
Can you include the same for Whisky? The same thing not finding it if spotlight index disabled. Ref. Discord thread |
I could not open the Discord link you provided(maybe because I am not in that certain channel). But I assume the default install path for Whisky is However, the implementation of |
I have no idea about the wine path but the main issue to start with relies on this line |
This comment was marked as resolved.
This comment was marked as resolved.
* chore: improve whisky discoverability on mac #3728 (comment) * style: update path for whisky wine
If user disabled Spotlight,
mdfind
will not work./Applications/CrossOver.app
is the default install location forCrossOver
, therefore adding it here as a backup.Fixes #3025
Use the following Checklist if you have changed something on the Backend or Frontend: