|
From: Eyal L. <ey...@ey...> - 2003-03-18 11:16:23
|
Julian Seward wrote: > > Eyal > > It looks like a good patch, and a useful thing too. I would merge > it in, except there is one critical problem which needs fixing first. As promised, here is a clean patch for the new --time-stamp option. I rewrote the logic so it is much simpler. Two general purpose functions were added VG_(time) as the standard time(), but uses type 'vg_time_t' VG_(localtime) as the standard localtime(), but uses type 'vg_tm'. Also, it always sets tm_isdst to zero and timezone is set through clo. I hope it complies with the current programming model and style (please let me know where it fails to do so). BTW, where is the CVS tree (is it public?). -- Eyal Lebedinsky (ey...@ey...) <http://samba.org/eyal/> |