[Differential] [Changed Subscribers] D4431: Add KUrlRequester::setMimeTypeFilters.

2017-02-04 Thread Christoph Feck
cfeck added inline comments. INLINE COMMENTS > kurlrequester.h:144 > /** > + * Returns the mimetype filters for the file dialog. > + * @see QFileDialog::setMimeTypeFilters() The APIDOX for these two functions are somehow swapped. REPOSITORY R241 KIO REVISION DETAIL

Jenkins-kde-ci: karchive master stable-kf5-qt5 » Linux,gcc - Build # 145 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/karchive%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/145/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 05 Feb 2017 00:51:13 + Build duration: 5 min 0 sec CHANGE SET Revision

Jenkins-kde-ci: karchive master stable-kf5-qt5 » Linux,gcc - Build # 145 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/karchive%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/145/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 05 Feb 2017 00:51:13 + Build duration: 5 min 0 sec CHANGE SET Revision

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 147 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/147/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 05 Feb 2017 00:51:13 + Build duration: 4 min 8 sec CHANGE SET Revision

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 147 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/147/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sun, 05 Feb 2017 00:51:13 + Build duration: 4 min 8 sec CHANGE SET Revision

[Differential] [Closed] D4437: Fix my fix for KCompressionDevice::seek

2017-02-04 Thread Albert Astals Cid
This revision was automatically updated to reflect the committed changes. Closed by commit R243:2225527bb6be: Fix my fix for KCompressionDevice::seek (authored by aacid). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D4437?vs=10921=10922#toc REPOSITORY R243 KArchive CHANGES SINCE

[Differential] [Accepted] D4437: Fix my fix for KCompressionDevice::seek

2017-02-04 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Ah yes of course, this is what I meant. TV distracted me :) REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D4437 EMAIL PREFERENCES

[Differential] [Updated, 15 lines] D4437: Fix my fix for KCompressionDevice::seek

2017-02-04 Thread Albert Astals Cid
aacid updated this revision to Diff 10921. aacid added a comment. We can save some seek calls, but we still need it in the if, there's nothing that guaranteers that "QIODevice::pos" will be d->deviceReadPos so we need to seek to d->deviceReadPos. REPOSITORY R243 KArchive CHANGES SINCE

[Differential] [Commented On] D4437: Fix my fix for KCompressionDevice::seek

2017-02-04 Thread David Faure
dfaure added inline comments. INLINE COMMENTS > kcompressiondevice.cpp:208 > +// asked us to be in > +if (!QIODevice::seek(d->deviceReadPos)) { > +return false; I wonder if we could avoid seeking twice, by doing seek(pos) only in the other cases (the deviceReadPos

[Differential] [Request, 7 lines] D4437: Fix my fix for KCompressionDevice::seek

2017-02-04 Thread Albert Astals Cid
aacid created this revision. aacid added a reviewer: dfaure. aacid set the repository for this revision to R243 KArchive. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REPOSITORY R243 KArchive REVISION DETAIL

[Differential] [Request, 4 lines] D4436: [Button Styles] Use Layout.fillHeight instead of parent.height in a Layout

2017-02-04 Thread Kai Uwe Broulik
broulik created this revision. broulik added a reviewer: Plasma. broulik set the repository for this revision to R242 Plasma Framework (Library). Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY This

Review Request 129921: [kio-extras] thumbnails should be a clean image representation without "hard-coded" borders or design elements

2017-02-04 Thread Diego S.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129921/ --- Review request for KDE Frameworks. Repository: kio-extras Description

Jenkins-kde-ci: kio master kf5-qt5 » Linux,gcc - Build # 414 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/414/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 20:54:22 + Build duration: 5 min 28 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kio master kf5-qt5 » Linux,gcc - Build # 414 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/414/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 20:54:22 + Build duration: 5 min 28 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 418 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/418/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:39:56 + Build duration: 19 min CHANGE SET Revision 4f8e9f45c600ea6f246fcf3b84a7adc7792276bc

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 418 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/418/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:39:56 + Build duration: 19 min CHANGE SET Revision 4f8e9f45c600ea6f246fcf3b84a7adc7792276bc

Jenkins-kde-ci: kguiaddons master kf5-qt5 » Linux,gcc - Build # 134 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kguiaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/134/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:52:39 + Build duration: 55 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kguiaddons master kf5-qt5 » Linux,gcc - Build # 134 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kguiaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/134/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:52:39 + Build duration: 55 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: kio master kf5-qt5 » Linux,gcc - Build # 413 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kio%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/413/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:47:41 + Build duration: 7 min 23 sec CHANGE SET Revision 4f8e9f45c600ea6f246fcf3b84a7adc7792276bc

Re: CI trouble

2017-02-04 Thread Ben Cooksley
On Sun, Feb 5, 2017 at 8:45 AM, Nicolás Alvarez wrote: > 2017-02-04 16:19 GMT-03:00 David Faure : >> Today the CI seems capricious ;) >> >> - build.kde.org was down for a bit > > Yep, Jenkins ran out of memory in the JVM. Restarted a while ago. > >> - DNS

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 238 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/238/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:24:40 + Build duration: 1 min 41 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 238 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/238/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 19:24:40 + Build duration: 1 min 41 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: kdesignerplugin master kf5-qt5 » Linux,gcc - Build # 149 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdesignerplugin%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/149/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:54:33 + Build duration: 5 min 29 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kdesignerplugin master kf5-qt5 » Linux,gcc - Build # 149 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdesignerplugin%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/149/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:54:33 + Build duration: 5 min 29 sec CHANGE SET No changes JUNIT RESULTS

Re: CI trouble

2017-02-04 Thread David Faure
On samedi 4 février 2017 20:27:59 CET Albert Astals Cid wrote: > El dissabte, 4 de febrer de 2017, a les 20:19:53 CET, David Faure va escriure: > > Today the CI seems capricious ;) > > Seems my karchive patch broke another karchive test, that for some reason i > had not seen failing before.

Re: Review Request 129839: KFileMetaData: add a new property DiscNumber for audio files from multi-disc albums

2017-02-04 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129839/#review102387 --- No need to move this (yet).

Re: CI trouble

2017-02-04 Thread Nicolás Alvarez
2017-02-04 16:19 GMT-03:00 David Faure : > Today the CI seems capricious ;) > > - build.kde.org was down for a bit Yep, Jenkins ran out of memory in the JVM. Restarted a while ago. > - DNS problems in >

[Differential] [Accepted] D4247: KIconEngine: Center icon in requested rect

2017-02-04 Thread David Faure
dfaure accepted this revision. dfaure added a reviewer: dfaure. This revision is now accepted and ready to land. REPOSITORY R302 KIconThemes BRANCH master REVISION DETAIL https://phabricator.kde.org/D4247 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 417 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/417/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:44:14 + Build duration: 7 min 55 sec CHANGE SET Revision

Re: CI trouble

2017-02-04 Thread Albert Astals Cid
El dissabte, 4 de febrer de 2017, a les 20:19:53 CET, David Faure va escriure: > Today the CI seems capricious ;) Seems my karchive patch broke another karchive test, that for some reason i had not seen failing before. If you're going to do the release *now* you can revert mine temporarily or

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 237 - Still Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/237/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:41:37 + Build duration: 1 min 57 sec CHANGE SET Revision

CI trouble

2017-02-04 Thread David Faure
Today the CI seems capricious ;) - build.kde.org was down for a bit - DNS problems in https://build.kde.org/view/Frameworks%20kf5-qt5/job/kparts%20master%20kf5-qt5/409/console - "QXcbConnection: Could not connect to display :99" problems in many cases like

[Differential] [Closed] D4431: Add KUrlRequester::setMimeTypeFilters.

2017-02-04 Thread David Faure
This revision was automatically updated to reflect the committed changes. Closed by commit R241:4f8e9f45c600: Add KUrlRequester::setMimeTypeFilters. (authored by dfaure). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D4431?vs=10912=10914#toc REPOSITORY R241 KIO CHANGES SINCE LAST

Jenkins-kde-ci: karchive master stable-kf5-qt5 » Linux,gcc - Build # 144 - Still Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/karchive%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/144/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:38:41 + Build duration: 57 sec CHANGE SET No changes JUNIT RESULTS Name:

Re: Differential commit log keyword is uncomfortable - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Elvis Angelaccio
On Sat, Feb 4, 2017 at 7:28 PM, Ben Cooksley wrote: > On Sun, Feb 5, 2017 at 6:30 AM, Albert Astals Cid wrote: >> El diumenge, 29 de gener de 2017, a les 8:32:21 CET, Ben Cooksley va >> escriure: >>> Hi everyone, >>> >>> We've just completed the registration of

Jenkins-kde-ci: kwindowsystem master kf5-qt5 » Linux,All,gcc - Build # 150 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kwindowsystem%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/150/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Sat, 04 Feb 2017 18:34:41 + Build duration: 4 min 13 sec CHANGE SET Revision

Jenkins-kde-ci: kwindowsystem master stable-kf5-qt5 » Linux,All,gcc - Build # 148 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kwindowsystem%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/148/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Sat, 04 Feb 2017 18:34:41 + Build duration: 4 min 11 sec CHANGE SET

Jenkins-kde-ci: kitemviews master kf5-qt5 » Linux,gcc - Build # 129 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kitemviews%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/129/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:22:41 + Build duration: 4 min 2 sec CHANGE SET Revision

[Differential] [Accepted] D4405: [KFilePreviewGenerator] Add ignoreMaximumSize

2017-02-04 Thread David Faure
dfaure accepted this revision. This revision is now accepted and ready to land. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D4405 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, emmanuelp, hein, dfaure Cc: #frameworks

[Differential] [Accepted] D4431: Add KUrlRequester::setMimeTypeFilters.

2017-02-04 Thread Mark Gaiser
markg accepted this revision. markg added a reviewer: markg. markg added a comment. This revision is now accepted and ready to land. Looks good to me. REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D4431 EMAIL PREFERENCES

Jenkins-kde-ci: kguiaddons master kf5-qt5 » Linux,gcc - Build # 133 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kguiaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/133/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:17:46 + Build duration: 52 sec CHANGE SET Revision

Jenkins-kde-ci: kdesignerplugin master kf5-qt5 » Linux,gcc - Build # 148 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kdesignerplugin%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/148/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:13:24 + Build duration: 1 min 6 sec CHANGE SET Revision

Re: Differential commit log keyword is uncomfortable - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Ben Cooksley
On Sun, Feb 5, 2017 at 6:30 AM, Albert Astals Cid wrote: > El diumenge, 29 de gener de 2017, a les 8:32:21 CET, Ben Cooksley va escriure: >> Hi everyone, >> >> We've just completed the registration of all mainline repositories >> (not including Websites or Sysadmin namespaced ones)

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 146 - Still unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/146/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:05:37 + Build duration: 1 min 13 sec CHANGE SET Revision

Jenkins-kde-ci: karchive master stable-kf5-qt5 » Linux,gcc - Build # 143 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/karchive%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/143/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 18:05:37 + Build duration: 1 min 25 sec CHANGE SET Revision

[Differential] [Request, 34 lines] D4431: Add KUrlRequester::setMimeTypeFilters.

2017-02-04 Thread David Faure
dfaure created this revision. dfaure added a reviewer: mlaurent. dfaure added a subscriber: Frameworks. Restricted Application added a project: Frameworks. REVISION SUMMARY Otherwise people need to call the deprecated fileDialog() method to set it, which creates the file dialog too early.

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread David Faure
On samedi 4 février 2017 10:46:41 CET Stephen Kelly wrote: > On 02/04/2017 08:14 AM, Matthew Dawson wrote: > > On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: > >> On 1 February 2017 at 14:34, David Faure wrote: > >>> One note though: this is a failure to link

Re: Differential commit log keyword is uncomfortable - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Kai Uwe Broulik
> Does anyone have love for the current syntax? I have a khotkeys macro for that... +1 for DIFFERENTIAL. It also seems that release scripts don't extract Phabricator links for inclusion in the changelog (it does that with REVIEW) Cheers, Kai Uwe

[Differential] [Commented On] D4422: Fix KCompressionDevice to work with Qt >= 5.7

2017-02-04 Thread David Faure
dfaure added a comment. Yeah using qiodevice_p.h is out of the question ;) For info the commit log syntax for autoclosing is Differential Revision: https://phabricator.kde.org/D4422 REPOSITORY R243 KArchive REVISION DETAIL https://phabricator.kde.org/D4422 EMAIL PREFERENCES

Differential commit log keyword is uncomfortable - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Albert Astals Cid
El diumenge, 29 de gener de 2017, a les 8:32:21 CET, Ben Cooksley va escriure: > Hi everyone, > > We've just completed the registration of all mainline repositories > (not including Websites or Sysadmin namespaced ones) on Phabricator. > Thanks go to Luigi Toscano for providing significant

Re: Phabricator differential is not good - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Albert Astals Cid
El dissabte, 4 de febrer de 2017, a les 12:44:54 CET, Ben Cooksley va escriure: > On Sat, Feb 4, 2017 at 11:41 AM, Albert Astals Cid wrote: > > El divendres, 3 de febrer de 2017, a les 21:06:08 CET, Ben Cooksley va > > > > escriure: > >> On Fri, Feb 3, 2017 at 12:18 PM, Albert

[Differential] [Commented On] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-04 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D4416#83019, @mck182 wrote: > In https://phabricator.kde.org/D4416#82951, @hein wrote: > > > Gnome-only spec? Why aren't they contributing to fd.o? > > > "Galago is dead, we took over, deal with it." > >

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 145 - Failure!

2017-02-04 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/145/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 16:03:34 + Build duration: 2 min 38 sec CHANGE SET Revision

[Differential] [Commented On] D4422: Fix KCompressionDevice to work with Qt >= 5.7

2017-02-04 Thread Albert Astals Cid
aacid added a comment. In https://phabricator.kde.org/D4422#82977, @dfaure wrote: > So your patch adds a member variable which duplicates QIODevice::d->devicePos, but there's no other solution without adding more API to QIODevice. > > I deem QIODevice API incomplete, and your patch

[Differential] [Closed] D4422: Fix KCompressionDevice to work with Qt >= 5.7

2017-02-04 Thread Albert Astals Cid
aacid closed this revision. aacid added a comment. I commited it, but instructions at https://community.kde.org/Policies/Commit_Policy#Special_keywords_in_GIT_and_SVN_log_messages where unclear and i failed to do it correctly

[Differential] [Commented On] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-04 Thread Martin Klapetek
mck182 added a comment. In https://phabricator.kde.org/D4416#82951, @hein wrote: > Gnome-only spec? Why aren't they contributing to fd.o? "Galago is dead, we took over, deal with it." https://bugzilla.gnome.org/show_bug.cgi?id=745634#c25 REPOSITORY R289 KNotifications

Jenkins-kde-ci: kglobalaccel master stable-kf5-qt5 » Linux,gcc - Build # 225 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kglobalaccel%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/225/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 14:11:53 + Build duration: 5 min 40 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kglobalaccel master stable-kf5-qt5 » Linux,gcc - Build # 225 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kglobalaccel%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/225/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 14:11:53 + Build duration: 5 min 40 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kservice master stable-kf5-qt5 » Linux,gcc - Build # 230 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/230/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 13:46:33 + Build duration: 1 min 46 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kservice master stable-kf5-qt5 » Linux,gcc - Build # 230 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/230/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 13:46:33 + Build duration: 1 min 46 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: kglobalaccel master stable-kf5-qt5 » Linux,gcc - Build # 224 - Failure!

2017-02-04 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/kglobalaccel%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/224/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 13:46:33 + Build duration: 6.4 sec CHANGE SET No changes

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 235 - Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/235/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 13:36:35 + Build duration: 5 min 36 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: kservice master stable-kf5-qt5 » Linux,gcc - Build # 229 - Still Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kservice%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/229/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 12:25:16 + Build duration: 5 min 10 sec CHANGE SET No changes JUNIT RESULTS

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread David Faure
On samedi 4 février 2017 10:46:41 CET Stephen Kelly wrote: > On 02/04/2017 08:14 AM, Matthew Dawson wrote: > > On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: > >> On 1 February 2017 at 14:34, David Faure wrote: > >>> One note though: this is a failure to link

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread Stephen Kelly
On 02/04/2017 12:49 PM, David Faure wrote: >> @David Faure, when do you plan on tagging this change? For now I committed >> a change to the auto test so it will at least build on Windows >> (b939b48f8d5e5eaf9a51a7e9bda2ad8cedca27d9) which should be included. Would >> there be time to remove

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread David Faure
On samedi 4 février 2017 09:14:08 CET Matthew Dawson wrote: > On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: > > On 1 February 2017 at 14:34, David Faure wrote: > > > One note though: this is a failure to link a unittest, your release > > > isn't > > >

[Differential] [Accepted] D4422: Fix KCompressionDevice to work with Qt >= 5.7

2017-02-04 Thread David Faure
dfaure accepted this revision. dfaure added a reviewer: dfaure. dfaure added a comment. This revision is now accepted and ready to land. Thanks for the fix. I had difficulties to understand "Don't use QIODevice:pos to track our pos, doesn't work.", so I investigated a bit and indeed

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread Luigi Toscano
Stephen Kelly ha scritto: > On 02/04/2017 08:14 AM, Matthew Dawson wrote: >> On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: >>> On 1 February 2017 at 14:34, David Faure wrote: One note though: this is a failure to link a unittest, your release isn't

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 234 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/234/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 12:24:45 + Build duration: 1 min 50 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: kservice master kf5-qt5 » Linux,gcc - Build # 234 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kservice%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/234/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 12:24:45 + Build duration: 1 min 50 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 144 - Still Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/144/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 12:24:59 + Build duration: 58 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: bluez-qt master stable-kf5-qt5 » Linux,gcc - Build # 128 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/bluez-qt%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/128/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 11:49:06 + Build duration: 1 min 47 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: bluez-qt master stable-kf5-qt5 » Linux,gcc - Build # 128 - Fixed!

2017-02-04 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/bluez-qt%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/128/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 11:49:06 + Build duration: 1 min 47 sec CHANGE SET No changes JUNIT RESULTS

Jenkins-kde-ci: karchive master kf5-qt5 » Linux,gcc - Build # 143 - Still Unstable!

2017-02-04 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/karchive%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/143/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 04 Feb 2017 11:47:31 + Build duration: 3 min 28 sec CHANGE SET No changes JUNIT RESULTS Name:

Re: Review Request 120096: replace static arrays with dynamic size with QVector

2017-02-04 Thread Patrick Spendrin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120096/ --- (Updated Feb. 4, 2017, 11:52 a.m.) Status -- This change has been

Re: Review Request 122795: Fix crashes when attempting to load invalid package

2017-02-04 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122795/#review102371 --- Patch does not apply anymore. - Albert Astals Cid On

Re: Review Request 123031: Let Kross be useable w/o searching for private deps

2017-02-04 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123031/#review102369 --- This patch does not apply anymore. - Albert Astals Cid

[Differential] [Commented On] D4416: Send desktopfilename as part of notifyByPopup hints

2017-02-04 Thread Aleix Pol Gonzalez
apol added a comment. In https://phabricator.kde.org/D4416#82951, @hein wrote: > Gnome-only spec? Why aren't they contributing to fd.o? That's most certainly not the best forum to discuss this. REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread Jasem Mutlaq
This is from GIT master. Problem still exists, but I could try disabling unit tests and continue. I hope it gets fixed anyway soon so that other users wouldn't run into the same problem. On Fri, Feb 3, 2017 at 12:08 AM, Jaroslaw Staniek wrote: > > > On 1 February 2017 at 14:34,

[Differential] [Changed Subscribers] D4425: Add support for flatpak portals

2017-02-04 Thread Kai Uwe Broulik
broulik added inline comments. INLINE COMMENTS > knotificationmanager.cpp:86 > +if (!runtimeDir.isEmpty()) { > +d->inSandbox = QFile::exists(runtimeDir + > QLatin1String("/flatpak-info")); > +} `QFileInfo::exists` supposedly is faster > knotificationmanager.cpp:91 > +//

[Differential] [Commented On] D4425: Add support for flatpak portals

2017-02-04 Thread Aleix Pol Gonzalez
apol added a comment. Some coding style suggestions. General +1. INLINE COMMENTS > knotificationmanager.cpp:85 > +const QString runtimeDir = qgetenv("XDG_RUNTIME_DIR"); > +if (!runtimeDir.isEmpty()) { > +d->inSandbox = QFile::exists(runtimeDir + >

Re: Phabricator differential is not good - WAS - Re: Phabricator: All repositories registered - upcoming workflow changes

2017-02-04 Thread Ben Cooksley
On Sat, Feb 4, 2017 at 1:15 PM, Kai Uwe Broulik wrote: >> This is not ok, the web interface for reviewboard was as good as rb-tools (I >> guess tbh i never used them) and "forcing" the use of a weird tool noone has >> heard of is not a good way to attract new

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread Stephen Kelly
On 02/04/2017 08:14 AM, Matthew Dawson wrote: > On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: >> On 1 February 2017 at 14:34, David Faure wrote: >>> One note though: this is a failure to link a unittest, your release isn't >>> blocked, you can just disable

Re: KConfig issues prevent compiling KDE applications under Windows

2017-02-04 Thread Matthew Dawson
On Thursday, February 2, 2017 10:08:53 PM EST Jaroslaw Staniek wrote: > On 1 February 2017 at 14:34, David Faure wrote: > > One note though: this is a failure to link a unittest, your release isn't > > blocked, you can just disable the building of unittests in kconfig. > > > > The