Review Request 111326: Install all calligralibs headers in ${INCLUDE_INSTALL_DIR}/calligra

[Inge Wallin]

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


No comment on the patch yet but it seems to me that you have a larger plan behind all of these patches.  Is it just to get out of the way from other applications or is your ultimate goal something else?  None of the other large KDE apps, like the Kontact suite for instance, does this as far as I know.  And the Gnome apps definitely don't. Instead they go the other way and try to occupy as much namespace as possible and also preferably as generic as possible.  Case in point: The "Document" Foundation instead of The LibreOffice Foundation.

- Inge Wallin


On June 30, 2013, 9:38 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111326/
> -----------------------------------------------------------
> 
> (Updated June 30, 2013, 9:38 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> While almost all headers of Calligra libs are prefixed with "Ko"/"ko" it feels better to install them to their own subdir named "calligra/":
> * Easier for people using the libs and wanting to browse through all the available headers (until we improve http://api.kde.org/bundled-apps-api/calligra-apidocs/)
> * Easier also for developers to just remove the complete calligra/ dir to clean up
> * subdirs "styles/" & "changetracker/" are used by kotext headers, which feels strange directly in the normal include dir, without any prefix
> 
> Questions:
> * Should that "calligra/" be made a configuration option? Who might need/want change that to something else?
> * Should all the other apps also install below calligra/$APP, or continue to use "words/", "sheets/", "stage/" etc.?
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 2504353 
>   interfaces/CMakeLists.txt 36eb42a 
>   libs/basicflakes/CMakeLists.txt 4750c5f 
>   libs/flake/CMakeLists.txt 233d3ce 
>   libs/kopageapp/CMakeLists.txt deb7d76 
>   libs/kotext/CMakeLists.txt 8929d8a 
>   libs/kundo2/CMakeLists.txt cf66a13 
>   libs/main/CMakeLists.txt 147b25c 
>   libs/odf/CMakeLists.txt 574f8a5 
>   libs/pigment/CMakeLists.txt 8cc00e8 
>   libs/rdf/CMakeLists.txt 875ce13 
>   libs/textlayout/CMakeLists.txt cf2b83b 
>   libs/widgets/CMakeLists.txt 38155e7 
>   libs/widgetutils/CMakeLists.txt bfe64ca 
> 
> Diff: http://git.reviewboard.kde.org/r/111326/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

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