commit -- Re: Makefile Modification Proposed

Jeff Hostetler (jeff@abisource.com)
Thu, 29 Apr 1999 13:09:30 -0500


fixed. OS_ARCH is now defined in the config/platform/*.mk
rather than in config/abi_defs.mk. you should now be able
to hack as you please on the beos version of the string....

jeff

At 03:00 AM 4/19/99 -0400, Thomas Fletcher wrote:
>All,
>
> I just noticed tonight, working on the BeOS version
>that the abi_defs.mk looks something like this for
>detecting the OS_ARCH:
>
>OS_ARCH := $(shell uname -m | sed -e s/i.86/i386/
> -e s/sun4u/sparc64/
> -e s/arm.*/arm/
> -e s/sa110/arm/
> | sed "s/\//-/")
>
>This looks to me like kind of a nasty kludge. Shouldn't
>this code be more generic like:
>
>OS_ARCH := $(shell uname -m | sed "s/\//-/")
>
>and then in each platform specific case (based on OS_NAME)
>something like this be done:
>
>OS_ARCH = $(shell echo "$(OS_ARCH)" | sed -e s/i.86/i386/
> -e s/sun4u/sparc64/
> -e s/arm.*/arm/
> -e s/sa110/arm/)
>
>I'm just wondering because that is what I'm planning to
>do with the BeOS .. whose uname -m is kinda mucked up.
>It reports BePC for x86 based machines (s/BePC/x86/) and
>ppc for Macs and BeBox for BeBoxen running with ppc's
>(s/BeBox/ppc). I didn't really want to stick more
>cases in the above clause, I think it is messy as is.
>
>Thoughts ...
>
>Thomas
>



This archive was generated by hypermail 1.03b2.