D5983: Check error status after every PolKitAuthority usage

2017-05-28 Thread Albert Astals Cid
aacid added a comment. Have you had these errors happen to you? How can one reproduce them? Or is it more of a "this should be the right code but i don't know how to trigger it" case? REPOSITORY R283 KAuth REVISION DETAIL https://phabricator.kde.org/D5983 To: skalinichev Cc: a

D5975: breeze-icons: don't look for bash on Windows

2017-05-28 Thread Albert Astals Cid
aacid added a comment. You want the impossible to remember Differential Revision: https://phabricator.kde.org/D https://community.kde.org/Policies/Commit_Policy#Special_keywords_in_GIT_and_SVN_log_messages REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org

D5975: breeze-icons: don't look for bash on Windows

2017-05-28 Thread Allen Winter
winterz closed this revision. winterz added a comment. committed in https://phabricator.kde.org/R266:09291a2b3ecf03577b93c6d4cedc28927668e571 REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D5975 To: winterz, aacid Cc: aacid, #frameworks

D5856: Use KDirWatch removeDir/addDir instead of stopDirScan/restartDirScan

2017-05-28 Thread Albert Astals Cid
aacid requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D5856 To: dfaure, aacid Cc: #frameworks

D5975: breeze-icons: don't look for bash on Windows

2017-05-28 Thread Albert Astals Cid
aacid accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D5975 To: winterz, aacid Cc: aacid, #frameworks

D6002: Identify PIE binaries (application/x-sharedlib) as executable files

2017-05-28 Thread Fabian Vogt
fvogt created this revision. Restricted Application added a project: Frameworks. REVISION SUMMARY x86_64 binaries compiled with PIE are just shared objects with the executable bit set. Without this patch, kio does not know that they can be executed as well, causing "kioclient5 exec" to ask f

D5289: Import Find{Clang,LLVM} from KDevelop for Python bindings generation

2017-05-28 Thread Stephen Kelly
skelly added a comment. My preference would be to use the ClangConfig.cmake instead of introducing these new files. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D5289 To: heikobecker, #frameworks, #build_system, skelly, kfunk Cc: rdieter, shaheed, kde-

Re: Problems with Python bindings in KF5

2017-05-28 Thread Stephen Kelly
On 05/26/2017 10:21 PM, Albert Astals Cid wrote: > El dijous, 25 de maig de 2017, a les 1:02:58 CEST, Stephen Kelly va escriure: >> On 04/15/2017 10:53 PM, Luca Beltrame wrote: >>> I can't understand the error thrown by ki18n (when it does not build) or >>> I'd have attempted to fix it myself... >>

Re: Compilation failure on Windows: kpeople

2017-05-28 Thread Ralf Habacker
Am 27.05.2017 um 23:31 schrieb Ben Cooksley: > On Wed, May 17, 2017 at 7:33 AM, Ralf Habacker > wrote: >> Hi >>> When someone gets the chance, could they please look into the build >>> log at >>> https://build-sandbox.kde.org/job/Frameworks%20kpeople%20kf5-qt5%20WindowsQt5.7/4/console >>> to se

D5856: Use KDirWatch removeDir/addDir instead of stopDirScan/restartDirScan

2017-05-28 Thread David Faure
dfaure added a comment. I know what the problem is with this. Previously, if nobody was watching the directory, stopDirScan/restartDirScan would basically do nothing. Now, after running a kio job in that directory, KDirWatch will start watching it, for no purpose, due to the unconditional

D5983: Check error status after every PolKitAuthority usage

2017-05-28 Thread Sergey Kalinichev
skalinichev added inline comments. INLINE COMMENTS > aacid wrote in Polkit1Backend.cpp:220 > This seems a bit weird since we have done nothing with authority here, so how > would it have an error? Thinking a bit more about it, it can actually happen if enumerateActions call failed. REPOSITORY

D5983: Check error status after every PolKitAuthority usage

2017-05-28 Thread Sergey Kalinichev
skalinichev updated this revision to Diff 14908. REPOSITORY R283 KAuth CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5983?vs=14871&id=14908 REVISION DETAIL https://phabricator.kde.org/D5983 AFFECTED FILES src/backends/polkit-1/Polkit1Backend.cpp To: skalinichev Cc: aacid, #fra