<br><br><div class="gmail_quote">On Wed, Sep 12, 2012 at 2:50 PM, Allen Winter <span dir="ltr"><<a href="mailto:winter@kde.org" target="_blank">winter@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On Wednesday 12 September 2012 08:12:58 PM Thomas Lübking wrote:<br>
> Am 12.09.2012, 20:04 Uhr, schrieb Dawit A <<a href="mailto:adawit@kde.org">adawit@kde.org</a>>:<br>
><br>
> > Qt's moc processor does not<br>
> > seem to correctly handle #if macros.<br>
><br>
> *sigh* "not at all" is more like it. **GREATSIGH***<br>
><br>
> You'll have to move the class into an extra header and make it's inclusion<br>
> and moc invocation (latter in cmake) optional.<br>
><br>
> **** DOUBLEGREATSIGH *****<br>
<br>
</div></div>Dawit,<br>
<br>
I agree with Thomas.  Can you try to put the new class into a separate file?<br>
and then only compile the new file if kde version >= 4.9.2<br></blockquote><div><br></div><div>Yes, I can and that is already what I have done locally to workaround the shortcomings of the moc preprocessor. However, I though I would ask to see if it would have been acceptable to avoid the addition of new files and use of conditional macros by changing the minimum kdelibs version requirement for the kde-baseapps git master repo instead. I guess the consensus is NO and that is fine by me. I will commit what I have already done.</div>

<div><br></div><div>Regards,</div><div>Dawit A.</div></div>