Review Request 111954: proper use filter_<app> products flag to enable/disable whole app filters

[Sebastian Sauer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111954/
-----------------------------------------------------------

Review request for Calligra and Friedrich W. H. Kossebau.


Description
-------

Those new product flags define for each top-level application a matching FILTER_<appname> flag. That flag allows to disable all filters for a certain app(s) but it (seems?) they are not evaluated and applied. Following patch works for me in that if I now add something like

calligra_disable_product(FILTER_WORDS "Qt5 TODO")
calligra_disable_product(FILTER_SHEETS "Qt5 TODO")
calligra_disable_product(FILTER_STAGE "Qt5 TODO")

to the top-level CMakeLists.txt then all filters for Words, Sheets and Stage are disabled means not build.


Diffs
-----

  filters/CMakeLists.txt 79a42a2 

Diff: http://git.reviewboard.kde.org/r/111954/diff/


Testing
-------

Note that testing was done with the Qt5-branch. I ask for review (and merge to master) so I get feedback I didn't got something wrong here how those product-flags are supposed to work. Since there are way more such switches I may have to add more such cases here and there and so it would make sense to at least have the initial patch reviewed asap before investing time into solving a not-existing problem and/or a solving it in a wrong way :)


Thanks,

Sebastian Sauer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130808/807bcaff/attachment.htm>