Re: Linker (?) problems

[Paul M. <le...@li...>]

On Mon, Apr 09, 2007 at 06:31:12PM -0400, Mike Frysinger wrote:
> On Monday 09 April 2007, Stuart MENEFY wrote:
> > The problem is that you're not getting shell arithmetic expansion. I'm
> > guessing that the shell invoked by your make (usually /bin/sh) is not
> > bash, or is one of the versions which doesn't support $[ ... ] for
> > arithmetic expansion (version 2.0 ?).
> 
> $[...] is a bashism ... that should be converted to POSIX $((...))

Indeed. I'm not sure why we were using this in the first place anyways,
something we never ran in to before I suppose.

On Mon, Apr 09, 2007 at 10:13:48PM +0100, Stuart MENEFY wrote:
> --- linux.orig/arch/sh/boot/Makefile
> +++ linux/arch/sh/boot/Makefile
> @@ -23,8 +23,8 @@ CONFIG_MEMORY_START     ?= 0
>  CONFIG_ZERO_PAGE_OFFSET ?= 0
> 
>  MKIMAGE := $(srctree)/scripts/mkuboot.sh
> -ULOADADDR := $(shell printf "0x%8x" $$[0x80000000 + $(CONFIG_MEMORY_START) + $(CONFIG_ZERO_PAGE_OFFSET)])
> -UENTRYADDR := $(shell printf "0x%8x" $$[$(ULOADADDR) + 0x1000])
> +ULOADADDR := $(shell printf "0x%8x" $$((0x80000000 + $(CONFIG_MEMORY_START) + $(CONFIG_ZERO_PAGE_OFFSET))))
> +UENTRYADDR := $(shell printf "0x%8x" $$(($(ULOADADDR) + 0x1000)))
> 
>  quiet_cmd_uimage = UIMAGE  $@
>        cmd_uimage = $(CONFIG_SHELL) $(MKIMAGE) -A sh -O linux -T kernel \
> 

Thanks, applied.