Jenkins build became unstable: kdelibs_stable #1118

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

Jenkins build is back to stable : kdelibs_stable #1119

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/kdelibs_stable/1119/ ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #563

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/563/changes Changes: [notmart] new shiny icons for kickoff by alake -- [...truncated 207 lines...] Automatic moc for target calendarplugin Automatic moc for target

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #563

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/563/changes Changes: [notmart] new shiny icons for kickoff by alake -- [...truncated 218 lines...] Generating plasmaextracomponentsplugin.moc

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #564

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/564/changes Changes: [notmart] adjust to pixel grid -- [...truncated 173 lines...] * Qt5Test (required version = 5.2.0) Required for tests * KF5CoreAddons --

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #564

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/564/changes Changes: [notmart] adjust to pixel grid -- [...truncated 192 lines...] Scanning dependencies of target KF5Plasma_automoc Scanning dependencies of target

Review Request 118838: Don't require mandatory subText in tooltip

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118838/ --- Review request for KDE Frameworks and Plasma. Repository:

Re: Review Request 118838: Don't require mandatory subText in tooltip

2014-06-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118838/#review60511 --- yeah, the code was wrong, just a small issue on the patch

Re: Review Request 118838: Don't require mandatory subText in tooltip

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118838/ --- (Updated June 19, 2014, 1:07 p.m.) Review request for KDE Frameworks and

Re: Review Request 118838: Don't require mandatory subText in tooltip

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

Re: Review Request 118838: Don't require mandatory subText in tooltip

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118838/ --- (Updated June 19, 2014, 1:11 p.m.) Status -- This change has been

Re: Review Request 118838: Don't require mandatory subText in tooltip

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

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #565

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/565/changes Changes: [kde] Don't require mandatory subText in tooltip -- [...truncated 207 lines...] Automatic moc for target plasmacomponentsplugin Automatic moc

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #565

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/565/changes Changes: [kde] Don't require mandatory subText in tooltip -- [...truncated 200 lines...] Scanning dependencies of target plasmacomponentsplugin_automoc

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #566

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/566/changes Changes: [kde] Replace incorrect documentation -- [...truncated 213 lines...] [ 4%] Automatic moc for target kded_platformstatus Generating

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #566

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/566/changes Changes: [kde] Replace incorrect documentation -- [...truncated 198 lines...] Scanning dependencies of target plasmacomponentsplugin_automoc Scanning

Re: Review Request 118775: Make KFileItem a Q_MOVABLE type

2014-06-19 Thread Frank Reininghaus
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118775/ --- (Updated June 19, 2014, 2:47 p.m.) Review request for KDE Frameworks and

Build failed in Jenkins: ktexteditor_master_qt5 #423

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/ktexteditor_master_qt5/423/changes Changes: [Kevin Funk] Marks: Fix appearance -- [...truncated 806 lines...] Scanning dependencies of target katetextbuffertest_automoc [ 4%] [ 5%] Scanning dependencies of target

Build failed in Jenkins: ktexteditor_master_qt5 #424

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/ktexteditor_master_qt5/424/ -- [...truncated 764 lines...] Generating moc_testutils.cpp Automatic moc for target katetextbuffertest Automatic moc for target kateview_test [ 5%] Built target commands_test_automoc Scanning

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #567

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/567/changes Changes: [notmart] Introduce the ColorScope class -- [...truncated 174 lines...] Required for tests * KF5CoreAddons -- Configuring done --

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #567

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/567/changes Changes: [notmart] Introduce the ColorScope class -- [...truncated 219 lines...]

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #568

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/568/changes Changes: [notmart] prune ColorRole enum -- [...truncated 216 lines...] Generating plasmaextracomponentsplugin.moc

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #568

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/568/changes Changes: [notmart] prune ColorRole enum -- [...truncated 193 lines...] Scanning dependencies of target KF5Plasma_automoc Scanning dependencies of target

KGlobalAccel Problems

2014-06-19 Thread Vishesh Handa
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 Shortcuts. Active Shortcuts are the ones currently in use, whereas the

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114283/ --- (Updated June 19, 2014, 6 p.m.) Review request for KDE Frameworks and

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread David Edmundson
On Dec. 3, 2013, 8:49 p.m., Sebastian Kügler wrote: I really like the idea. I think we should keep our Scroll* components in to ease porting, but move away from using it for our own stuff first, and then, before a first API promise, revisit if we can remove our own implementation

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

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

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114283/ --- (Updated June 19, 2014, 6:09 p.m.) Status -- This change has been

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #569

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/569/changes Changes: [notmart] configure icon for config dialogs -- [...truncated 208 lines...] Automatic moc for target KF5PlasmaQuick Generating moc_plasmapkg.cpp

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #569

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/569/changes Changes: [notmart] configure icon for config dialogs -- [...truncated 201 lines...] Scanning dependencies of target plasmapkg2_automoc Scanning

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #570

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/570/changes Changes: [kde] Replace ScrollArea with a QQuickControls ScrollView [kde] Fix typo -- [...truncated 199 lines...] Automatic moc for target KF5Plasma

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #570

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/570/changes Changes: [kde] Replace ScrollArea with a QQuickControls ScrollView [kde] Fix typo -- [...truncated 204 lines...] [ 1%] [ 1%] [ 2%] Scanning

Re: Review Request 118366: Porting keyboard module to Framework5

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

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #571

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/571/changes Changes: [notmart] position the slider bar at margins -- [...truncated 164 lines...] * KF5GuiAddons (required version = 4.100.0) * Gettext * KF5I18n

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #571

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/571/changes Changes: [notmart] position the slider bar at margins -- [...truncated 194 lines...] Scanning dependencies of target KF5PlasmaQuick_automoc Scanning

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114283/#review60561 --- Note: The property names don't line up 100% in the merge.

Building KF5

2014-06-19 Thread André Wöbbeking
Hi, 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 most of it compiled without any problem. But I had to install these files frameworks/kio/src/core/kssl/ksslsettings.h

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread Kai Uwe Broulik
On June 19, 2014, 8:42 p.m., David Edmundson wrote: Note: The property names don't line up 100% in the merge. flickableItem has now become contentItem (which is a default property) All instances in plasma-workspace/plasma-desktop are now updated. Kai Uwe Broulik wrote: That

git hooks for reviews mandatory?

2014-06-19 Thread Marco Martin
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 many times of not respecting that, mostly for

Re: Review Request 114283: Use QtQuickControls ScrollView internally in PlasmaExtraComponents.ScrollArea

2014-06-19 Thread Marco Martin
On June 19, 2014, 8:42 p.m., David Edmundson wrote: Note: The property names don't line up 100% in the merge. flickableItem has now become contentItem (which is a default property) All instances in plasma-workspace/plasma-desktop are now updated. Kai Uwe Broulik wrote: That

Review Request 118841: Add notify to colorGroup property

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118841/ --- Review request for KDE Frameworks and Plasma. Repository:

Re: git hooks for reviews mandatory?

2014-06-19 Thread Luigi Toscano
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 a REVIEW: line? ... or Reviewed by:

Re: git hooks for reviews mandatory?

2014-06-19 Thread Nicolás Alvarez
2014-06-19 18:21 GMT-03:00 Marco Martin notm...@gmail.com: 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

Re: Review Request 118841: Add notify to colorGroup property

2014-06-19 Thread Elias Probst
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118841/#review60565 --- Fixes for me these messages from battery monitor:

Re: Review Request 118841: Add notify to colorGroup property

2014-06-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118841/#review60566 --- Ship it! Ship It! - Marco Martin On June 19, 2014, 9:34

Re: Review Request 118841: Add notify to colorGroup property

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

Re: Review Request 118841: Add notify to colorGroup property

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

Build failed in Jenkins: plasma-framework_master_qt5 » NoX11,LINBUILDER #572

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=NoX11,label=LINBUILDER/572/changes Changes: [kde] Add notify property to colorGroup -- [...truncated 192 lines...] Scanning dependencies of target

Re: Review Request 118775: Make KFileItem a Q_MOVABLE type

2014-06-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118775/#review60568 --- Ship it! (again, with the agreement that the dirlister

Build failed in Jenkins: plasma-framework_master_qt5 » All,LINBUILDER #572

2014-06-19 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/Variation=All,label=LINBUILDER/572/changes Changes: [kde] Add notify property to colorGroup -- [...truncated 201 lines...] Scanning dependencies of target plasmaextracomponentsplugin_automoc

Re: Review Request 118366: Porting keyboard module to Framework5

2014-06-19 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118366/#review60569 --- kcms/keyboard/bindings.cpp

Re: Building KF5

2014-06-19 Thread Aleix Pol
On Thu, Jun 19, 2014 at 10:30 PM, André Wöbbeking woebbek...@kde.org wrote: Hi, 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 most of it compiled without any problem. But I had to install

Review Request 118842: Add reset function to QImageItem image property

2014-06-19 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118842/ --- Review request for KDE Frameworks and Plasma. Repository: kdeclarative

Re: Building KF5

2014-06-19 Thread Kevin Ottens
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 most of it compiled without any problem. Actually if it breaks with Clang it should

Re: git hooks for reviews mandatory?

2014-06-19 Thread Kevin Ottens
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 review, what about on repositories under frameworks/* adding an hook