Re: optional features and AC_ARG_WITH

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

 



Hello,

On Thu, Feb 17, 2005 at 11:28:08AM +0100, Gregorio Guidi wrote:
> > actually, I believe that both my proposals will get actually implemented.
> > But the AS_IF solution is already checked in to the autoconf CVS.
> 
> If there will be another solution, I would prefer to avoid AS_IF.

we cannot rely on any time frame for another solution.  So please use AS_IF
for now.  You can add this to the texinfo source, though:

@c FIXME: Remove AS_IF when the problem of AC_REQUIRE within `if' is solved.

> an example is meant to be a way to demonstate how easy it is to do something, 
> and using another macro can only scare away people that are already 
> intimidated by autoconf :)

I have a few comments.  Mostly aesthetics, but also fixing some bugs in your
examples (underquoted AS_HELP_STRING, for example):

+support for an external library (e.g. the readline library); if the user

Please use a comma for consistency with the rest of autoconf.texi:
	... (e.g., the ...   

Example 1:

AC_ARG_WITH(readline,
	[AS_HELP_STRING(--with-readline,
	   [support fancy command line editing <default=check>])],
	[],
	with_readline=check)
LIBREADLINE=
AS_IF([test "x$with_readline" != xno],
  [AC_CHECK_LIB(readline, main,
  	[AC_SUBST(LIBREADLINE, "-lreadline -lncurses")
	 AC_DEFINE(HAVE_LIBREADLINE, 1, [Define if you have libreadline])
	],
        [test "x$with_readline" = xcheck ||
	   AC_MSG_FAILURE([--with-readline was given, but test for readline failed])
	],
	-lncurses)])

Example2:

AC_ARG_WITH(readline,
	[AS_HELP_STRING(--with-readline,
	   [enable experimental support for readline <default=disabled>])],
	[],
	with_readline=no)
LIBREADLINE=
AS_IF([test "x$with_readline" != xno],
  [AC_CHECK_LIB(readline, main,
  	[AC_SUBST(LIBREADLINE, "-lreadline -lncurses")
	 AC_DEFINE(HAVE_LIBREADLINE, 1, [Define if you have libreadline])
	],
	[AC_MSG_FAILURE([--with-readline was given, but test for readline failed])],
	-lncurses)])

Example 3:

AC_ARG_WITH(readline,
	[AS_HELP_STRING(--without-readline,
	   [disable support for readline <default=enabled>])],
	[],
	with_readline=yes)
LIBREADLINE=
AS_IF([test "x$with_readline" != xno],
  [AC_CHECK_LIB(readline, main,
  	[AC_SUBST(LIBREADLINE, "-lreadline -lncurses")
	 AC_DEFINE(HAVE_LIBREADLINE, 1, [Define if you have libreadline])
	],
	[AC_MSG_FAILURE([test for readline failed, use --without-readline if you want to force readline support off])],
	-lncurses)])

And here:

+The examples above can be adapted with small effort to the case where
+@code{AC_ARG_ENABLE} should be preferred to @code{AC_ARG_WITH} (see
+@ref{Package Options}).

I'd say:
	These three examples can be easily adapted to the case...

Have a nice day,
	Stepan


_______________________________________________
Autoconf mailing list
Autoconf@xxxxxxx
http://lists.gnu.org/mailman/listinfo/autoconf

[Index of Archives]     [GCC Help]     [Kernel Discussion]     [RPM Discussion]     [Red Hat Development]     [Yosemite News]     [Linux USB]     [Samba]

  Powered by Linux