[Insight-developers] itk-module.cmake formatting

[David Cole]

On Fri, Jun 17, 2011 at 3:07 PM, Cory Quammen <cquammen at cs.unc.edu> wrote:

> Dear developers,
>
> Currently, within the itk-module.cmake files, all dependencies are
> listed on one line, e.g.
>
> itk_module(ITK-ImageGrid DEPENDS ITK-ImageFunction ITK-ImageFilterBase
> TEST_DEPENDS ITK-TestKernel ITK-RegistrationCommon DESCRIPTION
> "${DOCUMENTATION}")
>
>
>
> I propose the following format to make this easier to read.
>
>
> itk_module(ITK-ImageGrid
>  DEPENDS
>    ITK-ImageFunction
>    ITK-ImageFilterBase
>  TEST_DEPENDS
>    ITK-TestKernel
>    ITK-RegistrationCommon
>  DESCRIPTION
>    "${DOCUMENTATION}"
> )
>
>
> What do you think?
>
> Thanks,
> Cory
>
> --
> Cory Quammen
> Computer Integrated Systems for Microscopy and Manipulation (CISMM)
> Department of Computer Science
> University of North Carolina at Chapel Hill
> http://www.cs.unc.edu/~cquammen
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.html
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>


I think your proposal does make it easier to read. Easier to view diffs at
review time, too.

- David C.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110617/c9a7b681/attachment.htm>