[Insight-developers] PhilipsRecImageIO

[Bill Lorensen]

Maybe it should be BUILD_DEFAULT_MODULES?

I agree that BUILD_ALL_MODULES is deceptive.

On Tue, Dec 18, 2012 at 5:13 PM, Sean McBride <sean at rogue-research.com> wrote:
> Bill,
>
> Indeed.  But what I meant is that the name "ITK_BUILD_ALL_MODULES" is rather deceptive (reminds me of -Wall).  Perhaps it should be named ITK_BUILD_MOST_MODULES or ITK_BUILD_COMMON_MODULES is what I was getting at.
>
> Sean
>
>
> On Tue, 18 Dec 2012 12:02:38 -0500, Bill Lorensen said:
>
>>Sean,
>>
>>It is not on by default.
>>
>>Bill
>>
>>On Tue, Dec 18, 2012 at 11:11 AM, Sean McBride <sean at rogue-research.com>
>>wrote:
>>> On Tue, 18 Dec 2012 15:48:40 +0000, Williams, Norman K said:
>>>
>>>>I should have caught this but like you I didn't have PHilipsRecImageIO
>>>>turned on.
>>>>
>>>>Is there a good reason not to have this ON by default?  And I'm all for
>>>>making all the factory builds turn on everything that's practical to turn
>>>>on.
>>>
>>> I'd like to turn it on on my dashboards, as I use that code.  In fact,
>>I had assumed it was on.  I have ITK_BUILD_ALL_MODULES=1, but apparently
>>that doesn't mean what I thought...
>
>



-- 
Unpaid intern in BillsBasement at noware dot com