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

001-airconnect.1d.sh: Updates for Apple silicon #2058

Open
cuprousoxide opened this issue Sep 3, 2024 · 1 comment
Open

001-airconnect.1d.sh: Updates for Apple silicon #2058

cuprousoxide opened this issue Sep 3, 2024 · 1 comment

Comments

@cuprousoxide
Copy link

cuprousoxide commented Sep 3, 2024

fao @CartoonChess -

Looks like the default process name has changed from aircast-osx-multi to aircast-macos.
Also, line 75 has said process name hardcoded in a grep instead of referring to the processName variable defined up top by the user.
I'd do a PR myself for those two little items, but I also had to add a line to export my DYLD_LIBARY_PATH otherwise I get an error about libcrypto and the plugin doesn't actually work. It's my understanding this has to do with Homebrew putting packages somewhere different on Apple silicon (which I use). I'm a layman and not a developer and I'm not sure what the best practice is for handling Intel vs Apple issues like this other than adding a line for m* chip users to uncomment?

@sprak3000
Copy link
Collaborator

@cuprousoxide

You might need to set the shebang to include both the M* silicon and not. Here is an example from a recent open PR:

#!/usr/bin/env -S PATH="${PATH}:/opt/homebrew/bin:/usr/local/bin" php

The -S PATH="" sets up the script's path to include /opt/homebrew/bin for M* silicon and /usr/local/bin for older machines.

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

No branches or pull requests

2 participants