FS#31331 - [sdl] current version of SDL causes joystick x-axis not to respond

Attached to Project: Arch Linux
Opened by Mikuro Kagamine (MikuroK) - Wednesday, 29 August 2012, 14:37 GMT
Last edited by Jan Alexander Steffens (heftig) - Monday, 03 September 2012, 15:34 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jan Alexander Steffens (heftig)
Architecture x86_64
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
In the current version of SDL (extra/sdl 1.2.15-1), my USB-connected sixaxis controller does not respond on the left stick x-axis (left/right), the previous version (extra/sdl 1.2.14-10) works fine.

Steps to reproduce:
>plug in sixaxis controller
>attempt to setup the controller in an SDL program (tested both pcsx2 and dolphin-emu)
>left analog stick x-axis does not respond, so cannot be set or used

Downgrading to extra/sdl 1.2.14-10 fixed the issue, from what i read elsewhere, a newer svn version of sdl fixes the issue, might I suggest rolling back to 1.2.14 until the next sdl release is available.

Also, this is also very likely to affect the i686 package (but I have not tested this).
This task depends upon

Closed by  Jan Alexander Steffens (heftig)
Monday, 03 September 2012, 15:34 GMT
Reason for closing:  None
Comment by Jan Alexander Steffens (heftig) - Friday, 31 August 2012, 17:00 GMT
If you find the patch in question (e.g. on http://hg.libsdl.org), then I might be able to apply it to our package.

I'm not rolling back to 1.2.14 just for this.
Comment by Mikuro Kagamine (MikuroK) - Saturday, 01 September 2012, 09:58 GMT
Fair enough, I have found a user-created patch for this problem here:
http://anonscm.debian.org/gitweb/?p=pkg-sdl/packages/libsdl1.2.git;a=blob;f=debian/patches/fix_joystick_misc_axes.diff;h=75e3755c93c83965a76c9f0541205855b3b8a27b;hb=856a810b8cd61fb371163a480a96c7f6c3e461a4

Which is linked from this bug report:
http://bugzilla.libsdl.org/show_bug.cgi?id=1460

I tried to patch it myself, but it didn't work, I might not have done it right though, I've never used abs/makepkg before.
Comment by Jan Alexander Steffens (heftig) - Saturday, 01 September 2012, 11:11 GMT
Here's a package with the patch applied: http://pkgbuild.com/~heftig/sdl-1.2.15-1-x86_64.pkg.tar.xz
Comment by Mikuro Kagamine (MikuroK) - Saturday, 01 September 2012, 12:07 GMT
That worked, thanks. if you don't mind, could you tell me how you added the patch? all I did was add the .diff file into the same folder as the PKGBUILD, and referenced it the same way as the existing patches in that folder (along with its MD5 hash), then built it. I have a feeling it's because it's named .diff instead of .patch, so it could be a different format which needs different handling, but any info (or a link to a wiki page) would be great.

Thanks again, look forward to going back to the usual extra/ version with the new patch.
Comment by Jan Alexander Steffens (heftig) - Saturday, 01 September 2012, 12:24 GMT
To add a patch:
1. Drop the file in the PKGBUILD dir
2. Edit the PKGBUILD
2.1. Add the file to the sources=() array
2.2. Add the checksum for the file to the used checksums arrays (e.g. md5sums=())
2.3. Add a "patch" command using the file to the build() function
3. makepkg

There's no difference between .diff and .patch.
Comment by Mikuro Kagamine (MikuroK) - Monday, 03 September 2012, 15:30 GMT
  • Field changed: Percent Complete (100% → 0%)
While the patch works in 64bit programs, 32bit programs which rely on multilib/lib32-sdl are also affected, so the patch needs to be applied there, too.

I have tested the same patch myself and it works.
Comment by Jan Alexander Steffens (heftig) - Monday, 03 September 2012, 15:34 GMT
Please don't reopen the bug.

Loading...