Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
Sender: cygwin-developers-owner AT sourceware DOT cygnus DOT com
Delivered-To: mailing list cygwin-developers AT sourceware DOT cygnus DOT com
Message-ID: <377B690F.7ED57D68@vinschen.de>
Date: Thu, 01 Jul 1999 15:11:43 +0200
From: Corinna Vinschen <corinna AT vinschen DOT de>
X-Mailer: Mozilla 4.6 [en] (WinNT; I)
X-Accept-Language: de,en
MIME-Version: 1.0
To: Chris Faylor <cgf AT cygnus DOT com>
CC: cygdev <cygwin-developers AT sourceware DOT cygnus DOT com>
Subject: Re: strftime and timezone
References: <377AC37D DOT 63B4B4A3 AT vinschen DOT de>
Content-Type: multipart/mixed;
 boundary="------------E378A1DEC8E4F81D8655FC7D"

This is a multi-part message in MIME format.
--------------E378A1DEC8E4F81D8655FC7D
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Corinna Vinschen wrote:
> 
> This question is of interest, since timezone is handled correctly in winsup:
> 
> Shouldn't %Z now be implemented in strftime?
> 
> Corinna

The patch follows.

Regards,
Corinna

ChangeLog:
==========

Thu Jul 1 14:44:00  Corinna Vinschen  <corinna AT vinschen DOT de>

shellutils:
-----------
	* lib/strftime.c: Added define for __CYGWIN__
	* config.h.in: define HAVE_TZNAME by default.

newlib:
-------
	* libc/include/time.h: Added preprocessor control path for
	compile time behaviour.
	* libc/time/strftime.c (strftime): Calling tzset() as required
	by POSIX.1.
	Added support for '%e' (blank padded day of week) as suggested
	by POSIX.2 (and often used by date(1) command).
	Added support for '%Z' (timezone string).

winsup:
-------
	* times.cc (cygwin_tzset): examines TZ now.
--------------E378A1DEC8E4F81D8655FC7D
Content-Type: text/plain; charset=us-ascii;
 name="tz-patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="tz-patch"

--- shellutils/lib/strftime.c.old	Thu Jul 01 12:58:01 1999
+++ shellutils/lib/strftime.c	Thu Jul 01 12:40:29 1999
@@ -50,7 +50,11 @@
 # endif
 #endif
 #if HAVE_TZNAME
+#ifdef __CYGWIN__
+#define tzname _tzname
+#else
 extern char *tzname[];
+#endif
 #endif
 
 /* Do multibyte processing if multibytes are supported, unless
--- shellutils/config.h.in.old	Thu Jul 01 13:05:07 1999
+++ shellutils/config.h.in	Thu Jul 01 13:33:42 1999
@@ -53,7 +53,7 @@
 
 /* Define if you don't have tm_zone but do have the external array
    tzname.  */
-#undef HAVE_TZNAME
+#define HAVE_TZNAME 1
 
 /* Define if you have the vprintf function.  */
 #undef HAVE_VPRINTF
--- newlib/libc/include/time.h.old	Thu Jul 01 12:04:26 1999
+++ newlib/libc/include/time.h	Thu Jul 01 12:02:48 1999
@@ -73,6 +73,11 @@ struct tm *_EXFUN(gmtime_r,	(const time_
 struct tm *_EXFUN(localtime_r,	(const time_t *, struct tm *));
 
 #ifdef __CYGWIN32__
+#ifdef _COMPILING_NEWLIB
+extern time_t _timezone;
+extern int _daylight;
+extern char *_tzname[2];
+#else
 #ifndef __STRICT_ANSI__
 extern time_t _timezone __declspec(dllimport);
 extern int _daylight __declspec(dllimport);
@@ -80,6 +85,7 @@ extern char *_tzname[2] __declspec(dllim
 
 char *_EXFUN(timezone, (void));
 void _EXFUN(tzset, (void));
+#endif
 #endif
 #endif /* __CYGWIN32__ */
 
--- newlib/libc/time/strftime.c.old	Thu Jul 01 11:30:20 1999
+++ newlib/libc/time/strftime.c	Thu Jul 01 13:19:52 1999
@@ -168,6 +168,9 @@ _DEFUN (strftime, (s, maxsize, format, t
   size_t count = 0;
   int i;
 
+  /* POSIX.1 8.1.1 requires setting of tzname whenever strftime is called */
+  tzset ();
+
   for (;;)
     {
       while (*format && *format != '%')
@@ -256,6 +259,16 @@ _DEFUN (strftime, (s, maxsize, format, t
 	  else
 	    return 0;
 	  break;
+	case 'e':
+	  if (count < maxsize - 2)
+	    {
+	      sprintf (&s[count], "%2d",
+		       tim_p->tm_mday);
+	      count += 2;
+	    }
+	  else
+	    return 0;
+	  break;
 	case 'H':
 	  if (count < maxsize - 2)
 	    {
@@ -425,6 +438,11 @@ _DEFUN (strftime, (s, maxsize, format, t
 	    return 0;
 	  break;
 	case 'Z':
+          if (count < maxsize - strlen (_tzname[tim_p->tm_isdst]))
+            {
+              strcpy (&s[count], _tzname[tim_p->tm_isdst]);
+              count += strlen (_tzname[tim_p->tm_isdst]);
+            }
 	  break;
 	case '%':
 	  if (count < maxsize - 1)
--- winsup/times.cc	1999/06/28 09:00:09	1.1.1.1
+++ winsup/times.cc	1999/07/01 12:52:16
@@ -13,8 +13,10 @@ details. */
 #include <sys/timeb.h>
 #include <utime.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <errno.h>
 #include "winsup.h"
+#include <ctype.h>
 
 #define FACTOR (0x19db1ded53ea710LL)
 #define NSPERSEC 10000000LL
@@ -584,10 +586,6 @@ cygwin_tzset ()
   static NO_COPY char buf1[33], buf2[33];
 #endif
 
-  /* FIXME: This function should examine the TZ environment variable.
-     Right now it just always sets information based on the system
-     time zone.  */
-
   DWORD tzid;
   TIME_ZONE_INFORMATION tz;
 
@@ -598,12 +596,36 @@ cygwin_tzset ()
   else
     _timezone = (tz.Bias + tz.StandardBias) * SECSPERMIN;
   _daylight = (tzid == TIME_ZONE_ID_DAYLIGHT);
-  WideCharToMultiByte (CP_ACP, 0, tz.StandardName, -1, buf1, sizeof buf1 - 1,
-		       NULL, NULL);
-  buf1[sizeof buf1 - 1] = '\0';
-  _tzname[0] = buf1;
-  WideCharToMultiByte (CP_ACP, 0, tz.DaylightName, -1, buf2, sizeof buf2 - 1,
-		       NULL, NULL);
-  buf2[sizeof buf2 - 1] = '\0';
-  _tzname[1] = buf2;
+
+  char *tzv = getenv ("TZ");
+
+  if (tzv)
+    {
+      if (*tzv == ':')
+        ++tzv;
+      strcpy (buf1, tzv);
+      for (tzv = buf1; *tzv && isalpha (*tzv); ++tzv)
+        ;
+      if (*tzv)
+        {
+          for (*tzv++ = '\0'; *tzv && ! isalpha (*tzv); ++tzv)
+            ;
+          strcpy (buf2, tzv);
+        }
+      else
+        buf2[0] = '\0';
+      _tzname[0] = buf1;
+      _tzname[1] = buf2;
+    }
+  else
+    {
+      WideCharToMultiByte (CP_ACP, 0, tz.StandardName, -1, buf1,
+                           sizeof buf1 - 1, NULL, NULL);
+      buf1[sizeof buf1 - 1] = '\0';
+      _tzname[0] = buf1;
+      WideCharToMultiByte (CP_ACP, 0, tz.DaylightName, -1, buf2,
+                           sizeof buf2 - 1, NULL, NULL);
+      buf2[sizeof buf2 - 1] = '\0';
+      _tzname[1] = buf2;
+    }
 }


--------------E378A1DEC8E4F81D8655FC7D--