----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118510/#review60150 -----------------------------------------------------------
This review has been submitted with commit 7fe5f269ddfde50770383fee43480fc33c65d3e6 by Friedrich W. H. Kossebau to branch master. - Commit Hook On June 14, 2014, 2:07 a.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118510/ > ----------------------------------------------------------- > > (Updated June 14, 2014, 2:07 a.m.) > > > Review request for Calligra, Yue Liu, Boudewijn Rempt, and Jarosław Staniek. > > > 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) > > 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 RR. > > > Diffs > ----- > > CMakeLists.txt 886e2c3 > 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 4619358 > cmake/productsets/creative.cmake 2b8fc45 > cmake/productsets/desktop.cmake 83db3a1 > 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 a534b0e > extras/quickprint/CMakeLists.txt 9ad61d2 > extras/thumbnail/CMakeLists.txt 92156b3 > filters/flow/visio/import/CMakeLists.txt 7f040f0 > filters/flow/wpg/import/CMakeLists.txt caee3b4 > filters/karbon/karbon1.x/CMakeLists.txt 87bdb73 > filters/karbon/wmf/CMakeLists.txt c25ed2c > filters/karbon/wpg/CMakeLists.txt ffa7742 > 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 f8d5781 > filters/stage/kpr2odf/CMakeLists.txt 31c69b0 > filters/stage/powerpoint/CMakeLists.txt 60e1ccc > filters/stage/pptx/CMakeLists.txt 31aef35 > filters/words/docx/import/CMakeLists.txt d9f0cec > filters/words/msword-odf/CMakeLists.txt 8bd3806 > filters/words/rtf/import/CMakeLists.txt df6f50d > filters/words/wordperfect/import/CMakeLists.txt 6e8446f > filters/words/works/import/CMakeLists.txt 93d40fb > 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 > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
