[cairo] [PATCH] OS/2 backend support files

[Peter Weilbacher]

Behdad Esfahbod wrote:
> On Mon, 2006-07-31 at 18:37 -0400, Peter Weilbacher (Mozilla) wrote:
>> On Mon, 31 Jul 2006 16:11:38 -0400, Behdad Esfahbod wrote:
>> 
>> >This ain't gonna work.  You should at least make sure you know where
>> >this is being used and update that too!  The png_CFLAGS/etc are used by
>> >the CAIRO_BACKEND_ENABLE macro and they should match the first parameter
>> >passed to that macros.  So, if you want to change png_CFLAGS to
>> >PNG_CFLAGS, you need to update other places too.  And currently that
>> >macro doesn't support that, as the first argument has got to be in lower
>> >case.  So, to fix your problem, you need to add yet another argument to
>> >that macro and update all the rest of configure.in to use that.
>> 
>> Huh, seems like my testing procedure was broken, everything seemed to work. 
>> And I didn't look at the code of the macro, just at its documentation 
>> above.[1]
> 
> Yep, I already updated the docs of the macro to note the variables it
> uses.

Thanks.

>> Sounds like you are not in favor of making such a big change, and actually I 
>> am too busy with too many other things to do that. So let's just forget about 
>> it.
> 
> It's very error prune.  So, do you have any other solutions already?

Well, the "solution" for me was that I managed to install pkg-config on
OS/2 and tinker with the respective .pc files until I got configure to
run through.

But actually, there has to be another way to make the _{CFLAGS,LIBS}
variables consistently uppercased, because ./configure --help lists lots
of other variables that are all uppercase, too (POPPLER_* and FREETYPE_*
for example). I think I understand too little of autoconf to determine
why it works for them but not for png_* and xlib_xrender_*.

   Peter.