Guillem Jover wrote:
> On Tue, 2008-02-12 at 15:17:39 +0100, Denis Oliver Kropp wrote:
>> Do you have more information about this?
> 
> Sure.

Thanks, it's almost clear now :)

>> Guillem Jover wrote:
> Oh, yes -lm should go as well, can fix the patch and resend.

I'll take care of it after applying yours.

>> And what in the static case?
> 
> The static case is handled by directfb-config, and that one includes

What if only libpng.a and libz.a exist and it's only checking for -lpng to be 
working?

> all the indirect dependencies, would be better to switch it to use
> pkg-config, though.

Yeah, I found libpng.pc recently and was quite surprised :)

>>> diff --git a/directfb-config.in b/directfb-config.in
>>> index 568831b..5bd4b91 100644
>>> --- a/directfb-config.in
>>> +++ b/directfb-config.in
>>> @@ -189,7 +189,7 @@ while test $# -gt 0; do
>>>              echo_videoprovider="$echo_videoprovider -lmpeg3"
>>>              ;;
>>>            swf)
>>> -            echo_videoprovider="$echo_videoprovider -ljpeg -lz"
>>> +            echo_videoprovider="$echo_videoprovider -ljpeg"
>> same as with libpng
> 
> This one is only relevant for the static linking case, but I removed it
> because I don't see on my system that libjpeg.so.* is using libz.so.*.
> I can check the source code in case it could be possible that it could
> use it, in that case it will be safer to just re-add it. Unfortunately
> libjpeg does not have a pkg-config file.

I did not find any zlib dependencies in libjpeg, so this change is fine, too.

-- 
Best regards,
   Denis Oliver Kropp

.------------------------------------------.
| DirectFB - Hardware accelerated graphics |
| http://www.directfb.org/                 |
"------------------------------------------"

_______________________________________________
directfb-dev mailing list
directfb-dev@directfb.org
http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

Reply via email to