lib*.so in FILES_${PN}

lib*.so in FILES_${PN}

[Phil Blundell]

This recent checkin:

http://cgit.openembedded.net/cgit.cgi?url=openembedded/commit/&id=ba434bd1dc9dc8ed61953a1579ea9e993e81cbb2

added ${base_libdir}/lib*.so to the default FILES_${PN}, on the grounds
that this is necessary to get procps packaged correctly.

However, I think this cure is worse than the original disease, in that
it's now virtually impossible to prevent *.so development symlinks
(which obviously belong in ${PN}-dev) from leaking into ${PN}.  This
situation is particularly bad if ${base_libdir} and ${libdir} are the
same, since you then end up with a useless .so symlink being installed
in your rootfs for virtually every library on the system.

I propose to revert the patch in question, and instead just let
procps.bb adjust its own local setting of FILES as necessary.

Comments?

p.

Re: lib*.so in FILES_${PN}

[Leon Woestenberg]

Hello Phil,

On Sat, Jun 13, 2009 at 10:19 PM, Phil Blundell<philb@gnu.org> wrote:
> I propose to revert the patch in question, and instead just let
> procps.bb adjust its own local setting of FILES as necessary.
>
Agreed,
-- 
Leon

Re: lib*.so in FILES_${PN}

[Richard Purdie]

On Sat, 2009-06-13 at 21:19 +0100, Phil Blundell wrote:
> This recent checkin:
> 
> http://cgit.openembedded.net/cgit.cgi?url=openembedded/commit/&id=ba434bd1dc9dc8ed61953a1579ea9e993e81cbb2
> 
> added ${base_libdir}/lib*.so to the default FILES_${PN}, on the grounds
> that this is necessary to get procps packaged correctly.
> 
> However, I think this cure is worse than the original disease, in that
> it's now virtually impossible to prevent *.so development symlinks
> (which obviously belong in ${PN}-dev) from leaking into ${PN}.  This
> situation is particularly bad if ${base_libdir} and ${libdir} are the
> same, since you then end up with a useless .so symlink being installed
> in your rootfs for virtually every library on the system.
> 
> I propose to revert the patch in question, and instead just let
> procps.bb adjust its own local setting of FILES as necessary.
> 
> Comments?

Agreed, that shouldn't be a default.

Richard

Re: lib*.so in FILES_${PN}

[Phil Blundell]

On Mon, 2009-06-15 at 09:08 +0100, Richard Purdie wrote:
> On Sat, 2009-06-13 at 21:19 +0100, Phil Blundell wrote:
> > I propose to revert the patch in question, and instead just let
> > procps.bb adjust its own local setting of FILES as necessary.
>
> Agreed, that shouldn't be a default.

Thanks to you and Leon for the feedback.  I've reverted that patch now.

p.