You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by mark benedetto king <bk...@inquira.com> on 2002/09/03 13:32:00 UTC
[PATCH] Aesthetic fixes for configure --help
This patch makes the output of "configure --help" more nicely aligned,
and removes superfluous '\' characters from the Berkeley DB description.
* configure.in
(--enable-maintainer-mode): indented second line to match first.
* ac-helpers/svn-apache.m4
(--with-apache=DIR): indented first and second lines to match other
items in list.
* ac-helpers/berkeley-db.m4
(--with-berkeley-db=PATH): removed superfluous '\'s from description.
Index: configure.in
===================================================================
--- configure.in
+++ configure.in Tue Sep 3 09:20:54 2002
@@ -214,7 +214,7 @@
AC_ARG_ENABLE(maintainer-mode,
[ --enable-maintainer-mode Turn on debugging and very strict compile-time
- warnings],
+ warnings],
[
if test "$enableval" = "yes" ; then
if test "$enable_debugging" = "no" ; then
Index: ac-helpers/svn-apache.m4
===================================================================
--- ac-helpers/svn-apache.m4
+++ ac-helpers/svn-apache.m4 Tue Sep 3 09:25:09 2002
@@ -12,8 +12,8 @@
AC_MSG_CHECKING(for static Apache module support)
AC_ARG_WITH(apache,
-[ --with-apache=DIR Build static Apache module. DIR is the path
- to the top-level Apache source directory.],
+[ --with-apache=DIR Build static Apache module. DIR is the path
+ to the top-level Apache source directory.],
[
if test "$withval" = "yes"; then
AC_MSG_ERROR(You need to specify a directory with --with-apache)
Index: ac-helpers/berkeley-db.m4
===================================================================
--- ac-helpers/berkeley-db.m4
+++ ac-helpers/berkeley-db.m4 Tue Sep 3 09:23:39 2002
@@ -50,17 +50,17 @@
AC_ARG_WITH(berkeley-db,
[ --with-berkeley-db=PATH
- Find the Berkeley DB header and library in \`PATH/include' and
- \`PATH/lib'. If PATH is of the form \`HEADER:LIB', then search
+ Find the Berkeley DB header and library in `PATH/include' and
+ `PATH/lib'. If PATH is of the form `HEADER:LIB', then search
for header files in HEADER, and the library in LIB. If you omit
- the \`=PATH' part completely, the configure script will search
+ the `=PATH' part completely, the configure script will search
for Berkeley DB in a number of standard places.
The Subversion server requires Berkeley DB $db_version or newer. If
- you specify \`--without-berkeley-db', the server will not be
+ you specify `--without-berkeley-db', the server will not be
built. Otherwise, the configure script builds the server if and
only if it can find a new enough version installed, or if a copy
- of Berkeley DB exists in the subversion tree as subdir \`db'.],
+ of Berkeley DB exists in the subversion tree as subdir `db'.],
[
if test "$withval" = "yes"; then
status=required
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] Aesthetic fixes for configure --help
Posted by Greg Hudson <gh...@MIT.EDU>.
Not an objection to the patch, but a note related to some stuff already
in the file:
On Tue, 2002-09-03 at 09:32, mark benedetto king wrote:
> - you specify \`--without-berkeley-db', the server will not be
> + you specify `--without-berkeley-db', the server will not be
We should stop using the grave accent character as a left quote, instead
writing '--without-bekerkeley-db'. See:
http://www.cl.cam.ac.uk/~mgk25/ucs/quotes.html
Increasingly, the fonts people use make `foo' look bad, because ' is
displayed as a neutral (vertical) quote and ` as a grave accent.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] Aesthetic fixes for configure --help
Posted by mark benedetto king <bk...@inquira.com>.
On Tue, Sep 03, 2002 at 03:41:08PM +0200, Daniel Stenberg wrote:
> On Tue, 3 Sep 2002, mark benedetto king wrote:
>
> > AC_ARG_ENABLE(maintainer-mode,
> > [ --enable-maintainer-mode Turn on debugging and very strict compile-time
> > - warnings],
> > + warnings],
> > [
>
> Just a question here, isn't it smarter and easier to use the AC_HELP_STRING()
> to get good-looking help strings?
>
Oooh! I didn't know about AC_HELP_STRING()!.
Yes, I'll bet that it is. Retrofitting all of configure.in and ac-helpers
to use it might be painful, though. A quick check makes it seem like
only neon.m4 uses it.
--ben
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH] Aesthetic fixes for configure --help
Posted by Daniel Stenberg <da...@haxx.se>.
On Tue, 3 Sep 2002, mark benedetto king wrote:
> AC_ARG_ENABLE(maintainer-mode,
> [ --enable-maintainer-mode Turn on debugging and very strict compile-time
> - warnings],
> + warnings],
> [
Just a question here, isn't it smarter and easier to use the AC_HELP_STRING()
to get good-looking help strings?
--
Daniel Stenberg - http://daniel.haxx.se - +46-705-44 31 77
ech`echo xiun|tr nu oc|sed 'sx\([sx]\)\([xoi]\)xo un\2\1 is xg'`ol
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org