[CMake] install() and EXCLUDE_FROM_ALL issue ( patch review )

[Brad King]

Alexandru Ciobanu wrote:
> You are right. The patch does not do the right thing.
> I just observed that it also completely removes the
> preinstall rule for excluded directories.
> 
> So make install insde those dirs will fail because there
> is no preinstall.
> 
> Alexander Neundorf wrote:
>>
>> I guess the EXCLUDE_FROM_ALL targets will then also not be relinked if
>> they are installed, right ?
>>
>> Maybe for EXCLUDE_FROM_ALL targets the relinking should happen when
>> the target gets built, not when it's installed ?

Actually the patch may not have been as far off as you think.  It just
has to be applied to half of the contents inside the if statement.  The
full patch against 2.4.7 is attached.

I'll submit this for inclusion in the 2.4 branch.  It has been fixed in
the CVS version.

-Brad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: exclude_preinstall.patch
Type: text/x-patch
Size: 1548 bytes
Desc: not available
Url : http://public.kitware.com/pipermail/cmake/attachments/20070803/bfcc43af/exclude_preinstall.bin