classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] RFC: Build fixes for new target native layers


From: Tom Tromey
Subject: Re: [cp-patches] RFC: Build fixes for new target native layers
Date: 16 Jan 2006 12:26:20 -0700
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

>>>>> "Roman" == Roman Kennke <address@hidden> writes:

Roman> I need some help though. We need some configury to determine
Roman> the target system for which we build and select the right
Roman> options in native/target/Makefile.am. I just don*t know how to
Roman> do that. Anybody there who can help me with this?

Roman> +# FIXME: How do I set these variables in configure?
Roman> +#if TARGET_LINUX
Roman> +#  TARGET = Linux
Roman> +#endif

Instead of doing things this way, with a bunch of independent 'if's, I
would suggest computing TARGET in configure.ac and then AC_SUBSTing
it.

IMSNHO, it is bogus to have a separate 'SunOS' target.  This sort of
thing is better handled by feature tests.  I didn't look at the uses
of this, though... on occasion there's no alternative to a platform
test.

The way to set TARGET is by looking at what AC_CANONICAL_TARGET
defined, eg:

    case "$target_os" in)
      linux*)
        TARGET=Posix
        ;;
    ...
    esac
    AC_SUBST(TARGET)

Tom




reply via email to

[Prev in Thread] Current Thread [Next in Thread]