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

Friedrich W. H. Kossebau kossebau at kde.org
Thu Feb 21 01:55:15 GMT 2013


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

Review request for Calligra.


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
-----

  words/part/CMakeLists.txt 56b8c6f 
  words/part/Info.plist.template 97e1728 
  words/part/main.cpp 875eb5d 
  words/part/words.desktop 35bc4c3 
  words/app/main.cpp PRE-CREATION 
  words/app/words.desktop PRE-CREATION 
  words/app/CMakeLists.txt PRE-CREATION 
  words/app/Info.plist.template PRE-CREATION 
  stage/part/Info.plist.template 857a8d7 
  stage/part/main.cpp 5ef9509 
  stage/part/stage.desktop 447858f 
  tools/CMakeLists.txt eefe95b 
  words/CMakeLists.txt e6336a2 
  3rdparty/CMakeLists.txt a300bd2 
  CMakeLists.txt d9945a1 
  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 

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/20130221/2d0e63b4/attachment.htm>


More information about the calligra-devel mailing list