Bug 1354155
| Summary: | SDL_Init fails when SDL_VIDEODRIVER=wayland | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Joseph Mullally <jwmullally> | ||||||
| Component: | SDL2 | Assignee: | Igor Gnatenko <ignatenko> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | medium | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | 24 | CC: | ignatenko, tcallawa | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | x86_64 | ||||||||
| OS: | Linux | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | SDL2-2.0.4-6.fc24 SDL2-2.0.4-6.fc23 | Doc Type: | If docs needed, set a value | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2016-07-17 00:52:19 UTC | Type: | Bug | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 1277927 | ||||||||
| Attachments: |
|
||||||||
|
Description
Joseph Mullally
2016-07-10 06:55:57 UTC
This is how I was compiling that test C code: $ gcc -o test_sdl test_sdl.c -g -lSDL2 -lGL -I/usr/include/SDL2 Created attachment 1178090 [details]
Updated version of the original patch
Created attachment 1178091 [details] 0001-fix-Wayland-dynamic-symbol-loading-bz1354155.patch Yup, it looks like this fix from https://bugzilla.libsdl.org/show_bug.cgi?id=3255 was missing another step added later here: https://bugzilla.libsdl.org/show_bug.cgi?id=3265 https://hg.libsdl.org/SDL/rev/330f500d5815 Attached an update to the patch: - SDL2-wayland-dyn-wl_proxy_marshal_constructor_versioned.patch (updated version of the original patch) - 0001-make-static-subpackage.patch (git format-patch) After applying, SDL_Init() works as expected and test applications seem to work OK, so this takes care of B) above. $ SDL_VIDEODRIVER=wayland ./test_sdl SDL_Init() succeeded. SDL_GetCurrentVideoDriver(): wayland $ cd ~/rpmbuild/BUILD/SDL2-2.0.4/test $ SDL_VIDEODRIVER=wayland ./testdisplayinfo INFO: Using video target 'wayland'. INFO: See 1 displays. INFO: 0: "SyncMaster" (1680x1050, (0, 0)), 1 modes. ... $ SDL_VIDEODRIVER=wayland ./testgl2 <works OK> One extra question: Are the SDL test applications built into any RPMs? They could be very useful to include in SDL2-devel as common reference tests for the upcoming wave of SDL wayland troubleshooting... SDL2-2.0.4-6.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-a9097aa363 SDL2-2.0.4-6.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-d7e30fd6ac (In reply to Joseph Mullally from comment #1) > This is how I was compiling that test C code: > > $ gcc -o test_sdl test_sdl.c -g -lSDL2 -lGL -I/usr/include/SDL2 Thanks a lot for contribution! SDL2-2.0.4-6.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-a9097aa363 SDL2-2.0.4-6.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-d7e30fd6ac SDL2-2.0.4-6.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report. SDL2-2.0.4-6.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. |