Tcl Source Code

View Ticket
Login
Ticket UUID: 1656002
Title: clock format not recognizing timezones
Type: Bug Version: obsolete: 8.5a6
Submitter: lvirden Created on: 2007-02-09 13:00:10
Subsystem: 06. Time Measurement Assigned To: kennykb
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2007-04-18 03:33:37
Resolution: Accepted Closed By: kennykb
    Closed on: 2007-04-17 20:33:37
Description:
sparc solaris 9, sun c compiler, configured with options \"'--prefix=/projects/sprs_lwv/tcl85' '--enable-shared' '--enable-symbols' '--enable-stubs' '--enable-64bit' '--enable-threads' 'CC=cc'\"

 % clock format [clock seconds] -format "%X %x" -locale fi_FI -timezone EET
 time zone EET not found

I find this even if I say
 % clock format [clock seconds] -timezone EST
 time zone EST not found
User Comments: kennykb added on 2007-04-18 03:33:37:
Logged In: YES 
user_id=99768
Originator: NO

The 'TIME ZONES' section of the manual page does say that the legacy time zones are recognized only when they come in on %z and %Z format groups, but I concede that the sentence before it is confusing.  I'm committing a proposed change - feel free to suggest a better one.

lvirden added on 2007-04-16 22:08:56:
Logged In: YES 
user_id=15949
Originator: YES

Would it make sense to add the details of what kind of "time zone" value is expected for the -timezone argument to the man page then?

kennykb added on 2007-04-16 02:30:58:
Logged In: YES 
user_id=99768
Originator: NO

A three- or four-letter time zone abbreviation is matched only if it is coming in from a %z or %Z format group or appears in an input from free-form [clock scan].  Otherwise, Tcl uses time zone names such as :America/New_York and EST5EDT, just as would be expected in the TZ environment variable.

Part of the intent behind this is to avoid creeping ambiguity.  Is BST British Summer Time or Brazilian Standard Time?  And 'EET' is also highly unlikely to be a correct time zone specification year-round, since it will not observe summer time.  :Europe/Helsinki, :Europe/Athens, or whatever is actually appropriate would be a better choice.

lvirden added on 2007-02-09 21:14:39:
Logged In: YES 
user_id=15949
Originator: YES

note that I just tried this:

% clock format [clock seconds] -timezone est

 time zone est not found

The 8.5 clock man page reads as if that should work.

HOWEVER, 

clock format [clock seconds] -timezone :US/Eastern

does work, so it isn't that timezones, in general, are broken - just the expectation from the docs that using a plain abbreviation would work...