<table><tr><td style="">thomasfischer updated this revision to Diff 68072.<br />thomasfischer added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24641">View Revision</a></tr></table><br /><div><div><p>I rewrote the patch to address the comments:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Setting <tt style="background: #ebebeb; font-size: 13px;">ECM_SOURCE_UNDER_VERSION_CONTROL</tt> has been refactored to set variable <tt style="background: #ebebeb; font-size: 13px;">ECM_SOURCE_VERSION_CONTROL_WHICH</tt> as well to determine which of three known VCSes are used. <tt style="background: #ebebeb; font-size: 13px;">ECM_SOURCE_VERSION_CONTROL_WHICH</tt> contains a string like "git" or "svn".</li>
<li class="remarkup-list-item">The more expensive check where an external program ("git") has to be run got refactored into a macro (<tt style="background: #ebebeb; font-size: 13px;">ecm_source_version_control_stat</tt>). Support for Subversion, Mercurial, and Bazaar got removed. Messages will be printed to show the status of the analysis. It is a macro, not a function, to have changes to variables like <tt style="background: #ebebeb; font-size: 13px;">ECM_SOURCE_VERSION_CONTROL_BRANCH</tt> visible outside of the function.</li>
</ol>

<p>To clarify <tt style="background: #ebebeb; font-size: 13px;">ECM_SOURCE_VERSION_CONTROL_COMMIT_COUNT</tt>: it counts the number of commits in direct line of succession from the repository's initialization to the current commit. It does not include commits in other branches. Basically the number of commits listed in a plain <tt style="background: #ebebeb; font-size: 13px;">git log</tt>. The commit count gives a quick indication of the progress of a repository (or branch) without requiring to look up the repo's commit messages or hashes.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D24641?vs=67922&id=68072">https://phabricator.kde.org/D24641?vs=67922&id=68072</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24641">https://phabricator.kde.org/D24641</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>modules/ECMSourceVersionControl.cmake</div></div></div><br /><div><strong>To: </strong>thomasfischer, sitter, kossebau<br /><strong>Cc: </strong>kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, michaelh, ngraham, bruns<br /></div>