I thought Windows already had QtDBus with winDBus?<div><br></div><div>Keith Rusler<br><br><div class="gmail_quote">On 9 May 2010 05:03, Arno Rehn <span dir="ltr"><<a href="mailto:arno@arnorehn.de">arno@arnorehn.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div></div><div class="h5">On Sunday 09 May 2010 07:39:46 Maciej Mrozowski wrote:<br>
> My initial problem was to allow not building some SMOKE bindings (like<br>
> QtMultimedia, QtWebkit and such which now are mandatory dependencies), then<br>
> I've decided to come up with something to reduce copy&paste in toplevel<br>
> Smoke CMakeLists.txt and possible increase manageability.<br>
><br>
> Changes:<br>
> - introduce macro smoke_add_bindings(<var> <name> <subdir1> [subdir2 ...])<br>
> - it's basically a little smarter version of<br>
> macro_optional_add_subdirectory In this macro, I've also added CMake<br>
> option to disable particular bindings (DISABLE_SMOKE_<sth> is more correct<br>
> than ENABLE_SMOKE_<sth> because with the latter one cannot really assure<br>
> than all satisfied deps are found, and disabling always works)<br>
> - use macro instead of set(SMOKE_ENABLED... ) and add_subdirectory(...)<br>
</div></div>I'd rather not have this configurable. It just clutters the cmake variable<br>
cache for stuff that should be auto-detected. We should check whether<br>
QtMultimedia or QtWebKit are available, true, but please don't add an option<br>
to enable or disable a specific smoke module. We would have to do the same for<br>
QtRuby and Qyoto then and with ca. 40 smoke libs + 2*40 matching bindings we<br>
end up with 120 additional variables that nearly no-one ever modifies and mess<br>
up UIs like cmake-gui or ccmake.<br>
If a package is there, we build smoke libs for it. Period. If you don't want a<br>
certain package, use -DWITH_akonadi=FALSE  (or similar) to disable ALL<br>
bindings for it. This doesn't allow for as much fine-tuning as your solution,<br>
yes, but it's sufficient for nearly all situations.<br>
<br>
But please go ahead and make a patch that checks for Qt modules being there<br>
before adding the bindings. We'll need that anyway when we later port to<br>
Windows, where e.g. QtDBus is not available.<br>
<font color="#888888"><br>
--<br>
Arno Rehn<br>
<a href="mailto:arno@arnorehn.de">arno@arnorehn.de</a><br>
_______________________________________________<br>
Kde-bindings mailing list<br>
<a href="mailto:Kde-bindings@kde.org">Kde-bindings@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-bindings" target="_blank">https://mail.kde.org/mailman/listinfo/kde-bindings</a><br>
</font></blockquote></div><br></div>