On 08/17/2016 06:11 AM, Dāvis Mosāns wrote: >> Applied with minor tweaks and merged to `next` for testing: > > Great! Thanks!
I squashed in one warning fix: Windows: Encode child process output to internally-used encoding https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=12924660 However, then I had to revert the change from `next` because it causes the CTest.UpdateGIT test to fail on Windows machines. I think the problem is that we run Git with a `-z` option to produce binary output. In such cases we should not do any encoding conversions. cmProcessTools and RunSingleCommand will need to gain options for this. Also I noticed that if DecodeText buffers partial characters we may need a finalize step later to finish them off. Otherwise invalid byte sequences may be dropped if they appear at the end. Please fetch the above version and revise it as needed. Thanks, -Brad -- Powered by www.kitware.com Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Kitware offers various services to support the CMake community. For more information on each offering, please visit: CMake Support: http://cmake.org/cmake/help/support.html CMake Consulting: http://cmake.org/cmake/help/consulting.html CMake Training Courses: http://cmake.org/cmake/help/training.html Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Follow this link to subscribe/unsubscribe: http://public.kitware.com/mailman/listinfo/cmake-developers
