Unified moved_monitor_wait() for moved - #520
Merged
Merged
Conversation
adangert
reviewed
Jul 31, 2026
adangert
reviewed
Jul 31, 2026
adangert
reviewed
Jul 31, 2026
Contributor
|
Thanks for getting this out, when I boot the pi up again, I can test out with the newest Joustmania on linux (we've upgraded to the newest psmoveapi), and will report back |
thp
commented
Aug 3, 2026
adangert
approved these changes
Aug 4, 2026
Contributor
There was a problem hiding this comment.
Tested regular paring on Linux with both psmove controller types. still works well.
On the windows build running through Proton on Steam Deck: JoustMania starts a native Linux psmove daemon. The Windows psmoveapi.dll communicates with it using UDP on 127.0.0.1:17778,
I also tested the native Windows JoustMania build, which uses moved_monitor_wait(monitor, false). Controller detection works well. Thanks for working on this!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A cleanup on top of #519 for unifying monitor waiting between macOS, Linux and Windows.
Tested:
Not tested:
psmove pair -don any OSpsmove daemonon any OScc @adangert if you could check if this works for you, that'd be great :)