Opened 3 years ago
Last modified 3 years ago
#61943 assigned defect
libsdl2 @2.0.14 +x11: Doesn't work properly without upstream XQuartz.app installed/with only MacPorts' `xorg-server`
| Reported by: | larb0b (Larkin) | Owned by: | jmroot (Joshua Root) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.6.4 |
| Keywords: | haspatch | Cc: | evanmiller (Evan Miller) |
| Port: | libsdl2 |
Description (last modified by larb0b (Larkin))
MacPorts' libsdl2 X11 variant doesn't work with only MacPorts' xorg-server installed. When trying to actually use it, you get:
Failed to create window: Failed loading /opt/X11/lib/libGL.1.dylib: dlopen(/opt/X11/lib/libGL.1.dylib, 10): image not found!
This is because that path is hardcoded within src/video/x11/SDL_x11opengl.c in libsdl2. This could be rectified by changing the path to MacPorts' libGL.1.dylib (installed by the mesa port). mesa should probably also be made a dependency when building the x11 variant of libsdl2.
Attachments (4)
Change History (9)
comment:1 Changed 3 years ago by larb0b (Larkin)
| Description: | modified (diff) |
|---|---|
| Summary: | libsdl2 @2.0.14 +x11: Doesn't build against MacPorts' GL → libsdl2 @2.0.14 +x11: Doesn't work properly without upstream XQuartz.app installed/with only MacPorts' `xorg-server` |
Changed 3 years ago by larb0b (Larkin)
| Attachment: | Portfile-libsdl2.diff added |
|---|
Changed 3 years ago by larb0b (Larkin)
| Attachment: | use-macports-gl.patch added |
|---|
Patch changing the default GL dylib from XQuartz' to MacPorts'
comment:2 Changed 3 years ago by larb0b (Larkin)
| Keywords: | haspatch added |
|---|
comment:3 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)
| Owner: | set to jmroot |
|---|---|
| Status: | new → assigned |
Thanks for reporting the issue. The patch provided is not usable as such because we would need to use whatever ${prefix} is and not assume that it is /opt/local.
Changed 3 years ago by larb0b (Larkin)
| Attachment: | Portfile-libsdl2.2.diff added |
|---|
Patch that adds mesa as a dependency when building the X11 variant + finds and replaces /opt/X11 with your prefix in SDL_x11opengl.c
comment:4 Changed 3 years ago by larb0b (Larkin)
I've attached a new patch which addresses that issue. It works fine in my testing.
Changed 3 years ago by larb0b (Larkin)
| Attachment: | Portfile-libsdl2.3.diff added |
|---|
Same as last attached patch but doesn't use -W in reinplace since we're just working w/ one file
comment:5 Changed 3 years ago by evanmiller (Evan Miller)
| Cc: | evanmiller added |
|---|
Patch adding mesa to the x11 variant's depends_lib