On Thu, Mar 17, 2011 at 4:01 AM, Jakub Narebski <jnareb@xxxxxxxxx> wrote: Jakub, Thanks for all of your constructive feedback. I have taken your suggestions into account and posted an updated series of patches. > This does not describe why would one want such way of displaying > timestamps, and which views would be affected. > > BTW. should it be timezone of web server (machine where gitweb is > run), or local time of author / committer / tagger as described in the > timezone part of git timestamp? The case I am currently trying to improve is the one in which all developers are at a single site. In the open source world it is common to have developers scattered all over the globe, so some of them will inevitably have to perform timezone conversions. But Git is becoming a popular tool in the private sector and it is common to have most/all contributors based in a single office. In the latter case, it is helpful to display the local timezone instead of GMT. This also helps make the data more readable by program managers and other non-developers who have an interest in tracking the project. > Why project specific override is not supported? ÂI think it might make > sense to enable this feature on project-by-project basis; some > projects might be dispersed geographically, some might not. Mostly ease of testing. I did not need it for any of my projects. It turned out to be a simple change, and it is in v2. The cases I tested were: default 0 default 1 override 1, project unset override 1, project 0 override 1, project 1 > Is it still an RFC 2822 conformant date? ÂIf it is not, then above > change is invalid, and we have to implement this feature in different > way. I believe it is still valid. Original date: Thu, 17 Mar 2011 02:11:05 +0000 New date: Wed, 16 Mar 2011 19:11:05 -0700 Sample date from RFC 2822 Appendix A: Fri, 21 Nov 1997 09:55:06 -0600 > Hmmm... I wonder if it wouldn't be better to print both times (perhaps > reversed) in this case... I have submitted a third patch which does this. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html