<table><tr><td style="">kossebau created this revision.<br />kossebau added reviewers: FreeBSD, tcberner, bshah, mart, davidedmundson.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />kossebau requested review of this revision.
</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/D10607" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Also improves race condition on creating the JSON file and running<br />
automoc over the cpp file which refers to it, given there is no<br />
dependency chain defined at all for the lookandfeeltool target and<br />
that kcm_lookandfeel.json, which was prone to make highly parallel<br />
builds fail</p>

<p>Moving the K_PLUGIN_FACTORY_WITH_JSON into a separate file only added<br />
to the sources of kcm_lookandfeel, instead of hiding with an ifndef for<br />
the build of the lookandfeeltool target as tried before, should avoid<br />
any potential confusion of automoc as reported from some builds<br />
(on KDE neon?).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>fixkcmlooknfeelmoctake2</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10607" rel="noreferrer">https://phabricator.kde.org/D10607</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/lookandfeel/CMakeLists.txt<br />
kcms/lookandfeel/kcm.cpp<br />
kcms/lookandfeel/kcmmain.cpp</div></div></div><br /><div><strong>To: </strong>kossebau, FreeBSD, tcberner, bshah, mart, davidedmundson<br /><strong>Cc: </strong>plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>