Re: [PATCH 2/2] gnu: Add manaplus.

[Alex Kost]

Ludovic Courtès (2015-10-09 15:40 +0300) wrote:

> Alex Kost <address@hidden> skribis:
>
>> I don't know if there is a better workaround for the SDL headers, but
>> this problem is rather common: there is a workaround in 'abbaye' and a
>> special patch for 'pingus'.
>>
>> The problem is: the source code has lines like this:
>>
>>   #include <SDL_mixer.h>
>>
>> but the headers of all SDL packages are placed in “include/SDL/”
>> subdirectories.  And an upstream often doesn't use "pkg-config" for
>> every SDL package to define CFLAGS and assumes that all SDL headers are
>> placed in one directory.  So we have to invent workarounds for such
>> packages.
>>
>> I just mention this problem here, perhaps someone will come up with a
>> general solution.
>
> Can’t ‘sdl-union’ be used here?  I think it was created specifically to
> solve this problem.  (Currently it’s private to (gnu packages sdl) but
> you can export it.)

Oh, great, I didn't know about it, thanks!  It will reduce the workaround.

However, just using this package cannot solve the problem: 'sdl-config'
points directly to "/gnu/store/…-sdl-1.2.15", so you need to modify a
package anyway to make it find "/gnu/store/…-sdl-union-1.2.15/include/SDL"
(that's why 'guile-sdl' specifies "--with-sdl-prefix=").

-- 
Alex