Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review57086 --- This review has been submitted with commit 728bb1146db187e9c59e61114a2bbb496fc57f62 by Alex Richardson to branch master. - Commit Hook On May 1, 2014, 10:51 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated May 1, 2014, 10:51 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review57085 --- This review has been submitted with commit b2c25711e3a1f46a5ac181d3a236e621153ca397 by Alex Richardson to branch master. - Commit Hook On April 4, 2014, 7:20 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 7:20 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated May 1, 2014, 10:51 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review56323 --- Ship it! A few comments, but none of them is a blocker. Well, the last one should be fixed, but you can push after fixing it. Nice work, thanks. autotests/kfileitemtest.cpp https://git.reviewboard.kde.org/r/117078/#comment39371 Could be replaced with a Windows-specific test for KFileItem::isReadable, though. src/core/kioglobal_p_win.cpp https://git.reviewboard.kde.org/r/117078/#comment39372 So this won't compile on XP at all? src/core/kioglobal_p_win.cpp https://git.reviewboard.kde.org/r/117078/#comment39373 Nice implementation :-) Quite a corner case though. Even on Unix this is almost never called: it requires running an app as root in the first place. src/ioslaves/file/file.cpp https://git.reviewboard.kde.org/r/117078/#comment39374 Should be QFile::decodeName() rather than QString::fromLocal8Bit(). - David Faure On April 4, 2014, 7:20 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 7:20 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
On March 28, 2014, 4:19 p.m., David Faure wrote: src/core/kioglobal_p.h, line 93 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257279#file257279line93 How is this different from QFileInfo::symLinkTarget()? Why not just port to that? From a quick look through the Qt sources it looks like it does the same thing. On March 28, 2014, 4:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 83 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line83 Why not QFileInfo::isSymLink()? This only works for .lnk files, for real symbolic links it returns false. Guess that needs some more work in Qt. On March 28, 2014, 4:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 61 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line61 Bonus points for contributing this to Qt... There is QFile::link(), but that only creates .lnk files (and doesn't add the .lnk suffix automatically, so they usually don't work). I guess I could let it create a symbolic link first and if that fails fall back to the .lnk files. Same questiong here as with isProcessAlive() On March 28, 2014, 4:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 27 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line27 Bonus points for contributing a static method to QProcess... Makes sense, will try to contribute that. Can it be added here or is that a problem for contributing it to Qt? - Alexander --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54455 --- On April 4, 2014, 9:20 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 9:20 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp
Re: Review Request 117078: Allow compiling kio on windows
On March 28, 2014, 3:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 27 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line27 Bonus points for contributing a static method to QProcess... Alexander Richardson wrote: Makes sense, will try to contribute that. Can it be added here or is that a problem for contributing it to Qt? You can contribute the same code to both places. What can't be done is someone taking your KDE code and trying to submit it to Qt. But if it's your own, you can relicense it as many times as you want. So make sure it's not too buggy, otherwise if people fix non-trivial bugs in the KDE version, you won't be able to submit the result to Qt :-) On March 28, 2014, 3:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 83 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line83 Why not QFileInfo::isSymLink()? Alexander Richardson wrote: This only works for .lnk files, for real symbolic links it returns false. Guess that needs some more work in Qt. Fixing Qt sounds good :) - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54455 --- On April 4, 2014, 7:20 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 7:20 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org
Re: Review Request 117078: Allow compiling kio on windows
On March 28, 2014, 4:19 p.m., David Faure wrote: src/core/kioglobal_p_win.cpp, line 27 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257281#file257281line27 Bonus points for contributing a static method to QProcess... Alexander Richardson wrote: Makes sense, will try to contribute that. Can it be added here or is that a problem for contributing it to Qt? David Faure wrote: You can contribute the same code to both places. What can't be done is someone taking your KDE code and trying to submit it to Qt. But if it's your own, you can relicense it as many times as you want. So make sure it's not too buggy, otherwise if people fix non-trivial bugs in the KDE version, you won't be able to submit the result to Qt :-) Okay good, I'll add it here first, add some tests and then try to get it into Qt - Alexander --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54455 --- On April 4, 2014, 9:20 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 9:20 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 9:01 p.m.) Review request for KDE Frameworks. Repository: kio Description (updated) --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs (updated) - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated April 4, 2014, 9:20 p.m.) Review request for KDE Frameworks. Changes --- uploaded correct diff Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs (updated) - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1551959b6a3cf7060736bea361e840f82651a332 src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp a642a524c3022ce7f039f90d5bc1f577c88631dc src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 79f6144264c03f506309037ed6e8ce429f6c30f0 src/ioslaves/http/http.cpp de1a1ddde544229689bd22cd69491a46b8c0dddb src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp 8e0a9ba0a806fdb1c9e92de00dfb1c8a1449978c src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54455 --- Thanks for this work! src/core/kioglobal_p.h https://git.reviewboard.kde.org/r/117078/#comment38097 How is this different from QFileInfo::symLinkTarget()? Why not just port to that? src/core/kioglobal_p_win.cpp https://git.reviewboard.kde.org/r/117078/#comment38100 Bonus points for contributing a static method to QProcess... src/core/kioglobal_p_win.cpp https://git.reviewboard.kde.org/r/117078/#comment38098 Bonus points for contributing this to Qt... src/core/kioglobal_p_win.cpp https://git.reviewboard.kde.org/r/117078/#comment38099 Why not QFileInfo::isSymLink()? src/widgets/kpropertiesdialog.cpp https://git.reviewboard.kde.org/r/117078/#comment38101 Or just kcom-setItems(userNames), no loop needed; src/widgets/kpropertiesdialog.cpp https://git.reviewboard.kde.org/r/117078/#comment38102 (same here) src/widgets/kpropertiesdialog.cpp https://git.reviewboard.kde.org/r/117078/#comment38103 Now that you have a QStringList, you can remove this delete after the fact. Just move the new + setItems into the if (userNames.size() maxEntries) - David Faure On March 26, 2014, 11:16 a.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 11:16 a.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp 6387d7f29df2c0a26952be5255438361c37ed082 src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 49037701772f88e9a96bc3110f1f04fccb810ca8 src/ioslaves/http/http.cpp 3158628af2dc8ee88a0ac0d968748d6b50a8da33 src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 11:50 a.m.) Review request for KDE Frameworks and kdewin. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp a3011a8adac98287bcada281dfc50b27e1af8fa0 src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 src/ioslaves/ftp/ftp.cpp 49037701772f88e9a96bc3110f1f04fccb810ca8 src/ioslaves/http/http.cpp 3158628af2dc8ee88a0ac0d968748d6b50a8da33 src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp 6387d7f29df2c0a26952be5255438361c37ed082 src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 12:16 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs (updated) - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp 6387d7f29df2c0a26952be5255438361c37ed082 src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 49037701772f88e9a96bc3110f1f04fccb810ca8 src/ioslaves/http/http.cpp 3158628af2dc8ee88a0ac0d968748d6b50a8da33 src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp a3011a8adac98287bcada281dfc50b27e1af8fa0 src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks, Alexander Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117078: Allow compiling kio on windows
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54179 --- src/widgets/kpropertiesdialog.cpp https://git.reviewboard.kde.org/r/117078/#comment37876 camelCase. I would probably just name it as the function is named: isSuperUser src/widgets/kpropertiesdialog.cpp https://git.reviewboard.kde.org/r/117078/#comment37877 If you change it anyway, you might as well go for C++11: for(const QString userName : userNames) {...} Just a matter of taste though. Feel free to ignore :) src/widgets/kurlcompletion.cpp https://git.reviewboard.kde.org/r/117078/#comment37878 Now it looks like either case (if and else) just return the users's home dir. You new code is certainly doing that. I don't know exactly what the old case was doing here. What was it doing? - Mark Gaiser On March 26, 2014, 11:16 a.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 11:16 a.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp 6387d7f29df2c0a26952be5255438361c37ed082 src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 src/ioslaves/ftp/ftp.cpp 49037701772f88e9a96bc3110f1f04fccb810ca8 src/ioslaves/http/http.cpp 3158628af2dc8ee88a0ac0d968748d6b50a8da33 src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 src/widgets/config-getgrouplist.h.cmake 6847a19d60be4eb5c2b65fb86258f7368848e6ab src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 src/widgets/kpropertiesdialog.cpp a3011a8adac98287bcada281dfc50b27e1af8fa0 src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 Diff: https://git.reviewboard.kde.org/r/117078/diff/ Testing --- compiles, tests still the same as before (i.e. not passing) Thanks,
Re: Review Request 117078: Allow compiling kio on windows
On March 26, 2014, 1:57 p.m., Mark Gaiser wrote: src/widgets/kpropertiesdialog.cpp, line 1618 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257291#file257291line1618 camelCase. I would probably just name it as the function is named: isSuperUser I just kept the old name, but you are right that doesn't fit the coding style. On March 26, 2014, 1:57 p.m., Mark Gaiser wrote: src/widgets/kpropertiesdialog.cpp, line 3444 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257291#file257291line3444 If you change it anyway, you might as well go for C++11: for(const QString userName : userNames) {...} Just a matter of taste though. Feel free to ignore :) Would also prefer that, but not possible (yet) due to minimum supported compilers: https://community.kde.org/Frameworks/Policies#Frameworks_compiler_requirements_and_C.2B.2B11 Only avaiable from GCC 4.6 and MSVC 2012. I would personally like the minimum to be that since it also allows stuff like nullptr and strongly typed enums, but those versions have been decided upon. On March 26, 2014, 1:57 p.m., Mark Gaiser wrote: src/widgets/kurlcompletion.cpp, lines 1555-1568 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257292#file257292line1555 Now it looks like either case (if and else) just return the users's home dir. You new code is certainly doing that. I don't know exactly what the old case was doing here. What was it doing? No, it looks for the homedir of a user with name = userName in the second case (same as the code before) - Alexander --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54179 --- On March 26, 2014, 12:16 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 12:16 p.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h
Re: Review Request 117078: Allow compiling kio on windows
On March 26, 2014, 12:57 p.m., Mark Gaiser wrote: src/widgets/kpropertiesdialog.cpp, line 3444 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257291#file257291line3444 If you change it anyway, you might as well go for C++11: for(const QString userName : userNames) {...} Just a matter of taste though. Feel free to ignore :) Alexander Richardson wrote: Would also prefer that, but not possible (yet) due to minimum supported compilers: https://community.kde.org/Frameworks/Policies#Frameworks_compiler_requirements_and_C.2B.2B11 Only avaiable from GCC 4.6 and MSVC 2012. I would personally like the minimum to be that since it also allows stuff like nullptr and strongly typed enums, but those versions have been decided upon. Ohh, i thought for(.. : ..) was supported just fine. It's in GCC since 4.6 http://gcc.gnu.org/projects/cxx0x.html (range based for). Don't know for MSVC 2012. On March 26, 2014, 12:57 p.m., Mark Gaiser wrote: src/widgets/kurlcompletion.cpp, lines 1555-1568 https://git.reviewboard.kde.org/r/117078/diff/2/?file=257292#file257292line1555 Now it looks like either case (if and else) just return the users's home dir. You new code is certainly doing that. I don't know exactly what the old case was doing here. What was it doing? Alexander Richardson wrote: No, it looks for the homedir of a user with name = userName in the second case (same as the code before) Ahh right, now i see it. Thank you for explaining. - Mark --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54179 --- On March 26, 2014, 11:16 a.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/ --- (Updated March 26, 2014, 11:16 a.m.) Review request for KDE Frameworks. Repository: kio Description --- A series of commits (can submit separately if necessary): --- Add S_IXUSR, IRUSR, etc. definitions for Windows --- Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() kill() does not work on Windows, therefore a WIN32 implementation is needed in order to compile slave.cpp --- Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() This allows KFileItem to be compiled on Windows. The Windows implementation requires at least Windows Vista because GetFinalPathNameByHandleW is not available in earlier version. Windows XP support ends on the 8th of April so it should be okay to drop support for it. If XP support is deemed necessary GetFinalPathNameByHandleW can be loaded dynamically using GetProcAddress but that is not part of this patch. Fix windows build of tests --- Add KIOPrivate::changeOwnership() to avoid directly calling chown() Additionally use KUserId/KGroupId instead of uid_t/gid_t. This allows compiling chmodjob.cpp on Windows. KIOPrivate::changeOwnership is a stub on Windows. However, it has always been like that with the kdewin chmod() implementation, so this is not a regression from kdelibs4. --- Add KIOPrivate::createSymlink() to avoid using symlink() directly --- Minor Windows compile fixes --- Use KUser in KPropertiesDialog This means no more need for getpwent(), etc - works on Windows --- Export the KIOPrivate functions --- Fix kio_http build on Windows --- No longer use uid_t/gid_t in kio_file --- Allow compiling kurlcompletion.cpp on windows --- Add a fake QT_LSTAT for Windows This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a symlink --- Use KUser in kpropertiesdialog.cpp - no more getgrouplist Diffs - autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d src/core/kioglobal_p.h PRE-CREATION src/core/kioglobal_p_unix.cpp PRE-CREATION src/core/kioglobal_p_win.cpp PRE-CREATION src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c src/ioslaves/file/file.cpp