[Buildroot] [PATCH] package/cURL: fix static link whith openSSL

[Yann E. MORIN]

Thomas, All,

On Thursday 23 August 2012 11:00:34 Thomas Petazzoni wrote:
> Le Wed, 22 Aug 2012 22:37:03 +0200,
> "Yann E. MORIN" <yann.morin.1998 at free.fr> a écrit :
> 
> > When openSSL is selected, cURL is configured to use it.
> > 
> > But in this case, the libcurl.pc file /forgets/ to require link
> > against -ldl.
> > 
> > This can happen, for example, when BR2_PREFER_STATIC_LIB is not set,
> > but an executable wants to be linked statically (for various reasons
> > which are irrelevant here).
> > 
> > Fix that by conditionally append -ldl to the Libs.Private section of
> > the installed libcurl.pc if openSSL is enabled.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> 
> Shouldn't this rather be fixed through a patch that gets submitted to
> the upstream curl project?

Probably, yes. There's even a comment in the libcurl.pc.in template, that
reads:
  # This should most probably benefit from getting a "Requires:" field added
  # dynamically by configure.

I've tried to find a simple way to do that, but I did not find so far. It is
too much involved IMHO.

I'll try to reproduce a small test-case, and submit a bug report upstream.
Of course, when I find a bit of time, I'll try to fix that and push a proper
fix... :-/

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'