diff mbox

[v4] timezone: add a configure flag to disable program install

Message ID 20151017043929.GM28215@vapier.lan
State Committed
Delegated to: Mike Frysinger
Headers show

Commit Message

Mike Frysinger Oct. 17, 2015, 4:39 a.m. UTC
On 20 Sep 2015 21:31, ricaljasan wrote:
> On 09/18/2015 02:49 PM, Mike Frysinger wrote:
> > diff --git a/INSTALL b/INSTALL

INSTALL is a generated file, so spacing is out of our control.
consult the source texi file for any possible suggestions.

> > diff --git a/configure b/configure

same goes for this

here's a patch for the things i thought were def wrong
-mike

Comments

Rical Jasan Oct. 18, 2015, 1:12 a.m. UTC | #1
On 10/16/2015 09:39 PM, Mike Frysinger wrote:
> On 20 Sep 2015 21:31, ricaljasan wrote:
>> On 09/18/2015 02:49 PM, Mike Frysinger wrote:
>>> diff --git a/INSTALL b/INSTALL
> 
> INSTALL is a generated file, so spacing is out of our control.
> consult the source texi file for any possible suggestions.
> 
>>> diff --git a/configure b/configure
> 
> same goes for this

Cool, thank you for pointing that out.  I should've realized that about
configure, but didn't know that about INSTALL.

> here's a patch for the things i thought were def wrong
> -mike
> 
> --- a/configure.ac
> +++ b/configure.ac
> @@ -161,7 +161,7 @@ AC_ARG_ENABLE([profile],
>  	      [profile=no])
>  AC_ARG_ENABLE([timezone-tools],
>  	      AC_HELP_STRING([--disable-timezone-tools],
> -			     [do not install timezone tools (if using external ones) @<:@default=install@:>@]),
> +			     [do not install timezone tools @<:@default=install@:>@]),
>  	      [enable_timezone_tools=$enableval],
>  	      [enable_timezone_tools=yes])
>  AC_SUBST(enable_timezone_tools)
> --- a/manual/install.texi
> +++ b/manual/install.texi
> @@ -159,9 +159,9 @@ so that they can be invoked directly.
>  
>  @item --disable-timezone-tools
>  By default, timezone related utilities (@command{zic}, @command{zdump},
> -and @command{tzselect}) are installed with glibc.  If you are building
> +and @command{tzselect}) are installed with @theglibc{}.  If you are building
>  these independently (e.g. by using the @samp{tzcode} package), then this
> -option will allow disabling the install of these too.
> +option will allow disabling the install of these.
>  
>  Note that you need to make sure the external tools are kept in sync with
>  the versions that @theglibc{} expects as the data formats may change over

SGTM.  I know a number of my suggestions were a matter of taste, but I
think you grabbed the important ones.

Rical
diff mbox

Patch

--- a/configure.ac
+++ b/configure.ac
@@ -161,7 +161,7 @@  AC_ARG_ENABLE([profile],
 	      [profile=no])
 AC_ARG_ENABLE([timezone-tools],
 	      AC_HELP_STRING([--disable-timezone-tools],
-			     [do not install timezone tools (if using external ones) @<:@default=install@:>@]),
+			     [do not install timezone tools @<:@default=install@:>@]),
 	      [enable_timezone_tools=$enableval],
 	      [enable_timezone_tools=yes])
 AC_SUBST(enable_timezone_tools)
--- a/manual/install.texi
+++ b/manual/install.texi
@@ -159,9 +159,9 @@  so that they can be invoked directly.
 
 @item --disable-timezone-tools
 By default, timezone related utilities (@command{zic}, @command{zdump},
-and @command{tzselect}) are installed with glibc.  If you are building
+and @command{tzselect}) are installed with @theglibc{}.  If you are building
 these independently (e.g. by using the @samp{tzcode} package), then this
-option will allow disabling the install of these too.
+option will allow disabling the install of these.
 
 Note that you need to make sure the external tools are kept in sync with
 the versions that @theglibc{} expects as the data formats may change over