On Thursday 12 of November 2009 23:49:22 David Byron wrote:
> > On Thursday 12 of November 2009 21:51:30 Kamil Dudka wrote:
> >
> > I only want to keep working what used to work before. With
> > the change I am proposing it works before/after the
> > patch. Attached are both (incremental and merged) patches.
>
> I get the same behavior with your patch and mine. I still can't see a case
> where you'd get different behavior but I'm fine with your patch. It does
> seem like it violates a bit of abstraction. It assumes that pkgconfig
> names its file openssl.pc. With your patch, configure doesn't run
> CURL_CHECK_PKGCONFIG(openssl) if openssl.pc doesn't exist which I suppose
> saves a bit of time. With my patch, when openssl.pc doesn't exist that
> macro does run and sets PKGCONFIG to no, so neither SSL_CPPFLAGS nor
> SSL_LDFLAGS changes.
>
> Anyway, I'd love to see either patch committed.

Daniel, ok to apply the patch?

http://permalink.gmane.org/gmane.comp.web.curl.library/25817

Kamil
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2009-11-13