<table><tr><td style="">kossebau 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/D10724" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>See related discussion on CMake issue tracker: <a href="https://gitlab.kitware.com/cmake/cmake/issues/17750" class="remarkup-link" target="_blank" rel="noreferrer">https://gitlab.kitware.com/cmake/cmake/issues/17750</a></p>
<p>Similar patches will be also done to the other Framework modules which need such extra handling. Will see to upload them as well tonight.</p>
<p>Given KF 5.44 tagging is in two weeks, I do not want to rush the creation of the related desired ECM macro, but instead design it now and then add it only for 5.45, so we developers have some more time to test it before the API becomes stable. Still brushing over it, but should arrive as review request once the explicit code here has been reviewed and collected feedback.</p>
<p>The ARG2 in the var name is already a hint to what is coming, as there are also variants with ARG3 when the JSON file argument is the 3rd arg in the related macro call.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10724" rel="noreferrer">https://phabricator.kde.org/D10724</a></div></div><br /><div><strong>To: </strong>kossebau, Frameworks, Build System, apol<br /><strong>Cc: </strong>michaelh<br /></div>