On Friday, October 08, 2010 03:52:47 Joakim Tjernlund wrote:
> Joakim Tjernlund wrote on 2010/10/08 08:56:20:
> > Tried it with:
> > TARGET="powerpc-4.4.4_softfloat-linux-gnu"
> > export EXTRA_ECONF="--without-long-double-128 --with-dwarf2 --without-fp
> > -- with-fp=soft --disable-64-bit-bfd"
> > 
> >     crossdev \
> >    
> >    -s3 \
> >    --genv EXTRA_ECONF="--with-cpu=e300c2 --without-long-double-128
> >    --with-dwarf2 --without-fp --with-fp=soft --disable-64-bit-bfd" \
> >    --ex-gdb \
> >    --libc 2.11.2 \
> >    --binutils 2.20.1-r1 \
> >    --gcc 4.4.4-r2 \
> >    --kernel 2.6.30-r1 \
> >    --target ${TARGET}
> 
> Got better when I changed
>   --genv EXTRA_ECONF=" .... "
> to
>  --genv EXTRA_ECONF='" .... "'
> 
> This is should be documented or changed to not require ''

it isnt feasible to auto quote the contents.  you will need to account for 
them yourself.  consider the fact that this supports multiple variables, so 
your quoting example here is limited.

> hmm, this looks wrong, suggest:
> 
> -set_portage ${KCAT} ${KPKG} ${KVER} "${KVER}"
> -set_portage ${LCAT} ${LPKG} ${LVER} "${LVER}"
> +set_portage ${KCAT} ${KPKG} ${KVER} "${KENV}"
> +set_portage ${LCAT} ${LPKG} ${LVER} "${LENV}"

thanks.  committed & pushed.
-mike