From: Junio C Hamano Date: Tue, 15 Jan 2013 19:14:42 GMT Subject: Re: [PATCH] am: invoke perl's strftime in C locale Message-ID: <7v1udm45st.fsf@alter.siamese.dyndns.org> In-Reply-To: <20130115174015.GA7471@altlinux.org> "Dmitry V. Levin" writes: > On Tue, Jan 15, 2013 at 08:50:59AM -0800, Jeff King wrote: >> On Tue, Jan 15, 2013 at 05:42:12PM +0100, Antoine Pelisse wrote: >> >> > > This puts all of perl into the C locale, which would mean error messages >> > > from perl would be in English rather than the user's language. It >> > > probably isn't a big deal, because that snippet of perl is short and not >> > > likely to produce problems, but I wonder how hard it would be to set the >> > > locale just for the strftime call. >> > >> > Maybe just setting LC_TIME to C would do ... >> >> Yeah, that is a nice simple solution. Dmitry, does just setting LC_TIME >> fix the problem for you? > > Just setting LC_TIME environment variable instead of LC_ALL would end up > with unreliable solution because LC_ALL has the highest priority. > > If keeping error messages from perl has the utmost importance, it could be > achieved by > - perl -M'POSIX qw(strftime)' -ne 'BEGIN { $subject = 0 } > + perl -M'POSIX qw(strftime :locale_h)' -ne ' > + BEGIN { setlocale(LC_TIME, "C"); $subject = 0 } > but the little perl helper script we are talking about hardly worths so > much efforts. Yeah I agree that this is not worth it, I would think.