Hello,

On Wed, Jun 29, 2005 at 10:29:11PM +0200, Alexandre Duret-Lutz wrote:
> I'm installing the following on HEAD and branch-1-9.

thank you.

> (I think any justification of why unset must not be used really
> belongs to the Autoconf manual.)

Yes, of course.  I sent a question to the autoconf list; I hope we'll
receive an answer from a historician there.

Two tiny comments:

>  Václav Haisman               [EMAIL PROTECTED]
>  Vadim Zeitlin                [EMAIL PROTECTED]
> +Vincent Lefevre              Vincent Lefevre <[EMAIL PROTECTED]>

It seems this is the first time when you repeat the full name in the email.

> +if test "${TIME_STYLE+set}" = set; then
> +  TIME_STYLE=posix-long-iso
> +  export TIME_STYLE
> +fi

I see no reason for the `export' here.  If it's not exported, it's even
better.  We don't set the variable if it's not set, either.

Please find attached a patch which contains both these changes.

Happy summer,
        Stepan
2005-06-30  Stepan Kasal  <[EMAIL PROTECTED]>

        * lib/mdate-sh: Don't export TIME_STYLE; it's even better if it's
        not exported.

--- lib/mdate-sh        29 Jun 2005 20:29:24 -0000      1.18
+++ lib/mdate-sh        30 Jun 2005 12:01:58 -0000
@@ -64,7 +64,6 @@
 # variable to its documented default.
 if test "${TIME_STYLE+set}" = set; then
   TIME_STYLE=posix-long-iso
-  export TIME_STYLE
 fi
 
 save_arg1=$1
--- THANKS      2005-06-30 13:18:18.000000000 +0200
+++ THANKS      2005-06-30 13:15:45.000000000 +0200
@@ -269,7 +269,7 @@
 Ulrich Eckhardt                [EMAIL PROTECTED]
 VĂĄclav Haisman                [EMAIL PROTECTED]
 Vadim Zeitlin          [EMAIL PROTECTED]
-Vincent Lefevre                Vincent Lefevre <[EMAIL PROTECTED]>
+Vincent Lefevre                [EMAIL PROTECTED]
 Volker Boerchers       [EMAIL PROTECTED]
 Werner John            [EMAIL PROTECTED]
 Werner Koch            [EMAIL PROTECTED]

Reply via email to