diff options
author | Brad King <brad.king@kitware.com> | 2015-02-06 18:23:07 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2015-02-06 18:32:57 (GMT) |
commit | cd408d93fdf347ff63a8062f75f1f4ee3e898b17 (patch) | |
tree | 8fe5fcf05cd8092ccb1b80b0879c3928bec2af82 /Source/cmArchiveWrite.cxx | |
parent | 87be2e1427ba2b1b7697c9332487862917897dca (diff) | |
download | CMake-cd408d93fdf347ff63a8062f75f1f4ee3e898b17.zip CMake-cd408d93fdf347ff63a8062f75f1f4ee3e898b17.tar.gz CMake-cd408d93fdf347ff63a8062f75f1f4ee3e898b17.tar.bz2 |
Add setlocale() calls around use of libarchive APIs (#14934, #15377)
The libarchive APIs use nl_langinfo(CODESET) for iconv so they need the
locale to be set for LC_CTYPE. However, the rest of CMake does not
define any behavior for non-ASCII character classification/conversion so
we do not want to setlocale() globally. Add a RAII class to save, set,
and restore the locale around calls to libarchive APIs.
Inspired-by: Clinton Stimpson <clinton@elemtech.com>
Diffstat (limited to 'Source/cmArchiveWrite.cxx')
-rw-r--r-- | Source/cmArchiveWrite.cxx | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Source/cmArchiveWrite.cxx b/Source/cmArchiveWrite.cxx index a2aecac..9f28d42 100644 --- a/Source/cmArchiveWrite.cxx +++ b/Source/cmArchiveWrite.cxx @@ -12,6 +12,7 @@ #include "cmArchiveWrite.h" #include "cmSystemTools.h" +#include "cmLocale.h" #include <cmsys/ios/iostream> #include <cmsys/Directory.hxx> #include <cmsys/FStream.hxx> @@ -259,6 +260,9 @@ bool cmArchiveWrite::AddFile(const char* file, } const char* out = file + skip; + cmLocaleRAII localeRAII; + static_cast<void>(localeRAII); + // Meta-data. std::string dest = prefix? prefix : ""; dest += out; |