search

Lord-Kamina / SwiftDefaultApps

Replacement for RCDefaultApps, written in Swift.
Other
1.45k stars 62 forks source link

Setting default email client and browser presents Illegal instruction: 4 #13

Closed apizz closed 5 years ago

apizz commented 5 years ago

(BTW, this is awesome) Using the swda CLI tool.

Following the usage guide for setHandler - swda setHandler <application> <type> [options]- I am running the following commands to set the default email client & the default browser.

swda setHandler --app /Applications/Microsoft\ Outlook.app --email swda setHandler --app /Applications/Google\ Chrome.app --browser

In both cases, the result is "Illegal instruction: 4". In the case of the browser, however, this does produce the prompt to select the alternative browser or keep the current option.

I've tried changing the order of the parameters but I can't seem to get this to work using --email or --browser. Was trying to use what appeared to be a simpler option for configuring this, rather than --UTI.

Am I missing something obvious when using setHander with --email & --browser??

Lord-Kamina commented 5 years ago

I'll check it out and get back to you. Do you by any chance have a crash log or stack trace?

apizz commented 5 years ago

@Lord-Kamina here is the crash log swda_2019-07-24-124042.crash.zip

FYI - this is on a machine that I updated to 10.14.6, which was released on the 22nd.

Lord-Kamina commented 5 years ago

I've already found and fixed the bug; will try to release the fix during the weekend.

P.S. The bug is limited to types such as web, browser, mail and such; most of which are actually shortcuts for the real scheme and it occurs either when showing the result message, which happens after actually trying to change the association (which is why changing browser to Chrome works regardless of the error message. P.P.S. You don't need to specify the full path to an app, you can use its name or Bundle identifier also.

Lord-Kamina commented 5 years ago

0dd7b0be7627df6dfd929e396209b1cc82957f18 should fix this now. @apizz: Check https://github.com/Lord-Kamina/SwiftDefaultApps/releases/tag/v2.0.1

apizz commented 5 years ago

@Lord-Kamina I can confirm with 2.0.1 I do not get the error message when setting either default browser or default email client.

Thanks!