Hi, the code looks right to me. Thanks a lot for fixing that so quickly.
Brad is also right about the first byte. The command has the argument already attached. Michael On 06/08/2016 06:46 PM, Brad King wrote: > On 06/08/2016 12:43 PM, Rolf Eike Beer wrote: >>> Run ranlib on archives only if the tool is available >>> https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ea598671 >> Shouldn't the first check also be "cmd == ":" instead of only >> checking the first byte? > No, because the command in question is actually ": libfoo.a". > No valid command can start in ":" on any platform. > > -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
