Adding a new config option, mind if I check this in?

Rob Landley rob at landley.net
Tue Feb 24 18:53:03 UTC 2009


On Tuesday 24 February 2009 11:52:10 Denys Vlasenko wrote:
> On Tuesday 24 February 2009 06:50:23 pm Denys Vlasenko wrote:
> > On Tuesday 24 February 2009 08:21:19 am Rob Landley wrote:
> > > On Tuesday 24 February 2009 01:02:24 Peter Kjellerstedt wrote:
> > > > > Anyway, I'm trying to get the number of patches I carry in FWL
> > > > > down to a dull roar.  This is a patch I've had in FWL forever,
> > > > > cleaned up and ready to submit.  Sound reasonable?  (There's a
> > > > > help entry at the end explaining why.)
> > > > >
> > > > > Rob
> > > >
> > > > Given that we have a very similar patch in our local version
> > > > of uClibc, I would like to see this patch applied as well.
> > >
> > > Done. :)
> >
> >           You must disable this to use uClibc with a relocatable
> > toolchain, such as the prebuilt binary cross compilers at
> >           http://uclibc.org/downloads/binaries which may be installed at
> > an arbitrary location (such as in a user's home directory).
> >
> >           The amount of time saved by this optimization is actually too
> > small to measure.
> >
> >
> > Since switching it on does more harm than good, why is it on by default?

So as not to change existing behavior, while letting those of us bitten by it 
switch it off without having to patch the source anymore?

I submitted a patch to remove it entirely back in September, and again in 
October, but it was ignored:

  http://lists.uclibc.org/pipermail/uclibc/2008-September/041030.html
  http://lists.uclibc.org/pipermail/uclibc/2008-October/041227.html

I vaguely recall having posted it before that.  The patch has been in my 
repository since the end of 2006:

  http://impactlinux.com/hg/hgwebdir.cgi/firmware/rev/57f77c25b39d

> Also it is intended to be a bool, but it is declared as string:
>
> config HARDWIRED_ABSPATH
>         string "Hardwire absolute paths into linker scripts"
>         ^^^^^^
>         default y
>         help
>
> Please fix.

Ah, _that_ was because I was operating on 4 hours of sleep again.  My bad.  
(My local patch completely removes it, I only made a config option out of it 
yesterday evening.)

Fixed.

> --
> vda

Rob


More information about the uClibc mailing list