On 6. 6. 25 22:16, Timofei Zhakov wrote:
Hi,

I would like to suggest adding a line to svn --version command to display the current locale name.

This could be helpful for us receiving or investigating any encoding-related bugs, and seems generally useful to a part of platform information.

So, I decided to draft a patch (attached to the email) to implement this feature. Providing examples of the machine info section of the new svn --version --verbose command below:

* running on x86_64-microsoft-windows6.2.9200
  - Windows 10 Pro, build 26100 [6.3 Client Multiprocessor Free]
  - locale encoding: CP1252


I'd call it "character encoding", it's more precise. "Locale encoding" means how the locale is encoded, which isn't what's shown here. Similarly, the struct members and accessors and public functions should be renamed.


Index: subversion/libsvn_subr/opt_subcommand.c =================================================================== --- subversion/libsvn_subr/opt_subcommand.c (revision 1926036) +++ subversion/libsvn_subr/opt_subcommand.c (working copy) @@ -29,6 +29,7 @@ #include <assert.h> #include <apr_general.h> +#include <apr_portable.h> #include "svn_hash.h" #include "svn_cmdline.h" @@ -491,6 +492,9 @@ svn_version_ext_runtime_osname(info))); } + SVN_ERR(svn_cmdline_printf(pool, _(" - locale encoding: %s\n"), + svn_version_ext_locale_encoding(info))); + libs = svn_version_ext_linked_libs(info); if (libs && libs->nelts) {

You don't need <apr_portable.h> here.


-- Brane

Reply via email to