m4-patches
[Top][All Lists]
Advanced

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

Re: FYI: 39-gary-fix-gmp-test.patch reverted


From: Gary V. Vaughan
Subject: Re: FYI: 39-gary-fix-gmp-test.patch reverted
Date: Fri, 03 Oct 2003 11:58:25 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030925 Thunderbird/0.3

Akim Demaille wrote:
 >>> This patch is wrong, since the goal of ./testsuite is to be able to be
 >>> run easily by hand.  The functionality of TESTS_ENVIRONMENT you are
 >>> looking for is in atlocal.  If GMP does not work, dig in there.

 >> Ah yes.  At some point atlocal stopped being generated.  I am
 >> reverting this patch in favour of adding `tests/atlocal' to
 >> AC_CONFIG_FILES.  Is that the right way to do things?

 > Something is weird in here.  This should be done by
 > AC_CONFIG_TESTDIR([tests]).  That's what the doc says, but the code no
 > longer does this.  Autoconf and Bison include their own
 > AC_CONFIG_FILES for atlocal.  I don't know why, but I'm in favor of
 > restoring the documented behavior.  But I guess that in the meanwhile
 > M4 ought to have its AC_CONFIG_TESTDIR([tests/atlocal]), indeed.

I was wrong, and reading an old documentation.  I found the following
in the ChangeLog.  Sorry.

 Revision 1.10 / (download) - annotate - [select for diffs] , Wed Oct 31 
10:40:13 2001 UTC (23 months ago) by akim
CVS Tags: AUTOCONF-2_52f
Changes since 1.9: +0 -4 lines
Diff to previous 1.9 (colored)

* lib/autoconf/autotest.m4 (AC_CONFIG_TESTDIR): Do not
AC_CONFIG_FILES(atlocal) as it means a file atlocal.in *must* be
provided, while it is optional.
* configure.ac: Adjust.

Okay, thanks for the heads up.  I'll leave m4 as is then.

Cheers,
        Gary.
--
  ())_.  Gary V. Vaughan    gary@(lilith.warpmail.net|gnu.org)
  ( '/   Research Scientist http://www.oranda.demon.co.uk       ,_())____
  / )=   GNU Hacker         http://www.gnu.org/software/libtool  \'      `&
`(_~)_   Tech' Author       http://sources.redhat.com/autobook   =`---d__/





reply via email to

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