I agree.

Either adapt the parser to be locale aware (see localeconv(3))
or document that tools must be run in the C locale.
But don't try to play Pokémon with separators.  It'll blow up
horribly.

Please revert.

Yours,
Robert Clausecker

Am Sun, Jul 19, 2026 at 11:14:10AM +0200 schrieb Dag-Erling Smørgrav:
> Gleb Smirnoff <[email protected]> writes:
> > commit 77a201b1705dbd97ea9ebe5b25b1d4ddac8a7d38
> > Author:     Gleb Smirnoff <[email protected]>
> > AuthorDate: 2026-07-17 00:49:55 +0000
> > Commit:     Gleb Smirnoff <[email protected]>
> > CommitDate: 2026-07-17 01:40:03 +0000
> >
> >     ministat: parse comma used as decimal delimiter
> >     
> >     This allows to use output of '/usr/bin/time -ao foo' as direct input to
> >     ministat(1).
> >     
> >     While here make diagnostic message more verbose.
> 
> This is wrong.  What if delim is ","?  The correct solution is to either
> run your time command with LANG=C, or post-process the output.
> 
> DES
> -- 
> Dag-Erling Smørgrav - [email protected]
> 
> 

-- 
()  ascii ribbon campaign - for an encoding-agnostic world
/\  - against html email  - against proprietary attachments

Reply via email to