On Fri, Apr 6, 2012 at 1:35 PM, Peter Kümmel <[email protected]> wrote: > On 06.04.2012 19:24, David Cole wrote: >> >> >> The ninja-generator stuff has all been merged into 'master' already. >> Just start a new topic branch and push it to the stage: >> >> # first get up to date w.r.t. origin/master: >> git checkout master >> git pull >> >> # Then start a new branch from there: >> git checkout -b ninja-patch >> # make your changes, and commits >> # edit, edit, edit >> git add -u >> git commit >> git push stage HEAD >> ssh [email protected] stage cmake merge -b next ninja-patch >> > > Thanks! Patch is now in next. > > > Peter > -- > > Powered by www.kitware.com > > Visit other Kitware open-source projects at > http://www.kitware.com/opensource/opensource.html > > Please keep messages on-topic and check the CMake FAQ at: > http://www.cmake.org/Wiki/CMake_FAQ > > Follow this link to subscribe/unsubscribe: > http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
One nitpick with the patch, the KWStyle test fails because lines 58 & 63 exceed our max line length limit (79 characters is the max). Could you please put another commit on top of that one, fixing the lines that are too long, and then also push that to stage and merge it to next again? Thanks, David Details: $ ctest -R KWStyle -VV | grep -C 5 -i error Guessing configuration NoConfig 190: 190: Processing /Users/davidcole/Dashboards/My Tests/CMake/Source/cmNewLineStyle.h 190: 190: 190: Processing /Users/davidcole/Dashboards/My Tests/CMake/Source/cmNinjaNormalTargetGenerator.cxx 190: Error #0 (58) Line length exceed 80 (max=79) 190: Error #0 (63) Line length exceed 87 (max=79) 190: 190: 190: Processing /Users/davidcole/Dashboards/My Tests/CMake/Source/cmNinjaNormalTargetGenerator.h 190: 190: Errors while running CTest -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
