Re: New library addition

[Alistair Crooks <agc%pkgsrc.org@localhost>]

On Mon, May 09, 2011 at 04:46:48PM +0200, Nicolas Joly wrote:
> On Mon, May 09, 2011 at 04:39:19PM +0200, Alistair Crooks wrote:
> > On Mon, May 09, 2011 at 09:06:20AM +0200, Nicolas Joly wrote:
> > > On Sun, May 08, 2011 at 08:25:05PM +0200, Alistair Crooks wrote:
> > > > On Sun, May 08, 2011 at 01:27:08PM +0200, Emmanuel Dreyfus wrote:
> > > > > I am about to add libperfuse and perfused to the base system (hey have
> > > > > been in tree for a while but were not built). Adding a new library to
> > > > > the sets lists is not straightforward, it would be nice if someone 
> > > > > could
> > > > > review that change.
> > > > 
> > > > Gensetlist (make setlist -Dperfuse) shows me:
> > > > 
> > > > [src/distrib/sets/lists/comp/shl.mi]
> > > > ./usr/libdata/debug/usr/lib/libperfuse.so.0.0.debug     
> > > > comp-perfuse-debug debug
> > > > 
> > > > [src/distrib/sets/lists/comp/ad.mips64eb]
> > > > ./usr/libdata/debug/usr/lib/64/libperfuse.so.0.0     comp-compat-shlib  
> > > > compat,pic,debug
> > > > ./usr/libdata/debug/usr/lib/o32/libperfuse.so.0.0    comp-compat-shlib  
> > > > compat,pic,debug
> > > > 
> > > > [src/distrib/sets/lists/comp/ad.mips64el]
> > > > ./usr/libdata/debug/usr/lib/64/libperfuse.so.0.0     comp-compat-shlib  
> > > > compat,pic,debug
> > > > ./usr/libdata/debug/usr/lib/o32/libperfuse.so.0.0    comp-compat-shlib  
> > > > compat,pic,debug
> > > > 
> > > > [src/distrib/sets/lists/comp/md.amd64]
> > > > ./usr/libdata/debug/usr/lib/i386/libperfuse.so.0.0   comp-compat-shlib  
> > > > compat,pic,debug
> > > > 
> > > > [src/distrib/sets/lists/comp/md.sparc64]
> > > > ./usr/libdata/debug/usr/lib/sparc/libperfuse.so.0.0     
> > > > comp-compat-shlib  compat,pic,debug
> > > [...]
> > > > However, "make setlist" is a fairly dumb beast, and if I have anything
> > > > wrong, I'd be grateful if people would tell me so that I can make it
> > > > better.
> > > 
> > > Looks like all compat debug libraries are missing the .debug extension.
> > 
> > So the entry is right but is just missing the ".debug" part on the end
> > of the filename part of the entry?
> 
> Exactly.

Should be fixed in the version I just checked in:

New output:

[src/distrib/sets/lists/comp/ad.mips64eb]
./usr/libdata/debug/usr/lib/64/libperfuse.so.0.0.debug      comp-compat-shlib   
compat,pic,debug
./usr/libdata/debug/usr/lib/o32/libperfuse.so.0.0.debug     comp-compat-shlib   
compat,pic,debug

[src/distrib/sets/lists/comp/ad.mips64el]
./usr/libdata/debug/usr/lib/64/libperfuse.so.0.0.debug      comp-compat-shlib   
compat,pic,debug
./usr/libdata/debug/usr/lib/o32/libperfuse.so.0.0.debug     comp-compat-shlib   
compat,pic,debug

[src/distrib/sets/lists/comp/md.amd64]
./usr/libdata/debug/usr/lib/i386/libperfuse.so.0.0.debug      comp-compat-shlib 
compat,pic,debug

[src/distrib/sets/lists/comp/md.sparc64]
./usr/libdata/debug/usr/lib/sparc/libperfuse.so.0.0.debug     comp-compat-shlib 
compat,pic,debug

Many thanks!

Best,
Alistair