From: Teruaki Kawashima <kamekun_at_r6.dion.ne.jp>
Date: Tue, 07 Dec 2010 21:18:35 +0900

(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.
Also, "WARNING: PREFIX not supported on Android. You can however use
--rbdir" near line 118 in configure is false..

> _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
Received on 2010-12-07