Review Request 115236: Get closer to compiling KIO on windows

Alexander Richardson arichardson.kde at googlemail.com
Wed Jan 22 21:12:27 UTC 2014


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

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/20140122/78272582/attachment.html>


More information about the Kde-frameworks-devel mailing list