All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 3/3] gnutls: link against 'dl' library
Date: Thu, 02 Jun 2011 10:38:33 +0100	[thread overview]
Message-ID: <1307007513.27470.473.camel@rex> (raw)
In-Reply-To: <20110602063130.GG3281@jama.jama.net>

On Thu, 2011-06-02 at 08:31 +0200, Martin Jansa wrote:
> what do you have in configlog for LIBDL?
> 
> here it's all empty
> LIBDL=''
> LIBDL_PREFIX=''
> LTLIBDL=''
> 
> if I add 
> +             --with-libdl-prefix=${STAGING_DIR_HOST}${prefix} \
> +             --with-libpthread-prefix=${STAGING_DIR_HOST}${prefix} \
> to EXTRA_OE_CONF (we already have couple of --with-*-prefix there) it's
> found correctly (and this is at least better workarround then forcing
> -ldl directly to LDFLAGS):
> LIBDL='/OE/shr-core/tmp/sysroots/om-gta02/usr/lib/libdl.so'
> LIBDL_PREFIX='/OE/shr-core/tmp/sysroots/om-gta02/usr'
> LTLIBDL='-L/OE/shr-core/tmp/sysroots/om-gta02/usr/lib -ldl'
> 
> same problem is with libpthread..
> 
> maybe their m4/lib-link.m4 does something wrong in
> AC_DEFUN([AC_LIB_HAVE_LINKFLAGS],
> [
>   AC_REQUIRE([AC_LIB_PREPARE_PREFIX])
>   AC_REQUIRE([AC_LIB_RPATH])

I just looked at the code in lib/configure and concluded that its just
luck whether it figures out the correct values or does something really
nasty. Its poking about ${libdir} for this stuff so you end up with
something like the following combinations:

a) 64 bit system with 64 bit libs in /usr/lib64 and no 32 bit libs:
     Finds no -ldl

   Something else may or may not link it indirectly so the build may or 
   may not work.

b) 64 bit system with 64 bit libs in /usr/lib64 and 32 bit libs in /usr/lib, building for 32 bit:
     Finds a -ldl which "works".

   Things appear to work.

c) Other variations


The correct fix would appear to be to set the paths specifically using
the options Martin mentions above.

Cheers,

Richard




  parent reply	other threads:[~2011-06-02  9:42 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-01 20:09 [PATCH 0/3] Patches pending on O.S. Systems tree Otavio Salvador
2011-06-01 20:09 ` [PATCH 1/3] busybox: enable mdev by default Otavio Salvador
2011-06-01 20:37   ` Phil Blundell
2011-06-01 20:40     ` Otavio Salvador
2011-06-02 16:37       ` Richard Purdie
2011-06-02 16:40         ` Otavio Salvador
2011-06-03  1:06         ` Khem Raj
2011-06-03  6:37           ` Koen Kooi
2011-06-03  8:24             ` Richard Purdie
2011-06-03  8:59               ` Koen Kooi
2011-06-03  9:50                 ` Richard Purdie
2011-06-03 10:42                   ` Koen Kooi
2011-06-03  9:57           ` Phil Blundell
2011-06-01 20:09 ` [PATCH 2/3] qmake_base.bbclass: fix lrelease/lupdate binary names Otavio Salvador
2011-06-02 16:51   ` Paul Eggleton
2011-06-02 17:27     ` Richard Purdie
2011-06-01 20:09 ` [PATCH 3/3] gnutls: link against 'dl' library Otavio Salvador
2011-06-01 20:33   ` Phil Blundell
2011-06-01 20:39     ` Otavio Salvador
2011-06-01 20:53       ` Phil Blundell
2011-06-01 21:00         ` Otavio Salvador
2011-06-01 21:43           ` Saul Wold
2011-06-01 21:52             ` Martin Jansa
2011-06-01 22:19               ` Saul Wold
2011-06-02  6:31                 ` Martin Jansa
2011-06-02  9:29                   ` Koen Kooi
2011-06-02  9:38                   ` Richard Purdie [this message]
2011-06-02 11:00                     ` [PATCH] gnutls: add --with-libdl-prefix and --with-libpthread-prefix Martin Jansa
2011-06-02 13:53                       ` Richard Purdie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1307007513.27470.473.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.