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

Update Wiiuse library to 0.15.4, fixes support for newer Wiimotes on Linux #3626

Merged
merged 3 commits into from
Jan 3, 2019

Conversation

qwertychouskie
Copy link
Contributor

STK's Wiiuse had some modifications, but I tested and upstream seems to work fine, and fixes the -TR Wiimote.

@deveee
Copy link
Member

deveee commented Nov 26, 2018

Update is simple, but did you test it?

@qwertychouskie
Copy link
Contributor Author

Yes, and it works quite nicely, though for some reason Travis doesn't like it. Also AppVeyor fails but there should be an update to Wiiuse to address this soon.

@deveee
Copy link
Member

deveee commented Nov 26, 2018

Good. Fix travis is doable, most important is testing with real devices. I'm not sure if someone else has wii remote too...

@qwertychouskie qwertychouskie changed the title Update Wiiuse library to 0.15.3, fixes support for newer Wiimotes on Linux Update Wiiuse library to 0.15.4, fixes support for newer Wiimotes on Linux Nov 28, 2018
@qwertychouskie
Copy link
Contributor Author

Hmm. Both Travis and AppVeyor fail with not finding wiiuse.h, but compilation works fine on my system. @deveee Can you take a look and see why, and push a fix?

@deveee
Copy link
Member

deveee commented Nov 29, 2018

Previously source files were in lib/wiiuse, now there is a "src" directory. Changing
include_directories("${PROJECT_SOURCE_DIR}/lib/wiiuse/")
to
include_directories("${PROJECT_SOURCE_DIR}/lib/wiiuse/src")
in STK cmake file fixes compilation for me.

@qwertychouskie
Copy link
Contributor Author

OK, ready for merging.

@qwertychouskie
Copy link
Contributor Author

@deveee Any chance of getting this merged soon, since the compilation was fixed?

@deveee
Copy link
Member

deveee commented Dec 9, 2018

I think that it needs some testing by more people on different systems, but I can't do it myself.

@hiker hiker added this to the 0.10.0 milestone Dec 13, 2018
@MarioCMFlys
Copy link

MarioCMFlys commented Jan 3, 2019

Tested on Ubuntu-based Linux and works as expected

@hiker hiker merged commit 5031e7a into supertuxkart:master Jan 3, 2019
@hiker
Copy link
Contributor

hiker commented Jan 3, 2019

Thanks a lot for the patch, and for testing!!

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

Successfully merging this pull request may close these issues.

5 participants