Review Request 124918: fix interface name

Harald Sitter sitter at kde.org
Mon Aug 31 08:11:18 UTC 2015


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

(Updated Aug. 31, 2015, 8:11 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Daniel Vrátil.


Changes
-------

Submitted with commit 3551aca3f1016d44e0b8d5a6d454859ba6a89773 by Harald Sitter to branch master.


Repository: kscreen


Description
-------

I am not quite confident how this worked (best guess is target order in
cmake). upower_interface.h is not used anywhere, what is used is
freedesktop_interface.h (in device.cpp) an consequently the build can
fail on this file not being found.

Switch the generated header to the actually used name to avoid build
failures from not being able to find it.

e.g. https://launchpadlibrarian.net/215492814/buildlog_ubuntu-wily-i386.kscreen_4%3A5.3.95%2Bgit20150825.0935%2B15.10-0_BUILDING.txt.gz
the error happens and in another concurrent make job the interface is
built in the kded/ directory, so best guess is that this largely depends
on timing to go wrong.


Diffs
-----

  tests/kded/CMakeLists.txt e4b6bf5ec1383807826f312280f1fbea30fe388b 

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


Testing
-------

make (make test doesn't want to finish for some reason)


Thanks,

Harald Sitter

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150831/88d52347/attachment.html>


More information about the Plasma-devel mailing list