<table><tr><td style="">meven added inline comments.
</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/D22143">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D22143#inline-124738">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kossebau</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:21</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Instead of relying on an undocumented  cpp file name generated by ecm_qt_declare_logging_category, you rather want to have a separate SRC variable which carries the source files generated by the macro, and add its content here by instead adding this to the list:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">${KActivitiesStats_LOG_SRCS}</pre></div>

<p style="padding: 0; margin: 8px;">That variable will also make it easier to track where this file is actually coming from, instead having to guess it.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">While it is true, the solution here is not ideal, it is quite commonly used in kde projects and anyone opening the CMakeLists.txt should come up pretty quickly that this file is being generated elsewhere.<br />
I just lack a viable alternative. I had a hard time to uncover this solution in the first place.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R159 KActivities Statistics</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22143">https://phabricator.kde.org/D22143</a></div></div><br /><div><strong>To: </strong>meven, ivan, Frameworks<br /><strong>Cc: </strong>kossebau, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns<br /></div>