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>