Re: CVS commit: src/usr.sbin/puffs/rump_smbfs

[Nick Hudson <nick.hudson%gmx.co.uk@localhost>]

On Monday 14 September 2009 10:20:52 Antti Kantee wrote:
> On Mon Sep 14 2009 at 08:15:36 +0100, Nick Hudson wrote:
[...]
> > > The binutils regression is that between 2.16 and 2.19 __start_sectname
> > > for orphaned sections was changed to use PROVIDE instead of define and
> > > hence those symbols are not available anymore in library components
> > > produced with ld.  I have provided a quick workaround which allows the
> > > build to work with both with 2.16 and 2.19 while not silently breaking
> > > functionality like in your model.
> > >
> > > Please revert all of your incorrect commits.
> >
> > Feel free to fix rump and linksets correctly. It's been explained to you
> > why they can't work properly together.
>
> rump hasn't relied on linksets since August 2008 (and even then there was
> only a few day window of linkset use after introducing dynamic linking
> to rump).  The only thing that matters is the entry point, which,
> incidentally, is what binutils 2.19 chaged.

I believe you've analysed this completely incorrectly. afaict, librumnet has 
always had unresolved references to {__start,__stop)_link_set_domains. The 
change from binutils 2.16 to 2.19 is, I believe, that 2.19 is less forgiving 
to this error.

Nick