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

feat: get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment #190

Merged
merged 2 commits into from
Dec 23, 2024

Conversation

muvaf
Copy link
Contributor

@muvaf muvaf commented Dec 18, 2024

When usbmuxd is run without root user, it listens on a different path. usbmuxd itself uses this env var if it's given.

Tested manually with a real iPhone.

Copy link

linux-foundation-easycla bot commented Dec 18, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@mykola-mokhnach
Copy link
Contributor

@muvaf I don't see any changes since the last commit. Have you pushed your fixes?

@muvaf
Copy link
Contributor Author

muvaf commented Dec 23, 2024

@mykola-mokhnach I was waiting on your take for this question, just pushed the change after testing.

@muvaf muvaf changed the title Get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment fix: get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment Dec 23, 2024
@muvaf
Copy link
Contributor Author

muvaf commented Dec 23, 2024

@mykola-mokhnach I think you need to manually approve the workflows.

@mykola-mokhnach mykola-mokhnach merged commit 9a5be45 into appium:master Dec 23, 2024
5 checks passed
@mykola-mokhnach mykola-mokhnach changed the title fix: get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment feat: get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment Dec 23, 2024
github-actions bot pushed a commit that referenced this pull request Dec 23, 2024
## [2.8.0](v2.7.27...v2.8.0) (2024-12-23)

### Features

* get usbmuxd socket address from USBMUXD_SOCKET_ADDRESS environment ([#190](#190)) ([9a5be45](9a5be45))
Copy link

🎉 This PR is included in version 2.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jlipps
Copy link
Member

jlipps commented Jan 8, 2025

Hi @muvaf, congrats: the Appium project wants to compensate you for this contribution! Please reply to this comment mentioning @jlipps and @KazuCocoa and sharing your OpenCollective account name, so that we can initiate payment! Or let us know if you decline to receive compensation via OpenCollective. Thank you!

@muvaf
Copy link
Contributor Author

muvaf commented Jan 9, 2025

@jlipps @KazuCocoa Thanks, my account handle is muvaf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released size:S contribution size: S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants