mainAutoconf - Support: sr #108182, [doc] Clarification for behaviour...

 
 

sr #108182: [doc] Clarification for behaviour of AC_CHECK_FILES

Submitter:  Christophe CURIS <lobotomy>
Submitted:  Sat 17 Nov 2012 05:45:59 PM UTC
   
 
Priority:  * 5 - Unprioritized Severity:  2 - Minor
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 10 Jul 2020 03:52:44 PM UTC, comment #3: 

Closing; this patch will be included in the upcoming 2.70 release.

Zack Weinberg <zackw>
Group administrator
Sun 18 Nov 2012 01:49:46 PM UTC, comment #2: 

Hi Paul,

Thanks for the improvement, it looks a lot more clear now, and confirms it was worth specifying!

Best regards,
Christophe

Christophe CURIS <lobotomy>
Sat 17 Nov 2012 07:09:08 PM UTC, comment #1: 

Thanks for the bug report.  I committed this patch:

From 8d2596f8ea7f1ae8056efe9c033054aec2ccab54 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Sat, 17 Nov 2012 11:06:05 -0800
Subject: [PATCH] AC_CHECK_FILES: improve documentation

  • doc/autoconf.texi (Files): Document AC_CHECK_FILES better.

Reported by Christophe Curis in
<http://savannah.gnu.org/support/?108182>.
---
 doc/autoconf.texi | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index d1483b8..5b556da 100644
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -4457,7 +4457,7 @@ of the host machine, and therefore, they die when cross-compiling.
 @caindex file_@var{file}
 Check whether file @var{file} exists on the native system.  If it is
 found, execute @var{action-if-found}, otherwise do
-@var{action-if-not-found}, if given.  The result of this test is cached
+@var{action-if-not-found}, if given.  Cache the result of this test
 in the @code{ac_cv_file_@var{file}} variable, with characters not
 suitable for a variable name mapped to underscores.
 @end defmac
@@ -4466,11 +4466,12 @@ suitable for a variable name mapped to underscores.
   @ovar{action-if-not-found})
 @acindex{CHECK_FILES}
 @caindex file_@var{file}
-Executes @code{AC_CHECK_FILE} once for each file listed in @var{files}.
-Additionally, defines @samp{HAVE_@var{file}} (@pxref{Standard Symbols})
-for each file found.  The results of each test are cached in the
-@code{ac_cv_file_@var{file}} variable, with characters not suitable for
-a variable name mapped to underscores.
+For each file listed in @var{files}, execute @code{AC_CHECK_FILE}
+and perform either @var{action-if-found} or @var{action-if-not-found}.
+Like @code{AC_CHECK_FILE}, this defines @samp{HAVE_@var{file}}
+(@pxref{Standard Symbols}) for each file found and caches the results of
+each test in the @code{ac_cv_file_@var{file}} variable, with characters
+not suitable for a variable name mapped to underscores.
 @end defmac
 
 
--
1.7.11.7


Paul Eggert <eggert>
Group administrator
Sat 17 Nov 2012 05:45:59 PM UTC, original submission:  

Hello,

While browsing the on-line documentation for Autoconf, I found a point that could need some added information:

http://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Files.html

The macro AC_CHECK_FILES proposes two actions (if-found and if-not-found), but it is not clear:
 - are they executed once for each file?
 - or only once in the end?

In the later case (which is what I suppose the behaviour, - but please could you state explicitly it in the page?), would you mind also specifying that 'action-if-found' is executed only if all files are found? (provided it is the case, of course).

This may sound a bit nit-picking, but I fell it would be better to leave no point open to interpretation.

Christophe CURIS <lobotomy>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by zackw (Posted a comment)
  • -email is unavailable- added by eggert (Posted a comment)
  • -email is unavailable- added by lobotomy (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-07-10 zackw Open/ClosedOpen Closed
    2012-11-17 eggert StatusNone Done

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code