[PATCH] MAKEALL: make it generic
Mike Frysinger
vapier.adi at gmail.com
Fri Sep 24 14:13:10 EDT 2010
On Fri, Sep 24, 2010 at 13:44, Jean-Christophe PLAGNIOL-VILLARD wrote:
> +if [ ! "${LOGDIR}" ] ; then
> + LOGDIR="LOG"
> +fi
simpler syntax:
: ${LOGDIR:=LOG}
> +if [ ! "${BUILDDIR}" ] ; then
> + BUILDIR="makeall_buildir"
> fi
: ${BUILDDIR:=makeall_builddir}
i think your patch misses a "d" here at least ...
> +CROSS_COMPILE_arm=/opt/arm-2007q3/bin/arm-none-linux-gnueabi-
> +CROSS_COMPILE_m68k=m68k-uclinux-
> +CROSS_COMPILE_blackfin=/opt/uClinux/bfin-linux-uclibc/bin/bfin-linux-uclibc-
> +CROSS_COMPILE_ppc=/opt/powerpc/bin/powerpc-unknown-linux-gnu-
> +CROSS_COMPILE_sandbox=/usr/bin/
> +CROSS_COMPILE_x86=/usr/bin/
i dont think hardcoding paths is a good idea, and bfin-linux-uclibc-
might not work. so bfin-uclinux- might be better.
-mike
More information about the barebox
mailing list