[Kde-hardware-devel] Review Request 120635: Only build test logindinhibitionargument with WITH_NEW_POWER_ASYNC_API AND WITH_NEW_POWER_ASYNC_FREEDESKTOP

[Friedrich W. H. Kossebau]

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


Actually Solid curently also fails on build.kde.org with the same error: http://build.kde.org/view/Frameworks/job/solid_master_qt5/118/consoleText

- Friedrich W. H. Kossebau


On Okt. 18, 2014, 12:44 vorm., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120635/
> -----------------------------------------------------------
> 
> (Updated Okt. 18, 2014, 12:44 vorm.)
> 
> 
> Review request for Solid and Àlex Fiestas.
> 
> 
> Repository: solid
> 
> 
> Description
> -------
> 
> Build fails for me with
> ```
> [ 96%] Building CXX object autotests/CMakeFiles/logindinhibitionargument.dir/logindinhibitionargumenttest.cpp.o
> In file included from /home/koder/Kode/kdegit/kf5/frameworks/solid/autotests/logindinhibitionargumenttest.cpp:22:0:
> /home/koder/Kode/kdegit/kf5/frameworks/solid/autotests/../src/solid/power/backends/freedesktop/logindinhibitionargument.h:25:25: fatal error: solid/power.h: Datei oder Verzeichnis nicht gefunden
>  #include "solid/power.h"
>                          ^
> compilation terminated.
> make[2]: *** [autotests/CMakeFiles/logindinhibitionargument.dir/logindinhibitionargumenttest.cpp.o] Fehler 1
> make[1]: *** [autotests/CMakeFiles/logindinhibitionargument.dir/all] Fehler 2
> make: *** [all] Fehler 2
> ```
> 
> Actually I am puzzled how all the `"solid/power.h"` or `<solid/job.h>` includes are resolved on the very first build of Solid (when nothing has been installed yet). I could not find any such path in the source dirs or the build dirs. Is there some cmake magic which injects those path in a virtual way during the build?
> 
> In any case, that seems to be missing when building the test "logindinhibitionargument", so that include there cannot be resolved. This patch is just a workaround to make things building for those where "power/backends/freedesktop" would not be built anyway, even if that class does not have any external includes that have the same deps of that backend.
> 
> You might have a better idea what the real fix would be.
> 
> 
> Diffs
> -----
> 
>   autotests/CMakeLists.txt 621665c 
> 
> Diff: https://git.reviewboard.kde.org/r/120635/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20141018/7ed561bc/attachment.html>