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

Friedrich W. H. Kossebau kossebau at kde.org
Sun Mar 3 23:25:02 GMT 2013


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

(Updated March 3, 2013, 11:24 p.m.)


Review request for Calligra.


Changes
-------

Now with automatic inclusion of internal required products + backward support for -DBUILD_app

A few other ideas are TODO for now, but should be good for master already.

Try branch addProductSetBuildParameter-kossebau to see it working (hopefully) :)


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

  3rdparty/CMakeLists.txt a300bd2 
  CMakeLists.txt 564c0a0 
  cmake/modules/CalligraProductSetMacros.cmake PRE-CREATION 
  cmake/productsets/README PRE-CREATION 
  cmake/productsets/active.cmake PRE-CREATION 
  cmake/productsets/all.cmake PRE-CREATION 
  cmake/productsets/creative.cmake PRE-CREATION 
  cmake/productsets/desktop.cmake PRE-CREATION 
  devtools/CMakeLists.txt f0e7b62 
  devtools/cstester/CMakeLists.txt a12ea92 
  doc/CMakeLists.txt 205b34a 
  extras/CMakeLists.txt 70edca7 
  extras/quickprint/CMakeLists.txt 1cb9cc3 
  extras/thumbnail/CMakeLists.txt 9b2ef6f 
  filters/CMakeLists.txt 5acecef 
  filters/plan/mpxj/CMakeLists.txt 237d214 
  filters/sheets/CMakeLists.txt 351a8e2 
  filters/words/CMakeLists.txt 0c2107c 
  kexi/CMakeLists.txt 54cdabb 
  kexi/kexidb/drivers/CMakeLists.txt bfa46ad 
  kexi/migration/CMakeLists.txt aff9d47 
  libs/CMakeLists.txt 2036cf5 
  plan/plugins/filters/CMakeLists.txt c3115da 
  plugins/CMakeLists.txt 0e87b1e 
  plugins/kexi/CMakeLists.txt 704736c 
  plugins/reporting/CMakeLists.txt 72ae5ee 
  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 
  stage/part/stage.desktop 54c3be9 
  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 363860c 

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/20130303/2d2c9227/attachment.htm>


More information about the calligra-devel mailing list