<div dir="ltr">Yup, this is the bug we are addressing in the patch (<a href="http://review.source.kitware.com/#/c/12259/">http://review.source.kitware.com/#/c/12259/</a>). <div><br></div><div>And removing ITK_USE_REVIEW will make things less confused:</div>
<div><a href="http://review.source.kitware.com/#/c/12260/">http://review.source.kitware.com/#/c/12260/</a><br><div><br></div><div>Will merge the patches soon.<br><div><br></div></div></div></div><div class="gmail_extra"><br>
<br><div class="gmail_quote">On Thu, Aug 15, 2013 at 11:23 AM, Williams, Norman K <span dir="ltr"><<a href="mailto:norman-k-williams@uiowa.edu" target="_blank">norman-k-williams@uiowa.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
OK so this seems to make a difference: In addition to setting<br>
ITK_USE_REVIEW, I set Module_ITKREVIEW on the command line for the initial<br>
cmake.<br>
<br>
I notice also that if you run CMake a second time in the ITK build tree,<br>
it turns on installing review headers.<br>
<br>
So given how things appear to work, the business in the top-level<br>
CMakeList.txt here:<br>
<br>
if (ITK_USE_REVIEW)<br>
set(Module_ITKReview ON)<br>
endif()<br>
<br>
That doesn't work for some reason; probably because this happens before<br>
the Module macros run and enumerate through the ITK directory tree to<br>
discover the Modules to build and install.<br>
<br>
This is a strange CMake chicken & egg thing: I believe the top level code<br>
has the above clause, which sets a non-cache variable, but then lower down<br>
it does this:<br>
option(Module_${itk-module} "Request building ${itk-module}" OFF)<br>
<br>
But then in Modules/NonUnit/Review/CMakeLists.txt there's this rather<br>
forlorn, useless command:<br>
set(ITK_USE_REVIEW ON)<br>
<br>
<br>
So maybe in the next release remove ITK_USE_REVIEW because it's a relic of<br>
ITK 3 and doesn't effing work right. Or figure out how to telegraph into<br>
the Module enumeration logic that it should consider the module on.<br>
<div class="im HOEnZb"><br>
--<br>
Kent Williams <a href="mailto:norman-k-williams@uiowa.edu">norman-k-williams@uiowa.edu</a><br>
<br>
<br>
<br>
<br>
<br>
<br>
On 8/15/13 9:44 AM, "Matt McCormick" <<a href="mailto:matt.mccormick@kitware.com">matt.mccormick@kitware.com</a>> wrote:<br>
<br>
</div><div class="HOEnZb"><div class="h5">>Hi Kent,<br>
><br>
>We are in the process of deprecating ITK_USE_REVIEW due to incorrect<br>
>and complicated interaction with the Module_ITKReview variable. Are<br>
>you observing this behavior with recent master?<br>
><br>
>Thanks,<br>
>Matt<br>
><br>
>On Thu, Aug 15, 2013 at 2:26 PM, Williams, Norman K<br>
><<a href="mailto:norman-k-williams@uiowa.edu">norman-k-williams@uiowa.edu</a>> wrote:<br>
>> I don't have time to read through all of the CMake magic involved in ITK<br>
>> Modularization, so I'm hoping someone on the list knows what's up: Some<br>
>> projects that succeed when built against an ITK build tree fail when<br>
>>built<br>
>> against an ITK install, because they use headers from Review. For<br>
>>example<br>
>> itkLabelGeometryImageFilter.h<br>
>><br>
>> Is there something that is suppressing the installation of the<br>
>> NonUnit/Review stuff?<br>
>><br>
>> --<br>
>> Kent Williams <a href="mailto:norman-k-williams@uiowa.edu">norman-k-williams@uiowa.edu</a><br>
>><br>
>><br>
>><br>
>><br>
>><br>
>><br>
>> ________________________________<br>
>> Notice: This UI Health Care e-mail (including attachments) is covered<br>
>>by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is<br>
>>confidential and may be legally privileged. If you are not the intended<br>
>>recipient, you are hereby notified that any retention, dissemination,<br>
>>distribution, or copying of this communication is strictly prohibited.<br>
>>Please reply to the sender that you have received the message in error,<br>
>>then delete it. Thank you.<br>
>> ________________________________<br>
>> _______________________________________________<br>
>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>><br>
>> Visit other Kitware open-source projects at<br>
>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>><br>
>> Kitware offers ITK Training Courses, for more information visit:<br>
>> <a href="http://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</a><br>
>><br>
>> Please keep messages on-topic and check the ITK FAQ at:<br>
>> <a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br>
>><br>
>> Follow this link to subscribe/unsubscribe:<br>
>> <a href="http://www.itk.org/mailman/listinfo/insight-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
<br>
<br>
<br>
________________________________<br>
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged. If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited. Please reply to the sender that you have received the message in error, then delete it. Thank you.<br>
________________________________<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at<br>
<a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Kitware offers ITK Training Courses, for more information visit:<br>
<a href="http://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</a><br>
<br>
Please keep messages on-topic and check the ITK FAQ at:<br>
<a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.itk.org/mailman/listinfo/insight-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><br></div><div><br></div>---------------------------------------------<br><b>Xiaoxiao Liu</b>, Ph.D.<div><div><div><font face="arial, sans-serif"><span style="border-collapse:collapse">R & D Engineer</span></font></div>
<div><div><a href="http://www.kitware.com/" target="_blank">Kitware Inc</a>.</div></div><div><span style="border-collapse:collapse;font-family:arial,sans-serif;font-size:13px">Clifton Park, NY</span><div><div>Phone: (518) 881-4924 or <span style="border-collapse:collapse;color:rgb(136,136,136);font-family:arial,sans-serif;font-size:13px">(518) 371-3971 x124</span></div>
<div><font face="Verdana, Arial, Helvetica, sans-serif"><span style="line-height:31px;font-size:x-small"><br></span></font></div></div></div></div></div>
</div>