This is the mail archive of the mauve-patches@sourceware.org mailing list for the Mauve project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFC: update in batch_run


>>>>> "Olivier" == Olivier Jolly <olivier.jolly@pcedev.com> writes:

Olivier> So I added the ability to set the environnement variable
Olivier> COMPILER to "none", which assumes that all .class are already
Olivier> present.

Looks pretty good to me.  Just one nit.

Olivier>   If accepted, the batch_run wrapper in eclipse could be changed to
Olivier> benefit of this optimisation.

Sounds good.

Olivier> +if test -z "$NATIVE" -a "x$COMPILER" != "xnone"; then

It is a little more defensive to use &&:

    if test -z foo && test blah != blah; then

Maybe my old autoconf habits are showing... there were weird parsing
problems with -a in some versions of test, back in ancient times.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]