Review Request 115715: Remove X11 dependency from kprintutils
Martin Gräßlin
mgraesslin at kde.org
Thu Feb 13 07:43:35 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115715/
-----------------------------------------------------------
Review request for KDE Frameworks, kdewin and John Layt.
Repository: kprintutils
Description
-------
Remove X11 dependency from kprintutils
The availability of XLib was used to test whether Cups is available.
Cups has nothing to do with X11, thus the check is wrong.
The code does not use any platform specific code and has a runtime
check whether cups is available. Given the comment it should also
work correct on platforms which do not have Cups (e.g. Windows).
Diffs
-----
CMakeLists.txt 28342d0c9149d09ab0b9e41c5ed6d41b695130ed
src/CMakeLists.txt 927b02480db47bb74ea4240e582dbb0f6f6aeac2
src/config-kprintutils.h.cmake 89858d17de239cfc7eed1f40a8b828803de3299c
src/kcupsoptionswidget_p.cpp c88e848a41b72590b13d8b38783cd1c7b1d106d1
src/kdeprintdialog.cpp a53e19846d0f45073f1d6827e7b2eaa2bde859a3
Diff: https://git.reviewboard.kde.org/r/115715/diff/
Testing
-------
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140213/d9131c69/attachment.html>
More information about the Kde-frameworks-devel
mailing list