[cmake-developers] Development pause for sweeping style changes

David Cole david.cole at kitware.com
Fri Aug 10 10:59:59 EDT 2012


On Fri, Aug 10, 2012 at 10:57 AM, Brad King <brad.king at kitware.com> wrote:
> On 08/10/2012 10:52 AM, Stephen Kelly wrote:
>> Stephen Kelly wrote:
>>
>>> Brad King wrote:
>>>
>>>> Hi Folks,
>>>>
>>>> With 2.8.9 out we'd like to perform two sweeping style change commits:
>>>>
>>>> (1) Remove all remnants of trailing whitespace
>>>> (2) Convert all CMake language code to use lower-case commands
>>>>
>>
>> Do you also have a script to clear the contents of endforeach(), endif(),
>> endmacro(), endfunction()? Or do you not want to do that?
>
> That's a bit more risky because sometimes they span multiple lines.
> I'd rather not tackle it now.
>
> -Brad
> --
>
> 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


But after the sweeping changes, I would certainly welcome clean up
commits that clear the end*() arguments. "Clean up" topics are always
welcome. We won't merge them in during the middle of "release
candidates" -- but we'll do it immediately after a release for sure.

Thx,
D



More information about the cmake-developers mailing list