----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115236/#review48093 -----------------------------------------------------------
Has this been compile-tested on Linux? - David Faure On Jan. 22, 2014, 9:12 p.m., Alexander Richardson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115236/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 9:12 p.m.) > > > 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel