pkg/45741: tcsh doesn't understand 64bit time_t

[mlelstv%serpens.de@localhost]

>Number:         45741
>Category:       pkg
>Synopsis:       tcsh doesn't understand 64bit time_t
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Dec 25 10:10:00 +0000 2011
>Originator:     Michael van Elst
>Release:        NetBSD 5.99.58
>Organization:
-- 
                                Michael van Elst
Internet: mlelstv%serpens.de@localhost
                                "A potential Snark may lurk in every tree."
>Environment:
        
        
System: NetBSD dummy 5.99.58 NetBSD 5.99.58 (GENERIC) #3: Thu Dec 15 15:23:35 
CET 2011 
mlelstv@henery:/home/netbsd-current/obj.amiga/home/netbsd-current/src/sys/arch/amiga/compile/GENERIC
 amiga
Architecture: m68k
Machine: amiga
>Description:
tcsh 6.17 prints bad values for user and system time. This is caused
by the change to 64bit time_t values in NetBSD.

E.g.

% time /usr/tests/crypto/libcrypto/h_ecdsatest
0.425u 0.001s 7:10.95 99.2%   0+0k 0+0io 0pf+0w

Total CPU time is 427 seconds. The seconds seem to printed
as milliseconds.

>How-To-Repeat:
Use the builtint time command of tcsh.

>Fix:
--- sh.time.c.orig      2009-03-23 19:08:52.000000000 +0100
+++ sh.time.c   2009-03-23 19:09:22.000000000 +0100
@@ -687,7 +687,7 @@
     timeval_t td;
 
     tvsub(&td, t1, t0);
-    xprintf("%ld.%03ld", td.tv_sec, td.tv_usec / 1000L);
+    xprintf("%ld.%03ld", (long)td.tv_sec, (long)td.tv_usec / 1000L);
 }
 
 static void

>Unformatted: