[phobos] [D-Programming-Language/phobos] ac0407: std.datetime fixed so that it doesn't need the day...

Brad Roberts braddr at puremagic.com
Sun Feb 20 02:44:06 PST 2011


On 2/20/2011 2:27 AM, Jonathan M Davis wrote:
> 
> There are _multiple_ UTC time zone files for whatever reason. On Linux, all of 
> these files are identical:
> 
> /usr/share/zoneinfo/Etc/UTC
> /usr/share/zoneinfo/UTC
> /usr/share/zoneinfo/posix/Etc/UTC
> /usr/share/zoneinfo/posix/UTC
> 
> I don't know why there are so many. The time zone files could arguably be cleaned 
> up, because there are a number of them which are duplicates of one another (not 
> just UTC). The FreeBSD folks probably decided that there was no point in having 
> so many and didn't include some of them - kind of like how Mac OS X doesn have 
> have any of the time zone files which would be in /usr/share/zoneinfo/right. I 
> guess that this is just one more thing that goes to show that we'd benefit from 
> having an autotester on all of the platforms that we release for.
> 
> By the way, we may want to make sure that we leave a message on pull requests 
> when we merge them in, since it doesn't look like there's a way to determine who 
> merged in a pull request if they didn't have to make any additional changes to 
> it before they could push it into the main branch.
> 
> In this particular case, I assume that Brad did it, since he also made a change 
> to fix a test for FreeBSD in std.datetime, but I don't know that for sure. And if 
> for some reason I needed to contact whoever merged in a particular pull request, 
> I don't see any way to know who they are if they didn't comment on the pull 
> request or make any additional changes when doing the merge.
> 
> - Jonathan M Davis

On freebsd 8.1, the only UTC file is Etc/UTC, so rather little choice in the matter.

I did indeed do the merge, as can be seen from the news feed:

  https://github.com/organizations/D-Programming-Language

That's the only place I see it though, which is unfortunate.  It's also barely visible from the push email from github
to the phobos@ list.  I think it's worth filing an enhancement request with github to have who pushed it indicated in
the pull request.  It auto-closed it, so there's already some hook between the two.  I'll file it.

Later,
Brad


More information about the phobos mailing list