Review Request 113365: Fix #include <kdesu_export.h>
Alex Merry
kde at randomguy3.me.uk
Mon Oct 21 17:15:13 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113365/#review42119
-----------------------------------------------------------
Ship it!
Looks sane, and builds fine even when I remove the installed kdesu/kdesu_export.h file.
- Alex Merry
On Oct. 21, 2013, 12:27 p.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113365/
> -----------------------------------------------------------
>
> (Updated Oct. 21, 2013, 12:27 p.m.)
>
>
> Review request for KDE Frameworks and Martin Klapetek.
>
>
> Repository: kdelibs
>
>
> Description
> -------
>
> kdesu headers are installed to kdesu/ so the #include <kdesu_export.h> cannot be found if one tries to use kdesu includes in a project.
>
>
> Diffs
> -----
>
> tier3/kdesu/src/client.h 8f29b93
> tier3/kdesu/src/process.h c527e83
> tier3/kdesu/src/ssh.h 8ab6472
> tier3/kdesu/src/stub.h 68d7440
> tier3/kdesu/src/su.h e83781d
>
> Diff: http://git.reviewboard.kde.org/r/113365/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gräßlin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131021/e7b2c3f0/attachment.html>
More information about the Kde-frameworks-devel
mailing list