<table><tr><td style="">rjvbb 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/D8158" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Does drkonqui still automatically select the right version in Bugzilla if we do this ...? (It does that, right?)</p></blockquote>

<p>I haven't gone so far as to submit an actual bug report. I'd hope that some kind of pattern matching is done, either in DrKonqi or by the Bugzilla software itself. Which btw. only seems to know release versions.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Looking at it, no, you definitely cannot do it like this. You overwrite KDevelop_VERSION_PATCH, which is e.g. exported to kdevelop-config.cmake and will require e.g. kdev-php to depend on "kdevelop VERSION 5.1.80-24-g5623457794"...</p></blockquote>

<p>I wasn't sure about that because I can find no evidence that this version is exported in a headerfile or .cmake file. What did I overlook?<br />
Is it safer to rewrite just the full version string (and would it be acceptable/safe to include the 'v' in there - if not it can be removed in software)?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Also, you run this at build generation time, so if you do a change + commit which don't cause cmake to re-generate the build system, the version number won't change.</p>

<p>Also, as said, this will just cause the build to fail on Windows.</p></blockquote>

<p>How many bug reports do you get that are generated automatically on MS Windows (via DrKonqi or the built-in bug reporter)?</p>

<p>I see 2 options, presuming that the feature is deemed of interest:<br />
1 exclude MS Windows, which would make it possible (or simpler) to write something that always updates the full version string<br />
2 don't a helper script, execute git directly from the CMake file to rewrite just a full version string used by KAboutData, and do the required post-processing in main.cpp</p>

<p>With option 2) it would be the developer's responsibility to take care of regenerating the version string, for instance by removing $build.dir/kdevelop-config.h in post-commit and post-fetch hooks. We could even provide instructions for that.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8158" rel="noreferrer">https://phabricator.kde.org/D8158</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, brauch<br /><strong>Cc: </strong>brauch, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>