|
From: | Derek Robert Price |
Subject: | Re: 02AS_HELP_STRING.diff |
Date: | Fri, 28 Mar 2003 09:47:48 -0500 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20030208 Netscape/7.02 |
Akim Demaille wrote:
Derek> address@hidden wrote: >> Index: ChangeLog >> 2003-03-21 Derek Price <address@hidden>>> >> * lib/autoconf/general.m4 (AC_ARG_VAR): Use AS_HELP_STRING instead of>> obsolete AC_HELP_STRING. >> (AC_HELP_STRING): AU_DEFUN to... >> * lib/m4sugar/m4sh.m4 (AS_HELP_STRING): ...here. >> * tests/m4sh.at (AS_HELP_STRING): New test.>>Derek> Committed. Why this patch? AC_HELP_STRING is a *bad* thing, I did not mean to propagate it further. Are you using this in an AS script? Such a patch also needs that documentation, and NEWS.
NEWS and doc I can deal with. As for how I am using it, I want to use it in an Autotest script. In another patch or three I will add an AT_ macro to accept scripter-defined options. Since the functionality would be shared between AT and AC, I thought AS would be the best place. Is there a better place? Printing help strings and accepting options isn't exactly an uncommon idiom in scripts.
The deeper reason behind this is that I started converting the CVS test suite (on the CVS project itself, as opposed to the Autoconf test suite checked into CVS) to use autotest. The CVS tests suite is currently a molithic, 25k+ line Bourne shell script. You can imagine what a bear that is to maintain. In converting it, I am attempting to maintain backwards compatibility with the options the original test suite accepts and in a few cases I just can't imagine a better way of accomplishing what the options accomplish.
Derek -- *8^) Email: address@hidden Get CVS support at <http://ximbiot.com>! -- Were it left for me to decide whether we should have a government without newspapers, or newspapers without a government, I should not hesitate a moment to prefer the latter. - Thomas Jefferson(appeared http://hotwired.lycos.com/special/lawsuit/ )
[Prev in Thread] | Current Thread | [Next in Thread] |