[Insight-developers] KWStyle as an ITK submodule?

[Hans Johnson]

I would support David¹s suggestion.  This has the added benefit of easily
being able to use the system version if it exists, or build the local
version if necessary.

We use this paradigm often in the BRAINS package.

Hans



On 10/28/10 4:20 PM, "David Cole" <david.cole at kitware.com> wrote:

> We could use CMake's ExternalProject module to build KWStyle in a subdir of
> the ITK build directory itself, or in a "cached" location (like your HOME
> directory) if there is no KWStyle found at CMake configure time...
> 
> 
> On Thu, Oct 28, 2010 at 5:08 PM, Bill Lorensen <bill.lorensen at gmail.com>
> wrote:
>> Please do not populate the source tree with executables.
>> 
>> On Thu, Oct 28, 2010 at 5:05 PM, Matthew McCormick (thewtex)
>> <matt at mmmccormick.com> wrote:
>>> > Hi Gaëtan,
>>> >
>>>> >>
>>>> >> I think it would be nice to have KWStyle as a submodule of ITK so that
it
>>>> >> can be retrieved by SetupForDevelopment.sh and built at the same time
>>>> than
>>>> >> ITK. This would remove the step 3 in the list above.
>>>> >>
>>>> >> There would also be a few nice side effects:
>>>> >> - it would be easier to run the style check in ITK
>>>> >> - it would be easier to require a specific version of KWStyle
>>>> >>
>>>> >> What do you think?
>>>> >>
>>> >
>>> > Sounds great.
>>> >
>>> > A possible hangup -- the build directory could be anywhere.
>>> > Specifying the location of the KWStyle executable then becomes
>>> > difficult for the development setup scripts.  Maybe take the
>>> > unconventional approach to building the executable in the source tree
>>> > and adding it to .gitignore?
>>> >
>>> > Matt
>>> > _______________________________________________
>>> > Powered by www.kitware.com <http://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
>>> >
>> _______________________________________________
>> Powered by www.kitware.com <http://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
>> 
>> 
>> 
>> _______________________________________________
>> 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20101028/58e36851/attachment.htm>