On 03/17/2016 11:23 AM, Charles Huet wrote: > I added a utility function 'ConvertToNinjaFolderRule' that > does the same thing as ConvertToNinjaPath, but makes relative > to the source dir, and adds the '/all' suffix.
Great. I've applied the patch with a few style changes and logic fixes. I also added documentation and a test: Ninja: Add `$subdir/all` targets https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ca575fe9 Ninja: Add test for `$subdir/all` targets https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=86c851e6 Help: Add notes for topic 'ninja-directory-targets' https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=133aed8f Please try out that version to make sure it still works for you. 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
