Re: [PATCH] add hotplug2 package

[Denys Dmytriyenko <denis@denix.org>]

On Tue, Mar 31, 2009 at 09:01:17AM -0600, Angus Ainslie wrote:
> > > +LDFLAGS += "-lglibc"
> > 
> > I don't think that actually works, and if it did, uclibc would break
> 
> Without that I get undefined refeences
> 
>  loader.c:(.text+0x14): undefined reference to `dlclose'
>  workers/loader.o: In function `worker_load':
>  loader.c:(.text+0x44): undefined reference to `dlopen'
>  loader.c:(.text+0x5c): undefined reference to `dlsym'
>  loader.c:(.text+0x84): undefined reference to `dlerror'
> 
> > > +do_configure () {
> > > +}
> > 
> > missing colon between the braces. But more importantly: why inherit 
> > autotools when you're not going to run configure?
> 
> I was under the mistaken impression that autotools need to be included
> to get do_compile() and do_install(). It seems those are the defaults.
> 
> There is a deeper problem that without the inherit autotools then the
> LDFLAGS above fails to find glibc. Is there something missing in the
> default case ?

Wow, I'm surprised "-lglibc" above even worked for you:

$ find rootfs -name 'libglibc*'
$

Those undefined references above are provided by libdl.so, which needs:

LDFLAGS += " -ldl"

And no need for autotools.

-- 
Denys