Hello, On Wed, Apr 16, 2014 at 12:13:21PM -0700, Junio C Hamano wrote: > Interesting. What other strange forms can they record in their > repositories, I have to wonder. Can they do > 2014-01-07T5:8:6.048176Z > for example?
Roman Belinsky, the author of this fix, witnessed after large scale conversion that the problem happens with the hour part only. (SVN commits from the same origin did this with hours but not with minutes.) Recorded here: https://github.com/msysgit/git/pull/126#discussion_r9661916 > I am wondering if it is simpler and less error prone > to turn all these "we only accept two digits" into "\d+" not only > for the hour part but also minute and second parts. But Roman's proposed regexp nicely shows 1) what the standard is and 2) what is the deviation. Have a nice day, Stepan Kasal > > Signed-off-by: Stepan Kasal <ka...@ucw.cz> > > --- > > perl/Git/SVN.pm | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/perl/Git/SVN.pm b/perl/Git/SVN.pm > > index a59564f..09cff13 100644 > > --- a/perl/Git/SVN.pm > > +++ b/perl/Git/SVN.pm > > @@ -1321,7 +1321,7 @@ sub get_untracked { > > sub parse_svn_date { > > my $date = shift || return '+0000 1970-01-01 00:00:00'; > > my ($Y,$m,$d,$H,$M,$S) = ($date =~ /^(\d{4})\-(\d\d)\-(\d\d)T > > - (\d\d)\:(\d\d)\:(\d\d)\.\d*Z$/x) or > > + (\d\d?)\:(\d\d)\:(\d\d)\.\d*Z$/x) or > > croak "Unable to parse date: $date\n"; > > my $parsed_date; # Set next. > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
