<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">hi again,<div><br><div><div>On May 5, 2010, at 6:59 AM, Petr VanÄ›k wrote:</div><blockquote type="cite"><div><font class="Apple-style-span" color="#006312"><br></font>Yes, my solution is based on "private" variables. It's srupid. I'm in contact <br>with FindQt4.cmake maintainer and we are looking for better solution.<br><br>It looks like it will be more complicated because of different include file <br>locations in qt4 from macports and binary package. Please, wait with the <br>patch, I'll suggest a better one in few days.<br></div></blockquote></div><br></div><div>please look at this patch:</div><div><a href="http://qore.svn.sourceforge.net/viewvc/qore?view=rev&revision=3462">http://qore.svn.sourceforge.net/viewvc/qore?view=rev&revision=3462</a></div><div><br></div><div>main changes are in smoke/qt CMakeList.txt:</div><div><a href="http://qore.svn.sourceforge.net/viewvc/qore/module-qt4/trunk/smoke/qt/CMakeLists.txt?r1=3462&r2=3461&pathrev=3462&diff_format=u">http://qore.svn.sourceforge.net/viewvc/qore/module-qt4/trunk/smoke/qt/CMakeLists.txt?r1=3462&r2=3461&pathrev=3462&diff_format=u</a></div><div><br></div><div>it creates 2 new variables to handle mac inlcude dirs and includes. Then there are some tests if it's using really frameworks etc.</div><div><br></div><div>there is new file with includes for mac frameworks:</div><div><br></div><div><a href="http://qore.svn.sourceforge.net/viewvc/qore/module-qt4/trunk/smoke/qt/qt_includes_macframework.h?diff_format=u&revision=3462&view=markup&pathrev=3462">http://qore.svn.sourceforge.net/viewvc/qore/module-qt4/trunk/smoke/qt/qt_includes_macframework.h?diff_format=u&revision=3462&view=markup&pathrev=3462</a></div><div><br></div><div>its usage is set in the CMakeList.txt</div><div><br></div><div>now I'm opened to all critics and suggestions.</div><div>I tested it on: linux (opensuse), macosx (leopard) with 1) macports qt4-mac package and 2) qt4 binary package from nokia too.</div><div><br></div><div>cheers,</div><div>Petr</div><div><br></div><div>P.S.; all other files in this patch are unrelated to this problem (generator fix from upstream, disabled some classe we don't use now).</div></body></html>