Review Request 118510: small rework of product sets: products, features and product sets

Friedrich W. H. Kossebau kossebau at kde.org
Wed Jun 4 20:39:05 BST 2014


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

(Updated June 4, 2014, 7:39 p.m.)


Review request for Calligra, Yue Liu, Boudewijn Rempt, and Jarosław Staniek.


Changes
-------

Brushed over some more, should be good enough and an improvement for master now.


Repository: calligra


Description
-------

Based on the feedback, adaption and more own tinkering with the concepts, in this patch I propose to do this:

* split the old idea of a "product" into "product", "feature" and "productset"
* make no real difference between user-defined productsets and predefined ones
* move the definition of the products into a separate file, for better overview and discoverability
* put the documentation directly into that separate file instead of hiding it away in some README in some subdirectory noones knows about
* generate a file with the graph of the dependencies, showing also the build state (in DOT notation, see attached SVG file generated from that)
* more patterns for the ids of the products
* allow to have multiple productsets/products/features mentioned in the -DPRODUCTSET flag (taken with WANTED property for now) (as asked for with https://git.reviewboard.kde.org/r/118187)

Patch needs some more brush over, but I wanted to already get first feedback from you, to learn if you are okay with this direction or think this is going nuts.

Surely this whole thing is in danger to be overengineered :) Especially given that Krita devs have not picked up this for all the Sketch, Gemini, Steam etc variants makes me wonder if it is perhaps. But then it might just be underdocumented and uncommon and people have not yet discovered the nice build customizations this offers over what is usually possible. Time to roll the drums about this, mean, blog :)

One thing still missing: proper disabling of products with selected product sets. But lets leave that for a separate PR.


Diffs (updated)
-----

  CMakeLists.txt 256d810 
  CalligraProducts.cmake PRE-CREATION 
  README.PACKAGERS 4ebc026 
  active/CMakeLists.txt 4361fb6 
  cmake/modules/CalligraProductSetMacros.cmake 697c046 
  cmake/productsets/README 5460955 
  cmake/productsets/active.cmake 089a085 
  cmake/productsets/all.cmake 778f9c0 
  cmake/productsets/creative.cmake 2b8fc45 
  cmake/productsets/desktop.cmake 97ff7e8 
  cmake/productsets/kexi+sheets.cmake 3fe2f48 
  cmake/productsets/kexi.cmake 7dd2345 
  cmake/productsets/krita.cmake 2c814d3 
  cmake/productsets/osx.cmake 8e32b13 
  cmake/productsets/sheets.cmake 5de2103 
  devtools/CMakeLists.txt bfa5599 
  extras/CMakeLists.txt e603883 
  extras/quickprint/CMakeLists.txt 9ad61d2 
  extras/thumbnail/CMakeLists.txt 92156b3 
  filters/flow/visio/import/CMakeLists.txt 1de4385 
  filters/flow/wpg/import/CMakeLists.txt 524d1b1 
  filters/karbon/karbon1.x/CMakeLists.txt 87bdb73 
  filters/karbon/wmf/CMakeLists.txt c25ed2c 
  filters/karbon/wpg/CMakeLists.txt 5f578e9 
  filters/karbon/xfig/CMakeLists.txt 1e0e2c6 
  filters/sheets/excel/import/CMakeLists.txt 07aa186 
  filters/sheets/xlsx/CMakeLists.txt 80d51aa 
  filters/stage/keynote/import/CMakeLists.txt 1b44b52 
  filters/stage/kpr2odf/CMakeLists.txt 31c69b0 
  filters/stage/powerpoint/CMakeLists.txt 60e1ccc 
  filters/stage/pptx/CMakeLists.txt 31aef35 
  filters/words/docx/import/CMakeLists.txt 036e828 
  filters/words/msword-odf/CMakeLists.txt 8342552 
  filters/words/rtf/import/CMakeLists.txt df6f50d 
  filters/words/wordperfect/import/CMakeLists.txt 6a0e2d7 
  filters/words/works/import/CMakeLists.txt 76317ae 
  kexi/plugins/CMakeLists.txt bf389f2 
  krita/plugins/formats/ora/CMakeLists.txt cd1a605 
  libs/CMakeLists.txt d482a4b 
  libs/kotext/CMakeLists.txt 9f9e506 
  plan/plugins/CMakeLists.txt e06e339 
  plugins/textshape/CMakeLists.txt f8d75f5 
  sheets/CMakeLists.txt a45829d 
  sheets/plugins/CMakeLists.txt 6a45389 
  stage/CMakeLists.txt eaf8ecf 
  words/CMakeLists.txt 033db4d 
  words/part/CMakeLists.txt 2bf1b0b 
  words/part/tests/CMakeLists.txt 6fd9b0c 
  words/plugins/CMakeLists.txt 229cff8 

Diff: https://git.reviewboard.kde.org/r/118510/diff/


Testing
-------


File Attachments
----------------

Dependencies graph
  https://git.reviewboard.kde.org/media/uploaded/files/2014/06/04/b1958536-ca1c-4552-8c3d-120a9d8b786f__dependencies.svg
Dependencies graph with just Kexi & Sheets built
  https://git.reviewboard.kde.org/media/uploaded/files/2014/06/04/8924bbdf-b05c-426d-b469-d5bceb87bac7__kexisheetdependencies.svg


Thanks,

Friedrich W. H. Kossebau

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


More information about the calligra-devel mailing list