[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: sed problem in latest CVS autoconf
From: |
Nicolas Joly |
Subject: |
Re: sed problem in latest CVS autoconf |
Date: |
Fri, 13 Oct 2000 14:28:18 +0200 |
User-agent: |
Mutt/1.2.4i |
On Fri, Oct 13, 2000 at 12:36:55PM +0200, Akim Demaille wrote:
> Hi Nicolas,
>
> Could you try out this patch? If it works, I'll for an additional
> approval before committing.
>
> Index: ChangeLog
> from Akim Demaille <address@hidden>
> * tests/mktests.sh: If ever it fails, touch the output files
> ``a` la missing''.
>
> 2000-10-13 Akim Demaille <address@hidden>
>
> Index: tests/mktests.sh
> ===================================================================
> RCS file: /cvs/autoconf/tests/mktests.sh,v
> retrieving revision 1.4
> diff -u -r1.4 mktests.sh
> --- tests/mktests.sh 2000/10/12 15:02:42 1.4
> +++ tests/mktests.sh 2000/10/13 10:26:16
> @@ -18,6 +18,22 @@
> # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
> # 02111-1307, USA.
>
> +# If we fail, clean up, but touch the output files. We probably failed
> +# because we used some non portable tool, but we just don't care: this
> +# shell script is a maintainer tool, and we do expect good tools.
> +trap "rm -f acdefuns audefuns requires syntax.tm4 update.tm4
> + touch syntax.m4
> + touch update.m4
> + exit 1" \
> + 0 1 2 15
Do you really want to abort in this case ? It stops the testsuite.
Except this little point, it works.
> +
> +# If ever something goes wrong, fail, so that the trap be launched.
> +set -e
> +
> +# We need arguments.
> +test $# != 0
> +
> +# We need these arguments.
> src="$@"
>
>
> @@ -169,4 +185,7 @@
> mv update.tm4 update.m4
>
> rm -f acdefuns audefuns requires
> +
> +trap 0
> +
> exit 0
--
Nicolas Joly
Informatique Scientifique
Institut Pasteur, Paris.