Re: [CMake] FindSubversion does not handle REQUIRED option

2010-01-28 Thread David Cole
The wiki page that lists module maintainers is found here: http://www.itk.org/Wiki/CMake:Module_Maintainers For FindSubversion.cmake, Tristan Carel is listed as the maintainer. Perhaps patches for FindSubversion.cmake would more appropriately be added to patch files attached to a bug report / fea

[CMake] FindSubversion does not handle REQUIRED option

2010-01-28 Thread Marcel Loose
There's a bug in the current FindSubversion.cmake, which causes it to ignore the REQUIRED option, when option QUIET is not given. The following patch solves this. IMHO, it would be even better to use FPHSA for this. Furthermore, I'm not sure whether it's desirable to conditionally define macros. T