Re: Review Request 118366: Porting keyboard module to Framework5

2014-06-20 Thread Martin Gräßlin
On May 28, 2014, 8:10 a.m., Martin Gräßlin wrote: kcms/keyboard/kcmmisc.cpp, lines 77-78 https://git.reviewboard.kde.org/r/118366/diff/2/?file=275630#file275630line77 for new connects I would use the new compile time checked syntax. shivam makkar wrote: I tried it but it was

Re: KGlobalAccel Problems

2014-06-20 Thread Martin Gräßlin
On Thursday 19 June 2014 19:26:45 Vishesh Handa wrote: Hey guys I've been looking into kglobalaccel in order to fix some problems that I was having. I seems to have opened a can of worms. Background Information: KGlobalAccel has 2 kinds of shortcuts - Active Shortcuts and Default

Re: git hooks for reviews mandatory?

2014-06-20 Thread Marco Martin
On Thursday 19 June 2014 23:38:05 Luigi Toscano wrote: Marco Martin ha scritto: Hi all, I was thinking, since the policy for committing in frameworks is to always asking for a review, what about on repositories under frameworks/* adding an hook that accepts pushes only if the comment has

Re: Review Request 118842: Add reset function to QImageItem image property

2014-06-20 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118842/#review60578 --- Ship it! Ship It! - Marco Martin On June 20, 2014, 12:26

Re: Review Request 118692: Fix reading of entries for language/country combinations

2014-06-20 Thread Martin Gräßlin
On June 18, 2014, 12:50 a.m., Matthew Dawson wrote: autotests/kdesktopfiletest.cpp, line 88 https://git.reviewboard.kde.org/r/118692/diff/2-3/?file=280487#file280487line88 Please add a test for both: - when both the country/language match and a modifier is included. -

Re: Review Request 118780: Disable the xcb flush connection workaround for Qt = 5.3.1

2014-06-20 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118780/ --- (Updated June 20, 2014, 9:03 a.m.) Status -- This change has been

Re: Review Request 118780: Disable the xcb flush connection workaround for Qt = 5.3.1

2014-06-20 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118780/#review60580 --- This review has been submitted with commit

Build failed in Jenkins: kio_master_qt5 #267

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kio_master_qt5/267/changes Changes: [scripty] SVN_SILENT made messages (.desktop file) -- [...truncated 844 lines...] [ 17%] [ 17%] Automatic moc for target kioslave-httpauthenticationtest Automatic moc for target

Build failed in Jenkins: frameworkintegration_master_qt5 #81

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/frameworkintegration_master_qt5/81/changes Changes: [mgraesslin] Disable the xcb flush connection workaround for Qt = 5.3.1 -- [...truncated 56 lines...] kiconthemes - Branch master kbookmarks - Branch master

Build failed in Jenkins: kdelibs4support_master_qt5 #224

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kdelibs4support_master_qt5/224/changes Changes: [scripty] SVN_SILENT made messages (.desktop file) -- [...truncated 540 lines...] Scanning dependencies of target kdialogbuttonboxtest_automoc [ 14%] [ 14%] [ 14%] Automatic moc

Re: Review Request 118725: use renamed .direcory files

2014-06-20 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118725/#review60583 --- This review has been submitted with commit

Re: Review Request 118725: use renamed .direcory files

2014-06-20 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118725/ --- (Updated June 20, 2014, 9:50 a.m.) Status -- This change has been

Review Request 118844: Introduce convenient methods to set active and default shortcut

2014-06-20 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118844/ --- Review request for KDE Frameworks and Vishesh Handa. Repository:

Re: Review Request 118842: Add reset function to QImageItem image property

2014-06-20 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118842/ --- (Updated June 20, 2014, 10:41 a.m.) Status -- This change has been

Re: Review Request 118842: Add reset function to QImageItem image property

2014-06-20 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118842/#review60584 --- This review has been submitted with commit

Re: git hooks for reviews mandatory?

2014-06-20 Thread Aleix Pol
On Fri, Jun 20, 2014 at 7:40 AM, Kevin Ottens er...@kde.org wrote: On Friday 20 June 2014 01:46:10 Aleix Pol wrote: On Thu, Jun 19, 2014 at 11:21 PM, Marco Martin notm...@gmail.com wrote: Hi all, I was thinking, since the policy for committing in frameworks is to always asking for a

Re: Review Request 118783: Set the normal shortcut when setting the default shortcut as well

2014-06-20 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118783/ --- (Updated June 20, 2014, 11:18 a.m.) Status -- This change has been

Re: Review Request 118844: Introduce convenient methods to set active and default shortcut

2014-06-20 Thread Martin Gräßlin
On June 20, 2014, 1:17 p.m., Vishesh Handa wrote: I approve. The only problem I have is with the documentation, but we can improve that in another patch. well let's improve directly. What do you think is missing with these new methods or is that more a general comment? - Martin

Jenkins build became unstable: kwindowsystem_master_qt5 » All,LINBUILDER #73

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kwindowsystem_master_qt5/Variation=All,label=LINBUILDER/73/ ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: kde-baseapps_frameworks_qt5 #29

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kde-baseapps_frameworks_qt5/29/changes Changes: [scarlett] kde-baseapps frameworks dolphin with snapshot frameworks branch baloo/baloo-widgets path fixes. -- [...truncated 1142 lines...] ^

Review Request 118846: Turn KLauncher into a QGuiApplication

2014-06-20 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118846/ --- Review request for KDE Frameworks, David Faure and Harald Sitter. Bugs:

Build failed in Jenkins: kcmutils_master_qt5 #60

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kcmutils_master_qt5/60/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 4 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kcmutils_master_qt5/ws/ Running Prebuild steps [kcmutils_master_qt5] $

Build failed in Jenkins: kded_master_qt5 #63

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kded_master_qt5/63/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 4 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kded_master_qt5/ws/ Running Prebuild steps [kded_master_qt5] $ /bin/sh -xe

Build failed in Jenkins: kdesu_master_qt5 #52

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kdesu_master_qt5/52/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 4 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kdesu_master_qt5/ws/ Running Prebuild steps [kdesu_master_qt5] $ /bin/sh -xe

Build failed in Jenkins: kemoticons_master_qt5 #60

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kemoticons_master_qt5/60/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 4 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kemoticons_master_qt5/ws/ Running Prebuild steps [kemoticons_master_qt5]

Build failed in Jenkins: kdewebkit_master_qt5 #43

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kdewebkit_master_qt5/43/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 1 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kdewebkit_master_qt5/ws/ Running Prebuild steps [kdewebkit_master_qt5] $

Build failed in Jenkins: kio_master_qt5 #268

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kio_master_qt5/268/changes Changes: [faure] Use QDBusConnection instead of KDBusConnectionPool. [adawit] Do not send config reparse notification to suspended ioslaves. [scripty] SVN_SILENT made messages (.desktop file) --

Build failed in Jenkins: knotifications_master_qt5 #83

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/knotifications_master_qt5/83/ -- [...truncated 4 lines...] + /home/jenkins/scripts/setup-env.sh Preparing to perform KDE Continuous Integration build == Setting Up Sources From git://anongit.kde.org/knotifications

Build failed in Jenkins: kparts_master_qt5 #75

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/kparts_master_qt5/75/ -- Started by user Ben Cooksley Building remotely on LinuxSlave - 4 (PACKAGER LINBUILDER) in workspace http://build.kde.org/job/kparts_master_qt5/ws/ Running Prebuild steps [kparts_master_qt5] $ /bin/sh

Re: Review Request 118846: Turn KLauncher into a QGuiApplication

2014-06-20 Thread Harald Sitter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118846/#review60591 --- Ship it! looks good to me - Harald Sitter On June 20,

Jenkins build became unstable: ktexteditor_master_qt5 #425

2014-06-20 Thread KDE CI System
See http://build.kde.org/job/ktexteditor_master_qt5/425/changes ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 118844: Introduce convenient methods to set active and default shortcut

2014-06-20 Thread Vishesh Handa
On June 20, 2014, 11:17 a.m., Vishesh Handa wrote: I approve. The only problem I have is with the documentation, but we can improve that in another patch. Martin Gräßlin wrote: well let's improve directly. What do you think is missing with these new methods or is that more a

Re: Review Request 118692: Fix reading of entries for language/country combinations

2014-06-20 Thread Matthew Dawson
On June 17, 2014, 6:50 p.m., Matthew Dawson wrote: autotests/kdesktopfiletest.cpp, line 88 https://git.reviewboard.kde.org/r/118692/diff/2-3/?file=280487#file280487line88 Please add a test for both: - when both the country/language match and a modifier is included. -

Re: Review Request 118846: Turn KLauncher into a QGuiApplication

2014-06-20 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118846/#review60615 --- Strange, why wasn't this necessary in kdelibs4? - David

Re: KCoreAddons does not install most of its headers on my system

2014-06-20 Thread Alex Merry
On 19/06/14 00:32, Aleix Pol wrote: On Wed, Jun 18, 2014 at 10:56 PM, Ben Cooksley bcooks...@kde.org mailto:bcooks...@kde.org wrote: Note though that I consider this a workaround for an upstream regression which we have detected. One of the reasons the CI system runs the next

Re: git hooks for reviews mandatory?

2014-06-20 Thread Alex Merry
On 19/06/14 22:21, Marco Martin wrote: Hi all, I was thinking, since the policy for committing in frameworks is to always asking for a review, what about on repositories under frameworks/* adding an hook that accepts pushes only if the comment has a REVIEW: line? I have been guilty too

Re: git hooks for reviews mandatory?

2014-06-20 Thread David Faure
On Friday 20 June 2014 19:25:19 Alex Merry wrote: On 19/06/14 22:21, Marco Martin wrote: Hi all, I was thinking, since the policy for committing in frameworks is to always asking for a review, what about on repositories under frameworks/* adding an hook that accepts pushes only if the

Re: Review Request 118692: Fix reading of entries for language/country combinations

2014-06-20 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118692/#review60621 --- Ship it! Ship It! - Matthew Dawson On June 20, 2014, 4:56

Re: Review Request 118039: Simplify KConfig::readEntry/writeEntry

2014-06-20 Thread Nicolás Alvarez
2014-05-07 14:15 GMT-03:00 Thomas Braxton kde.brax...@gmail.com: This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118039/ Review request for KDE Frameworks and kdewin. By Thomas Braxton. Summary Simplify KConfig::readEntry/writeEntry

Re: git hooks for reviews mandatory?

2014-06-20 Thread Albert Astals Cid
El Divendres, 20 de juny de 2014, a les 01:46:10, Aleix Pol va escriure: On Thu, Jun 19, 2014 at 11:21 PM, Marco Martin notm...@gmail.com wrote: Hi all, I was thinking, since the policy for committing in frameworks is to always asking for a review, what about on repositories under

Re: Building KF5

2014-06-20 Thread Albert Astals Cid
El Divendres, 20 de juny de 2014, a les 07:33:34, Kevin Ottens va escriure: Hello, On Thursday 19 June 2014 22:30:26 André Wöbbeking wrote: I wanted to give KF5 a try and followed the instructions from http://community.kde.org/Frameworks/Building. After Albert told me not to use Clang

Re: Review Request 118846: Turn KLauncher into a QGuiApplication

2014-06-20 Thread Martin Gräßlin
On June 20, 2014, 7:05 p.m., David Faure wrote: Strange, why wasn't this necessary in kdelibs4? see the linked bug report. It's basically a regression due to introducing platform detection (X11 vs. Wayland) in both klauncher and KWindowSystem. My first idea to fix it was to check for

Re: Review Request 118366: Porting keyboard module to Framework5

2014-06-20 Thread shivam makkar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118366/ --- (Updated June 20, 2014, 9:24 p.m.) Review request for kde-workspace, KDE

Re: Review Request 118366: Porting keyboard module to Framework5

2014-06-20 Thread shivam makkar
On May 28, 2014, 6:10 a.m., Martin Gräßlin wrote: kcms/keyboard/kcmmisc.cpp, lines 77-78 https://git.reviewboard.kde.org/r/118366/diff/2/?file=275630#file275630line77 for new connects I would use the new compile time checked syntax. shivam makkar wrote: I tried it but it was

Re: Review Request 118366: Porting keyboard module to Framework5

2014-06-20 Thread shivam makkar
On June 19, 2014, 11:43 p.m., Aleix Pol Gonzalez wrote: kcms/keyboard/flags.cpp, line 21 https://git.reviewboard.kde.org/r/118366/diff/4/?file=276646#file276646line21 remove? see the latest revision, apparently the diff is showing all the revisions made to the same file. On

Re: Review Request 118846: Turn KLauncher into a QGuiApplication

2014-06-20 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118846/#review60630 --- Ship it! Ship It! - Aleix Pol Gonzalez On June 20, 2014,

Review Request 118852: [kconfig] Fix build on Visual C++ 2010

2014-06-20 Thread Nicolás Alvarez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118852/ --- Review request for KDE Frameworks and kdewin. Repository: kconfig

Review Request 118853: Log name + size of not found icons in debug output

2014-06-20 Thread Elias Probst
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118853/ --- Review request for KDE Frameworks and Christoph Feck. Repository:

Re: Review Request 118692: Fix reading of entries for language/country combinations

2014-06-20 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118692/#review60635 --- This review has been submitted with commit