[Marble-devel] Review Request: Endorse marble/class.h #includes to libraries (instead of simple class.h)

Commit Hook null at kde.org
Mon May 14 19:55:24 UTC 2012


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


This review has been submitted with commit 87cce17ba8156d7aee03a799b7fda8e78702f290 by Dennis Nienhüser to branch master.

- Commit Hook


On Jan. 8, 2012, 10:30 a.m., Dennis Nienhüser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103656/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2012, 10:30 a.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> Our current FindMarble.cmake script installed to the cmake module path puts the marble suffix into the include search path. This means that an
> #include <marble/MarbleWidget.h>
> does not work in general (only if the path below happens to be in the search path by some other reasons, e.g. because it is the standard path /usr/include). This is quite bad as something like #include <global.h> is likely to clash with other libraries' includes.
> 
> 
> This addresses bug 282865.
>     http://bugs.kde.org/show_bug.cgi?id=282865
> 
> 
> Diffs
> -----
> 
>   FindMarble.cmake 7f81911 
> 
> Diff: http://git.reviewboard.kde.org/r/103656/diff/
> 
> 
> Testing
> -------
> 
> The examples in examples/cpp (which have <marble/class.h> includes) now work for local installations when configuring cmake with
> 
> cmake -DCMAKE_MODULE_PATH=~/marble/export/share/marble/cmake \
>       -DCMAKE_INCLUDE_PATH=~/marble/export/include \
>       -DCMAKE_LIBRARY_PATH=~/marble/export/lib \
>       .
> 
> Previously MARBLE_INCLUDE_DIR hat to be adjusted manually.
> 
> 
> Thanks,
> 
> Dennis Nienhüser
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20120514/c2465040/attachment.html>


More information about the Marble-devel mailing list