From: Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>
Date: Tue, 07 Dec 2010 14:27:21 +0100

On 07.12.2010 13:18, Teruaki Kawashima wrote:
> (2010/12/07 7:28), mailer_at_svn.rockbox.org wrote:
> > Date: 2010-12-06 23:28:14 +0100 (Mon, 06 Dec 2010)
> > New Revision: 28753
> >
> > Log Message:
> > Fix configure and lib path
> >
> > Modified:
> > trunk/firmware/export/rbpaths.h
> > trunk/tools/configure
> >
> > Modified: trunk/tools/configure
> > ===================================================================
> > --- trunk/tools/configure 2010-12-06 22:26:31 UTC (rev 28752)
> > +++ trunk/tools/configure 2010-12-06 22:28:14 UTC (rev 28753)
> > _at__at_ -3144,14 +3145,7 _at__at_
> > defendian="ROCKBOX_LITTLE_ENDIAN"
> > fi
> >
> > -if [ "$ARG_RBDIR" ]; then
> > - if [ "$need_full_path" != "yes" ]; then
> > - if [ -z `echo $ARG_RBDIR | grep '^/'` ]; then
> > - rbdir="/"$ARG_RBDIR
> > - else
> > - rbdir=$ARG_RBDIR
> > - fi
> > - fi
> > +if [ "$ARG_RBDIR" != "" ]; then
> > echo "Using alternate rockbox dir: ${rbdir}"
> > fi
> >
>
> Hi, kugel.
>
> Could you explain why you removed these lines?
> It only seems to break --rbdir option for native builds.
> I use this option, and it is inconvenient to me that it doesn't work.

Sorry, somehow I thought rbdir was set somewhere else.

> Also, "WARNING: PREFIX not supported on Android. You can however use
> --rbdir" near line 118 in configure is false..


That's not entirely true. It doesn't have a huge effect, but it's used
for paths in *.cfg and make zip. You could specify /rockbox and install
themes onto the sd card a bit more easily.
But it doesn't affect from where rockbox is run or so, right.


>
> > _at__at_ -3401,8 +3398,9 _at__at_
> > export ENCODER=_at_ENCODER_at_
> > export USE_ELF=_at_USE_ELF_at_
> > export RBDIR=_at_RBDIR_at_
> > -export ROCKBOX_BINARY_PATH=_at_binpath_at_
> > +export ROCKBOX_SHARE_PATH=_at_sharepath_at_
> > export ROCKBOX_LIBRARY_PATH=_at_libpath_at_
> > +export ROCKBOX_LIBRARY_PATH=_at_libpath_at_
> > export SDLCONFIG=_at_SDLCONFIG_at_
> >
> > CONFIGURE_OPTIONS=_at_CMDLINE_at_
> >
> btw, this chagnge looks strange to me.
> it adds second ROCKBOX_LIBRARY_PATH and removesROCKBOX_BINARY_PATH.
>
> best regards,
> teru

Thanks again for spotting.

Best regards.
Received on 2010-12-07