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]