Re: [RFT PATCH v3 3/9] add --with-sysroot

[Paolo Bonzini]

On 08/01/2010 12:48 PM, Ralf Wildenhues wrote:
> Next one: 3/9.   This is a bit of a preliminary review, because I
> haven't read all patches and all of the discussion yet.
>
> * Paolo Bonzini wrote on Thu, Jul 29, 2010 at 01:23:16AM CEST:
> > * libltdl/m4/libtool.m4 (_LT_HOST_NONCANONICAL, _LT_WITH_SYSROOT): New.
> > (LT_SETUP): Require _LT_WITH_SYSROOT.
> >
> > Signed-off-by: Paolo Bonzini<address@hidden>
> > ---
> >          Right now the default is to use a sysroot.  Probably it is
> >          a bit too aggressive, especially because most existing
> >          cross-compilation setups are using the wrong prefix that
> >          includes the sysroot.  They would thus stop working until
> >          they switched to --prefix and installing with DESTDIR.
>
> Can you explain this comment in a bit more detail?  What do you mean
> with "use the wrong prefix"?  Is that, they use
>    ./configure --prefix=/win-mount/usr
>    make
>    make install
>
> but should be using
>    ./configure --prefix=/usr
>    make
>    make install DESTDIR=/win-mount
>
> instead?

Yes.  However, the default is _not_ to use a sysroot, so there's no 
visible change without supplying the options.

> Thanks.  Ideally, the answer is put in the form of a diff against the
> manual.  ;-)

Of course. :)  In the meanwhile I updated the NEWS entry in 8/8.

Paolo