[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: svn commit: r1683387 - /subversion/trunk/subversion/mod_dav_svn/status.c

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 3 Jun 2015 18:47:20 +0200

> -----Original Message-----
> From: ivan_at_apache.org [mailto:ivan_at_apache.org]
> Sent: woensdag 3 juni 2015 18:31
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1683387 -
> /subversion/trunk/subversion/mod_dav_svn/status.c
>
> Author: ivan
> Date: Wed Jun 3 16:31:08 2015
> New Revision: 1683387
>
> URL: http://svn.apache.org/r1683387
> Log:
> Fix abort() in svn-status handler on platforms that doesn't support C99
> format specifiers for strftime(): %F and %z are new in C99 and C89
> compiler/runtime doesn't support them [1]:
> [[[
> Those listed here are supported by the latest C and C++ standards (both
> published in 2011), but those in yellow were introduced in C99 (only
> required for C++ implementations since C++11), and may not be supported by
> libraries that comply with older standards.
> ]]]
>
> [1] http://www.cplusplus.com/reference/ctime/strftime/
>
> * subversion/mod_dav_svn/status.c
> (DEFAULT_TIME_FORMAT): Use '%Y-%m-%d' instead of '%F' and '%Z instead
> of
> '%z'.

+1 on the %F part. Good catch!

%z and %Z are handled in ap_ht_time(), so I don't think a change is necessary there. Either should work on all platforms.

        Bert
Received on 2015-06-03 19:03:12 CEST

This is an archived mail posted to the Subversion Dev mailing list.