Review Request 115236: Get closer to compiling KIO on windows
Alexander Richardson
arichardson.kde at googlemail.com
Thu Jan 23 14:59:30 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115236/
-----------------------------------------------------------
(Updated Jan. 23, 2014, 2:59 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks.
Repository: kio
Description
-------
3 Commits to get closer to compiling on windows:
1. Include qplatformdefs.h where possible
unistd.h and others are not available on e.g. Windows,
qplatformdefs.h includes the equivalent for each platform
2. add include(CheckLibraryExists) to CMakeLists.txt
On Linux this is apparently pulled in by some other file
whereas it is missing on Windows
3. Make KIO::MetaData completely inline to prevent linker errors on MSVC
Diffs
-----
src/ioslaves/ftp/ftp.h ad94979829a5d0e71ec57177b51f67e115c5445e
src/ioslaves/ftp/ftp.cpp 9ea642587bd754f0b4295fcb7d4db1b427f4326e
src/ioslaves/help/kio_help.h 0eab4ce1b393b48e552ca94d877f4c069450cee0
src/ioslaves/http/http.cpp 4f97b335c0e206f0a2ff8cfd515d0fae79614ad2
src/ioslaves/http/http_cache_cleaner.cpp ffa8ab9580acf554b27027617cdac06b3f7755bf
src/widgets/kdirmodel.cpp ce6e4c9aaddada5715b5aeef36ad163c77d3c635
src/widgets/kpropertiesdialog.cpp 8ddd37f0326e37109ce239a46bfa67d2f4c35411
src/widgets/krun.cpp 92dcfd8da04b9f3d80b632a37758017c088093ab
src/widgets/kurlcompletion.cpp ed77fd3213db0524b5a934c94eb7645d98bd27c7
tests/kioslavetest.cpp dc56240dc166dfa93ec099db5cdd262cc06249d4
tests/kruntest.cpp 68720ddf788e6b44468baf18dd04937094741274
CMakeLists.txt 9894ad5e1696fe31d8a3f065d29747cde0474d2c
autotests/fileundomanagertest.cpp 4ddee49eed254be6379a1302af2dd17aae28c15c
autotests/kurlcompletiontest.cpp 10048e2e15059596e446b03cedb0b302bf038cd5
src/core/chmodjob.cpp b60cb9bf3d3b1a71a795ee5db7c72253dbdf0ad2
src/core/kacl.h 3c9c88d0408c855d02eaacb34d14f45c55343eb2
src/core/kacl.cpp 201b30a02ef93a137b9a8507cabaece6926d8739
src/core/kfileitem.h 553dbf8ccf96983803d0224cf8dc9d72f0107b6a
src/core/kfileitem.cpp fdc0fc0279713887dc18ce1da8d3b00d422f6a9b
src/core/kprotocolmanager.cpp a8746a4f9a78e6a01eaed6a70e99146ff40129d2
src/core/krecentdocument.cpp ad0a97e7c1af85a94bb7483124b5035f0fca38a6
src/core/metadata.h cd62e3b009c21485537ddd5449a6c347748c3caf
src/core/metadata.cpp ad05032dff58314a305256993fcc7fe7b94751bc
src/core/slave.h 43b5cd8e8aa11da55d6f8a8d9fdf4eb3d0780d11
src/core/slave.cpp ef9b3c7a57b4f4384343c4d4296ec88add857eb6
src/core/slavebase.cpp a7ac4d5a6a87c256da9a3947b7223649927eba39
src/core/slaveinterface.h d75eb6b02374375a73e1aaa57d5c2979a3bc365f
src/core/slaveinterface.cpp faa4bd7dc1b5a733f0060f60626e8a0313bfea8a
src/core/slaveinterface_p.h 0ed4980a06f8aa8b3fc8c0717e4a88991483e98a
src/filewidgets/kfileplaceeditdialog.cpp a7c433c8baecca8082f8d80077deab68e4b0cec2
src/filewidgets/knewfilemenu.cpp dfc086b6e98b965739c062d7291bbd9139633beb
src/ioslaves/file/file.h 453298159791a78d877c4d81d2be73db073db3f2
src/ioslaves/file/file.cpp e3ede0daa8054fc33d12acb9966a707f9484cd98
src/ioslaves/file/file_win.cpp 53e0f8f133bd5c4cbbd07afa945854efffc7297c
Diff: https://git.reviewboard.kde.org/r/115236/diff/
Testing
-------
Thanks,
Alexander Richardson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140123/047a2c74/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list