<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>(For the record)</div><div><br></div><div>Following your advice, I just added the following condition</div><div><br></div><div>IF( EXISTS ${CMAKE_SOURCE_DIR}/.svn/ )</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>...</div><div>ENDIF()</div><div><br></div><div>And now it works fine!</div><div><br></div><div>Even if this condition is not included in FindSubversion.cmake, I guess it could be added in the documentation (header) of this file.</div><div><br></div><div>Let's see what Tristan says</div><div><br></div><div>Arnaud</div><div><br><div><div>On Apr 9, 2010, at 11:35 AM, David Cole wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Fri, Apr 9, 2010 at 11:25 AM, Arnaud Gelas <span dir="ltr"><<a href="mailto:Arnaud_Gelas@hms.harvard.edu">Arnaud_Gelas@hms.harvard.edu</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> <div style="word-wrap:break-word"><div>David,</div><div><br></div><div>you're right Suversion_WC_INFO should only focus on svn working copy! Such kind of features should be included in a specific module (git-svn one for instance).</div> <div><br></div><div>I have just checked the documentation of git svn info which returns a lot of important information, but does not support yet git svn info --revision... So, I guess we should wait for this one... </div> <div><br></div><div>Now regarding Subersion_WC_INFO, could it be modified in such a way that if first check of a svn working copy, send a message status if not and return a default value ?</div><div><br></div><div>Just want to make sure, this could be an "expected" behavior...</div> <div><br></div></div></blockquote><div><br></div><div>I think that might be useful... but it is not there in existing cmake releases. So... if a project must use an existing cmake release, it needs to add the check for the working copy (however it thinks best to implement such a thing...) in its own source code.</div> <div><br></div><div>Modifications to Subversion_WC_INFO should be filtered through its module maintainer. If he thinks it's ok, then it's probably ok.</div><div><br></div><div>This page:</div><div><a href="http://www.cmake.org/Wiki/CMake:Module_Maintainers">http://www.cmake.org/Wiki/CMake:Module_Maintainers</a></div> <div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br></span></font></div><div><span class="Apple-style-span" style="font-family: sans-serif; font-size: 13px; line-height: 19px; ">...lists "tristan dot carel at gmail dot com" as the FindSubversion module maintainer...</span></div> <div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br></span></font></div><div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;">Let's see what he says.</span></font></div> <div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br></span></font></div><div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br> </span></font></div><div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;">David</span></font></div><div><font class="Apple-style-span" face="sans-serif"><span class="Apple-style-span" style="line-height: 19px;"><br> </span></font></div></div> <span><ATT00001..txt></span></blockquote></div><br></div></body></html>