Review Request 109071: Add a PRODUCTSET build config parameter (values ACTIVE, CREATIVE, DESKTOP, ALL)

Friedrich W. H. Kossebau kossebau at kde.org
Sun Feb 24 00:11:21 GMT 2013


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

(Updated Feb. 24, 2013, 12:11 a.m.)


Review request for Calligra.


Changes
-------

Added use of macros for easier to read code, and moved product set definitions completely into toplevel CMakeLists.txt (sure, quite long now, but read on...)

Actually those product set definitions could be done in included cmake files, especially if the filters will be also included into the product set definitions.
With one separate file per product set this will make it even easier to define your own product set, without having to hack the buildsystem.

And those separate files could also be generated from a product set/build configuration tool, like Jaros?aw proposed to have. 

But lets do it in little steps and start with this patch :)

Separating the "app/" for Stage and Words is waiting the final "Ship it" at https://git.reviewboard.kde.org/r/109082/ (hint, hint...), so this patch would be rebased and committed only once that other is committed.


Description
-------

PRODUCTSET is a substitute for the old non-exclusive options CREATIVEONLY and TINY (which then are handled exclusively, eek), with migration support for CREATIVEONLY flag. Predefined hardcoded productsets are ACTIVE, CREATIVE, DESKTOP, and ALL (as fallback and default).
    
Patch also turns buildsystem to have a SHOULD_BUILD for each product (app/plugin), which then all get turned on centrally in groups depending on the productset, instead of everywhere having overlapping and hard to oversee if-else blocks deciding what gets build and what not.

Not the perfect final solution, but a first step into the right direction IMHO.

Known issues:
* BUILD_AUTHOR is not yet set

Patch can be also tested as branch addProductSetBuildParameter-kossebau.


Diffs (updated)
-----

  stage/part/stage.desktop 447858f 
  words/CMakeLists.txt e6336a2 
  words/app/CMakeLists.txt PRE-CREATION 
  words/app/Info.plist.template PRE-CREATION 
  words/app/main.cpp PRE-CREATION 
  words/app/words.desktop PRE-CREATION 
  words/part/CMakeLists.txt 56b8c6f 
  words/part/Info.plist.template 97e1728 
  words/part/main.cpp 875eb5d 
  words/part/words.desktop 35bc4c3 
  CMakeLists.txt 564c0a0 
  devtools/CMakeLists.txt f0e7b62 
  extras/CMakeLists.txt 70edca7 
  filters/CMakeLists.txt 5acecef 
  filters/sheets/CMakeLists.txt 351a8e2 
  filters/words/CMakeLists.txt 0c2107c 
  libs/CMakeLists.txt 2036cf5 
  plugins/CMakeLists.txt 0e87b1e 
  sheets/CMakeLists.txt 9f96e41 
  stage/CMakeLists.txt 94dd31c 
  stage/app/CMakeLists.txt PRE-CREATION 
  stage/app/Info.plist.template PRE-CREATION 
  stage/app/main.cpp PRE-CREATION 
  stage/app/stage.desktop PRE-CREATION 
  stage/part/CMakeLists.txt de57a0f 
  stage/part/Info.plist.template 857a8d7 
  stage/part/main.cpp 5ef9509 
  3rdparty/CMakeLists.txt a300bd2 

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


Testing
-------


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130224/37ddb582/attachment.htm>


More information about the calligra-devel mailing list