Re: [PATCH] Fix minor inconsistency in documentation of AC_{LINK, RUN}_IFELSE.

2010-05-25 Thread Eric Blake 
On 05/25/2010 09:15 AM, Stefano Lattarini wrote:
 A nitpicking consistency patch.
 
 * doc/autoconf.texi (Runtime) AC_RUN_IFELSE: Suggest to use
 `./conftest$ac_exeext' rather than `./conftest$EXEEXT' to run
 the compiled program.  This is done for consistency with the
 documentation of AC_LINK_IFELSE.

I'd actually rather go the other way - it is better to document external
variables like $EXEEXT than to expose internal details like $ac_exeext;
so it seems like AC_LINK_IFELSE should be the subject of the patch.

-- 
Eric Blake   ebl...@redhat.com+1-801-349-2682
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] Fix minor inconsistency in documentation of AC_{LINK, RUN}_IFELSE.

2010-05-25 Thread Stefano Lattarini 
At Tuesday 25 May 2010, Eric Blake ebl...@redhat.com wrote:
 On 05/25/2010 09:15 AM, Stefano Lattarini wrote:
  A nitpicking consistency patch.
 
  * doc/autoconf.texi (Runtime) AC_RUN_IFELSE: Suggest to use
  `./conftest$ac_exeext' rather than `./conftest$EXEEXT' to run
  the compiled program.  This is done for consistency with the
  documentation of AC_LINK_IFELSE.
 
 I'd actually rather go the other way - it is better to document
  external variables like $EXEEXT than to expose internal details
  like $ac_exeext;
Right.
  so it seems like AC_LINK_IFELSE should be the subject of the patch.
And AC_COMPILE_IFELSE too, since its documentation mentions 
$ac_object.

I think I can amend the patch this evening, if that's ok for you.

Thanks,
Stefano