Created attachment 148595 [details]
- apply NDEBUG=1 when DEBUG is off.
- depend on libGL instead of libGLU.
- add missing -lX11 -lz to LDFLAGS.
Created attachment 148605 [details]
I'll take it.
I have doubt about the last line that add "-lz".
portsmon is showing irrlicht is building everywhere, including head. Why is -lz added? could it be obsolete? will it build without it?
(In reply to John Marino from comment #3)
> I have doubt about the last line that add "-lz".
> portsmon is showing irrlicht is building everywhere, including head. Why is
> -lz added? could it be obsolete? will it build without it?
The library has few unresolved symbols, which are visible when -Wl,--no-undefined is added to LDFLAGS (however, they are resolved via sub-dependencies).
Also, there is missed dependency to libXxf86vm - I'm not sure, what to do with it (turn it into option + possibly add Xrandr option or just add USE_XORG=x11 xxf86vm)
is there a major drawback to 'USE_XORG=x11 xxf86vm' solution? seems okay to me.
(In reply to John Marino from comment #5)
> is there a major drawback to 'USE_XORG=x11 xxf86vm' solution? seems okay to
No, it's ok (this is the default config). I think, that other options (like output via SDL) can be added later (on demand).
Created attachment 148942 [details]
Add USE_XORG=x11 xxf86vm
A commit references this bug:
Date: Sun Nov 2 18:15:21 UTC 2014
New revision: 372081
x11-toolkits/irrlicht: Fix DEBUG option, missing linker symbols
* apply NDEBUG=1 when DEBUG is off
* Depend on libGL instead of libGLU
* Add missing -lX11 -lZ to LDFLAGS
* Add USE_XORG= x11 xxf86vm
Submitted by: Andriy Voskoboinyk