Re: [Patch] Exit gdb/configure.tgt for unsupported targets

Ralf Corsepius
2010-02-24T17:00:22Z

On 02/24/2010 05:51 PM, Tom Tromey wrote:
>>>>>> "Ralf" == Ralf Corsepius<ralf.corsepius@rtems.org> writes:
>>>>>>
> Ralf> 2010-02-23 Ralf CorsÃ©pius<ralf.corsepius@rtems.org>
> Ralf> * configure.tgt: Exit if ${gdb_target_obs}" is not set.
>
> Ralf> +if test -z "${gdb_target_obs}"; then
> Ralf> + echo "*** Configuration ${targ} is unsupported.">&2
> Ralf> + exit 1
> Ralf> +fi
>
> I think it would be better to do the test in configure.ac and use
> AC_MSG_ERROR.
>
I wasn't aware "${gdb_target_obs}" is exported to configure.ac and
intended to keep configure.tgt self-contained ;)

Now that it apparently is exported and used in configure.ac, I agree
with you, configure.ac+AC_MSG_ERROR would be better.

Ralf