Re: Review Request 128606: [kapidox] Read library fancyname from metainfo.yaml before parsing it from CMakeLists.txt

2016-08-04 Thread Daniel Vrátil
marked as submitted. Review request for KDE Frameworks and Olivier Churlaud. Changes --- Submitted with commit c10f62202065b4b760764882689fee00b4f35d75 by Daniel Vrátil to branch master. Repository: kapidox Description --- We have some a few repositories called "akonadi-some

Review Request 128606: [kapidox] Read library fancyname from metainfo.yaml before parsing it from CMakeLists.txt

2016-08-04 Thread Daniel Vrátil
apidox_generate - the library is listed as "Akonadi Calendar" in the Library list table 2) Ran kapidox_generate on akonadi-contacts.git without adding `fancyname` to metainfo.yaml, the library was correctly listed as "Akonadi-Contacts". Thanks, Daniel Vrátil __

Re: Review Request 128575: KRearrangeColumnsProxyModel: fix assert when running against debug build of Qt

2016-08-02 Thread Daniel Vrátil
rc/krearrangecolumnsproxymodel.cpp 72e4f14 Diff: https://git.reviewboard.kde.org/r/128575/diff/ Testing --- Triggered by Kleopatra on start, no more assert with this patch. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-fra

Re: Review Request 128578: KRearrangeColumnsProxyModel: fix assert in index(0, 0) on empty model

2016-08-02 Thread Daniel Vrátil
- > > (Updated Aug. 2, 2016, 2:39 p.m.) > > > Review request for KDE Frameworks and Daniel Vrátil. > > > Repository: kitemmodels > > > Description > --- > > KRearrangeColumnsProxyModel: fix assert in index(0, 0) on empty model > > >

Re: Review Request 128575: KRearrangeColumnsProxyModel: fix assert when running against debug build of Qt

2016-08-02 Thread Daniel Vrátil
n automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128575/#review97996 --- On Aug. 2, 2016, 2:14 p.m., Daniel Vrátil wrote: > > --- > This is an

Review Request 128575: KRearrangeColumnsProxyModel: fix assert when running against debug build of Qt

2016-08-02 Thread Daniel Vrátil
/128575/diff/ Testing --- Triggered by Kleopatra on start, no more assert with this patch. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Jenkins-kde-ci: kdbusaddons master kf5-qt5 » Linux, gcc - Build # 11 - Still Unstable!

2016-04-22 Thread Daniel Vrátil
__ > > Kde-frameworks-devel mailing list > > Kde-frameworks-devel@kde.org > > https://mail.kde.org/mailman/listinfo/kde-frameworks-devel > > _______ > Kde-frameworks-devel mailing list > Kde-frameworks-devel@kde.org > https://mail.kde.org/mailm

KService/sycoca assert in tests on CI

2016-04-22 Thread Daniel Vrátil
cker). Any ideas what's going on there? Is it a problem on the CI side, our side or maybe something in sycoca? Thanks Dan [0] https://build.kde.org/view/KDE-PIM/job/kblog %20Applications-16.04%20stable-kf5-qt5/7/PLATFORM=Linux,compiler=gcc/ testReport/junit/%28root%29/TestSuite/kblog_testblogger1/ --

Re: MimeType issues on CI

2016-04-03 Thread Daniel Vrátil
On 2016-04-03 15:28, Scarlett Clark wrote: On Sun, Apr 3, 2016 at 5:16 AM, Daniel Vrátil <dvra...@kde.org> wrote: On Sunday, April 3, 2016 12:58:24 PM CEST David Faure wrote: On Sunday 03 April 2016 12:43:27 Daniel Vrátil wrote: Turns out when you include /usr/share in XDG_DATA_DIR

Re: MimeType issues on CI (Was: Re: Delaying KF 5.21)

2016-04-03 Thread Daniel Vrátil
On Sunday, April 3, 2016 12:58:24 PM CEST David Faure wrote: > On Sunday 03 April 2016 12:43:27 Daniel Vrátil wrote: > > Turns out when you include /usr/share in XDG_DATA_DIRS, all mime types > > that > > might have been provided by other dependencies in > > /srv/jenk

MimeType issues on CI (Was: Re: Delaying KF 5.21)

2016-04-03 Thread Daniel Vrátil
ve all of the frameworks > > anymore. Would sometime have the time to go through the list and check > > what else is missing? > Cheers, > Ben > > > -- > > David Faure, fa...@kde.org, http://www.davidfaure.fr > > Working on KDE Frameworks 5 > > ________

Re: Review Request 127091: Fix IconItem regression in 5.19

2016-02-16 Thread Daniel Vrátil
/r/127091/diff/ Testing --- I can see sni-qt apps icons in Plasma systray again. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 127091: Fix IconItem regression in 5.19

2016-02-16 Thread Daniel Vrátil
- src/declarativeimports/core/iconitem.cpp 1d7921a Diff: https://git.reviewboard.kde.org/r/127091/diff/ Testing --- I can see sni-qt apps icons in Plasma systray again. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks

[Kde-hardware-devel] All hail the new KScreen king

2016-01-27 Thread Daniel Vrátil
-- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key: 0x4D69557AECB13683 Fingerprint: 0ABD FA55 A4E6 BEA9 9A83 EA97 4D69 557A ECB1 3683 signature.asc Description: This is a digitally signed message part

Re: [Kde-hardware-devel] Review Request 126630: Do not unconditionally enable logging

2016-01-06 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126630/#review90715 --- Ship it! Ship It! - Daniel Vrátil On Jan. 5, 2016, 1:08

Re: [Kde-hardware-devel] Review Request 126630: Do not unconditionally enable logging

2016-01-05 Thread Daniel Vrátil
g/r/126630/#comment61957> Keep this one, fake backend is only for unit-tests. - Daniel Vrátil On Jan. 5, 2016, 12:27 p.m., Peter Wu wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 126481: Avoid deliberately overflowing a full buffer.

2015-12-26 Thread Daniel Vrátil
committing. Thanks! src/kcodecs.cpp (line 721) <https://git.reviewboard.kde.org/r/126481/#comment61730> Move the "else {" to this line. - Daniel Vrátil On Dec. 23, 2015, 3:50 a.m., Michael Pyne wrote: > > --- >

Re: [Kde-hardware-devel] Review Request 126381: kwayland backend for libkscreen

2015-12-21 Thread Daniel Vrátil
nitpicks, I think it's time to merge this :) Thanks a lot for you work, you rock! - Daniel Vrátil On Dec. 21, 2015, 1:59 p.m., Sebastian Kügler wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: [Kde-hardware-devel] Review Request 126381: kwayland backend for libkscreen

2015-12-20 Thread Daniel Vrátil
nd deep copy, use Q_FOREACH instead. tests/kwayland/waylandtestserver.cpp (line 105) <https://git.reviewboard.kde.org/r/126381/#comment61506> Q_FOREACH tests/kwayland/waylandtestserver.cpp (line 156) <https://git.reviewboard.kde.org/r/126381/#comment61504> Q_FOREACH - Da

Re: [Kde-hardware-devel] Review Request 126381: kwayland backend for libkscreen

2015-12-16 Thread Daniel Vrátil
kde.org/r/126381/#comment61360> const& tests/kwayland/waylandconfigreader.cpp (lines 98 - 102) <https://git.reviewboard.kde.org/r/126381/#comment61362> Use initializer list - Daniel Vrátil On Dec. 16, 2015, 3:05 a.m., Sebastian Kügler wrote: > >

Re: [Kde-hardware-devel] Review Request 126381: kwayland backend for libkscreen

2015-12-16 Thread Daniel Vrátil
> ------- > > (Updated Dec. 16, 2015, 3:05 a.m.) > > > Review request for Plasma, Solid, Daniel Vrátil, and Martin Gräßlin. > > > Repository: libkscreen > > > Description > --- > > This

Re: [Kde-pim] kdesrc-build setup for kdepim

2015-12-13 Thread Daniel Vrátil
Frameworks release soon)? > Apart from anything else, moving previously-released stuff into the KF5 > namespace when they join Frameworks will probably help packagers. > > Alex > ___ > KDE PIM mailing list kde-...@kde.org > https://mail.kd

Re: [kf5-]gpgmepp

2015-11-23 Thread Daniel Vrátil
Frameworks. Andre is the current maintainer of GpgME++, he might new what the current situation is. Cheers, Dan > > R. > ___ > Kde-frameworks-devel mailing list > Kde-frameworks-devel@kde.org > https://mail.kde.org/mailman/listinfo/kd

Re: [Kde-hardware-devel] Review Request 126101: allow loading backends in-process

2015-11-20 Thread Daniel Vrátil
the plugin binary from the process (just deleting QPluginLoader does not). src/setconfigoperation.cpp (line 58) <https://git.reviewboard.kde.org/r/126101/#comment60758> Remove - Daniel Vrátil On Nov. 20, 2015, 1:16 a.m., Sebastian Kügler wrote: > >

Re: [Kde-hardware-devel] Review Request 126101: allow loading backends in-process

2015-11-18 Thread Daniel Vrátil
ading based on current env variables. src/configoperation_p.h (line 43) <https://git.reviewboard.kde.org/r/126101/#comment60687> Maybe we don't even need to store it? Looking at the implementation in both Operations, it might just fine if loadBackend() returned the backend pointer... - Daniel Vrá

Re: Review Request 125532: KTextEdit: fix loading Sonnet default settings, add context menu to switch spell-checking language

2015-10-12 Thread Daniel Vrátil
/ktextedit.cpp f549825 Diff: https://git.reviewboard.kde.org/r/125532/diff/ Testing --- Switched KNoteEditor in KNotes in KDE PIM (which uses PimCommon::CustomTextEditor) to use this patched KTextEdit, everything works as before. Thanks, Daniel Vrátil

Re: Review Request 125570: Handle kabc deprecated classes in kabc->kcontact porting script

2015-10-09 Thread Daniel Vrátil
> On Oct. 9, 2015, 2:21 p.m., Daniel Vrátil wrote: > > kf5/convert-kabc-to-kcontacts.pl, line 78 > > <https://git.reviewboard.kde.org/r/125570/diff/1/?file=410354#file410354line78> > > > > This will still break build (as expected): &g

Re: Review Request 125570: Handle kabc deprecated classes in kabc->kcontact porting script

2015-10-09 Thread Daniel Vrátil
better to keep the deprecated classes references in the code to make porting easier and to avoid unnecessary porting mistakes. - Daniel Vrátil On Oct. 9, 2015, 2:09 p.m., R.Harish Navnit wrote: > > --- > This is an automatically ge

Review Request 125532: KTextEdit: fix loading Sonnet default settings, add context menu to switch spell-checking language

2015-10-05 Thread Daniel Vrátil
, everything works as before. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: [Kde-hardware-devel] Review Request 121915: Add lidClosedChanged signal to org.kde.Solid.PowerManagement

2015-09-02 Thread Daniel Vrátil
lid is closed/opened. Thanks, Daniel Vrátil ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel

Re: RFC: KDE Bugzilla Bugs Expiration

2015-07-31 Thread Daniel Vrátil
, but that I don't know how to solve. Dan Cheers, Thomas -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key: 0x4D69557AECB13683 Fingerprint: 0ABD FA55 A4E6 BEA9 9A83 EA97 4D69 557A ECB1 3683 signature.asc Description

Re: RFC: KDE Bugzilla Bugs Expiration

2015-07-31 Thread Daniel Vrátil
and the amount of developers (3), bugs often take more than a year to get to and to be fixed. I think this would make a lot of time consuming bug triaging much easier. Cannot agree more :) Cheers, Daniel Greetings Christoph -- Daniel Vrátil Email: dvra...@kde.org Jabber: dan.vra...@kdetalk.net

Re: Review Request 124372: Fix crash when MigrationAgent::isEmptyOldWallet() check fails

2015-07-27 Thread Daniel Vrátil
` instance pointer as an argument. Diffs - src/runtime/kwalletd/migrationagent.cpp a2a6498 Diff: https://git.reviewboard.kde.org/r/124372/diff/ Testing --- Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org

[Kde-hardware-devel] KScreen OSD

2015-07-24 Thread Daniel Vrátil
functionality are you planning and what's the general usecase for your OSD? We could then discuss how to implement it in a way that will be possible to merge into main branches and get into releases? Cheers, Daniel -- Daniel Vrátil Email: dvra...@kde.org Jabber: dan.vra...@kdetalk.net IRC: dvratil

Re: [Kde-hardware-devel] Fw:KScreen OSD

2015-07-24 Thread Daniel Vrátil
Cheers, Daniel Regards, Leslie Zhai - a KDE developer -- Daniel Vrátil Email: dvra...@kde.org Jabber: dan.vra...@kdetalk.net IRC: dvratil on Freenode (#kde, #kontact, #akonadi) ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org

Re: Moving akonadi from kdesupport and akonadi-search from playground

2015-07-24 Thread Daniel Vrátil
On Friday, July 24, 2015 10:24:10 AM laurent Montel wrote: Le Friday 24 July 2015, 00:13:48 Albert Astals Cid a écrit : El Dimarts, 21 de juliol de 2015, a les 13:52:01, Daniel Vrátil va escriure: On Monday, July 20, 2015 04:17:16 PM Daniel Vrátil wrote: Hi all, we (the KDE PIM

Re: Moving akonadi from kdesupport and akonadi-search from playground

2015-07-21 Thread Daniel Vrátil
On Monday, July 20, 2015 04:17:16 PM Daniel Vrátil wrote: Hi all, we (the KDE PIM team) kinda screwed up when we forgot to communicate our intentions about next KDE PIM release with the release team and we ended up in a situation that we have some repositories in modules from which

Moving akonadi from kdesupport and akonadi-search from playground

2015-07-20 Thread Daniel Vrátil
in during the years :) I'd like to move both repositories to kde/pim module where other KDE PIM repos (mostly those split from kdepimlibs) currently live. If that's fine with everyone I'd request the move ASAP. Cheers, Daniel -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil

Re: Moving akonadi from kdesupport and akonadi-search from playground

2015-07-20 Thread Daniel Vrátil
On 20.07.2015 16:44, Martin Sandsmark wrote: On Mon, Jul 20, 2015 at 04:17:16PM +0200, Daniel Vrátil wrote: We only ported the code to KF5 Unless I'm misunderstanding something, isn't this a quite significant change? From experience even seemingly simple ports can introduce pretty serious

Re: Review Request 124371: Fix potential endless recursion in PlatformStatus::findLookAndFeelPackage()

2015-07-16 Thread Daniel Vrátil
--- On July 16, 2015, 1:38 p.m., Daniel Vrátil wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124371

Review Request 124372: Fix crash when MigrationAgent::isEmptyOldWallet() check fails

2015-07-16 Thread Daniel Vrátil
;-), this patch makes the `invokeAndCheck()` template to take the `MigrationAgent` instance pointer as an argument. Diffs - src/runtime/kwalletd/migrationagent.cpp a2a6498 Diff: https://git.reviewboard.kde.org/r/124372/diff/ Testing --- Thanks, Daniel Vrátil

Review Request 124371: Fix potential endless recursion in PlatformStatus::findLookAndFeelPackage()

2015-07-16 Thread Daniel Vrátil
discovered this in Fedora, because we ship our own ugly LnF package, and we renamed it at some point, which caused KDED to go nuts. Diffs - src/platformstatus/platformstatus.cpp e03c16d Diff: https://git.reviewboard.kde.org/r/124371/diff/ Testing --- Thanks, Daniel Vrátil

Re: Is KF5::KCodecs::base64Encode now including a trailing newline?

2015-05-20 Thread Daniel Vrátil
On Tuesday, May 19, 2015 03:32:03 PM Albert Astals Cid wrote: El Dimarts, 19 de maig de 2015, a les 11:17:09, Daniel Vrátil va escriure: On Monday, May 18, 2015 07:32:54 PM Robby Stephenson wrote: Hi, In the process of porting Tellico to KF5, one of my unit tests that depends

Weird linkage failure on OSX CI

2015-05-20 Thread Daniel Vrátil
Hi, I'm trying to get kdepim to build on the OSX CI, but this one is beyond me. Hopefully someone more knowledgable of the OSX build system might help. The build is currently failing with following error: 00:28:09.901 Undefined symbols for architecture x86_64: 00:28:09.902

Re: [Kde-hardware-devel] Review Request 123860: Handle backend being deleted during GetConfigOperation

2015-05-20 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123860/#review80661 --- Ship it! Ship It! - Daniel Vrátil On May 20, 2015, 2:36

Review Request 123861: Make KCodecs::base64Encode() RFC compliant

2015-05-20 Thread Daniel Vrátil
- src/kcodecsbase64.cpp fc03262 Diff: https://git.reviewboard.kde.org/r/123861/diff/ Testing --- Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Is KF5::KCodecs::base64Encode now including a trailing newline?

2015-05-19 Thread Daniel Vrátil
On Monday, May 18, 2015 07:32:54 PM Robby Stephenson wrote: Hi, In the process of porting Tellico to KF5, one of my unit tests that depends on KCodecs::base64Encode() started to fail. After poking at it a little bit, my impression is that KF5::KCodecs is including a newline at the end of

Re: Review Request 123791: Cache the theme in the integration plugin

2015-05-14 Thread Daniel Vrátil
--- On May 14, 2015, 1:48 p.m., Daniel Vrátil wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123791

Re: Review Request 123791: Cache the theme in the integration plugin

2015-05-14 Thread Daniel Vrátil
email in KMail and it correctly used the new theme. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 123791: Cache the theme in the integration plugin

2015-05-14 Thread Daniel Vrátil
--- Switching between emails in KMail is faster now (even more noticable when the system is under load). Switched to another emoticon theme in System Settings, then opened another email in KMail and it correctly used the new theme. Thanks, Daniel Vrátil

Review Request 123791: Cache the theme in the integration plugin

2015-05-14 Thread Daniel Vrátil
is under load). Switched to another emoticon theme in System Settings, then opened another email in KMail and it correctly used the new theme. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Circular dependency in kde-build-metadata

2015-05-04 Thread Daniel Vrátil
Hi, commit a5f6c247d99db3a3f565f5fa8d1346420e5a892c (frameworksintegration tests want oxygen-fonts) in kde-build-metadata has introduced a circular dependency between frameworkintegration and oxygen, leading to $ kdesrc-build --refresh-build kf5 plasma5 * Downloading projects.kde.org project

New framework: KF5Syndication

2015-04-22 Thread Daniel Vrátil
by Akregator. I would like to submit KF5Syndication for the standard 2 week review period, and if everything is OK, then move it to Frameworks. Cheers, Daniel -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde

Re: New framework: KF5Syndication

2015-04-22 Thread Daniel Vrátil
On Wednesday, April 22, 2015 10:51:15 PM Mark Gaiser wrote: On Wed, Apr 22, 2015 at 9:44 PM, Daniel Vrátil dvra...@kde.org wrote: Hi all, I'd like to ask for review of another Framework from kdepimlibs: KF5Syndication KF5Syndication is an RSS/Atom parsing library. It also provides

Re: New framework: KF5Syndication

2015-04-22 Thread Daniel Vrátil
it. If Akregator ever gets ported to Akonadi :), then the retrieval will be handled by the resource and other applications using Syndication will probably prefer their own retrieval methods and can always fall back to just calling KIO::get themselves. Dan Frank -- Daniel Vrátil www.dvratil.cz | dvra

Re: [Kde-pim] KF5-QT5 branchgroup - jobs that need developeer attention

2015-04-15 Thread Daniel Vrátil
home page at http://pim.kde.org/ -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo

Re: The Future or KDE PIM Releases

2015-04-14 Thread Daniel Vrátil
On Tuesday, April 14, 2015 12:17:39 AM Albert Astals Cid wrote: El Diumenge, 12 d'abril de 2015, a les 11:31:26, Daniel Vrátil va escriure: Greetings from Toulouse! Instead we will aim towards releasing KF5-based KDE PIM with Akonadi 1 in the next release of KDE Applications in August

Re: OSX/CI: kdepimlibs fails to build on branch master

2015-04-12 Thread Daniel Vrátil
On Sunday, April 12, 2015 09:57:54 AM Marko Käning wrote: Hi Daniel, On 12 Apr 2015, at 09:50 , Daniel Vrátil dvra...@kde.org wrote: it's referring to a file in kdepim-runtime.git/resources (where Knut originally was before it was moved to kdepimlibs auto tests). I see. I don't

The Future or KDE PIM Releases

2015-04-12 Thread Daniel Vrátil
from other PIM devs who are not here in Toulouse. If everyone is fine with this proposal, we will put this plan to work as soon as possible. Regards, Daniel -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key

The Future or KDE PIM Releases

2015-04-12 Thread Daniel Vrátil
from other PIM devs who are not here in Toulouse. If everyone is fine with this proposal, we will put this plan to work as soon as possible. Regards, Daniel -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key

Re: OSX/CI: kdepimlibs fails to build on branch master

2015-04-12 Thread Daniel Vrátil
/listinfo/kde-frameworks-devel -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key: 0x4D69557AECB13683 Fingerprint: 0ABD FA55 A4E6 BEA9 9A83 EA97 4D69 557A ECB1 3683 signature.asc Description: This is a digitally signed

Re: libmm-qt moved to frameworks/modemmanager-qt

2015-04-04 Thread Daniel Vrátil
:) Regards, Jan -- Daniel Vrátil www.dvratil.cz | dvra...@kde.org IRC: dvratil on Freenode (#kde, #kontact, #akonadi, #fedora-kde) GPG Key: 0x4D69557AECB13683 Fingerprint: 0ABD FA55 A4E6 BEA9 9A83 EA97 4D69 557A ECB1 3683 signature.asc Description: This is a digitally signed message part

Re: New framework: KXmlRpcClient

2015-02-12 Thread Daniel Vrátil
On Friday, January 30, 2015 09:12:01 AM David Faure wrote: On Wednesday 28 January 2015 17:12:04 Daniel Vrátil wrote: I guess I should update kde-build-metadata and release-tools and ask for the _stable_qt5 build once the repo is moved to frameworks on projects.k.o, so that changes don't

Re: Review Request 122383: Remove internal copy of KXmlRpcClient

2015-02-12 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122383/#review75906 --- Ship it! KXmlRpcClient is a framework now! - Daniel Vrátil

Review Request 122541: KService: fix regression in stringlist properties handling

2015-02-12 Thread Daniel Vrátil
only when the `result` value returned from KPluginMetaData is also a string. Diffs - src/services/kplugininfo.cpp 572f14b Diff: https://git.reviewboard.kde.org/r/122541/diff/ Testing --- KDevelop can load plugins again. Thanks, Daniel Vrátil

Re: Review Request 122541: KService: fix regression in stringlist properties handling

2015-02-12 Thread Daniel Vrátil
can load plugins again. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: New framework: KXmlRpcClient

2015-01-28 Thread Daniel Vrátil
On Monday, January 26, 2015 08:15:14 PM Albert Astals Cid wrote: El Dilluns, 26 de gener de 2015, a les 15:25:34, Daniel Vrátil va escriure: On Thursday, January 08, 2015 06:08:00 PM Daniel Vrátil wrote: Hi all, KXmlRpcClient provides a complete XMLRPC client for applications to use

Re: New framework: KXmlRpcClient

2015-01-28 Thread Daniel Vrátil
On Wednesday, January 28, 2015 04:17:53 PM David Faure wrote: On Monday 26 January 2015 15:25:34 Daniel Vrátil wrote: Bump. We would really love to get this to 5.7, so unless anyone objects, I'll wait couple days, then file a sysadmin request and flip the 'Release' flag in the YAML

Re: New framework: KXmlRpcClient

2015-01-26 Thread Daniel Vrátil
On Thursday, January 08, 2015 06:08:00 PM Daniel Vrátil wrote: Hi all, KXmlRpcClient provides a complete XMLRPC client for applications to use. It's used most notably by DrKonqi to talk to b.k.o. It's the first framework split from kdepimlibs that we feel is ready to be shipped with KDE

Re: [Kde-hardware-devel] Review Request 122151: Invert the KJob::Exec check logic

2015-01-20 Thread Daniel Vrátil
to invoke the helper tool (because of the broken service file), the ExecJob got simply stuck. This was only fixed be rebuilding PowerDevil against the fixed KAuth... - Daniel Vrátil On Jan. 19, 2015, 4:46 p.m., Hrvoje Senjan wrote

Re: [Kde-hardware-devel] Review Request 122048: Don't suspend when closing the lid and an external monitor is connected

2015-01-14 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122048/#review73999 --- +1, looks good to me - Daniel Vrátil On Jan. 14, 2015, 12

Re: Review Request 122029: Fix generation of KAuth DBus helper service file

2015-01-14 Thread Daniel Vrátil
against KAuth with this fix, PowerDevil KDED module fully initialized and plasma-nm auth dialog works again. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks

Review Request 122029: Fix generation of KAuth DBus helper service file

2015-01-13 Thread Daniel Vrátil
works again. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: [Kde-hardware-devel] Review Request 121915: Add lidClosedChanged signal to org.kde.Solid.PowerManagement

2015-01-09 Thread Daniel Vrátil
--- On Jan. 8, 2015, 1:04 p.m., Daniel Vrátil wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121915

Re: [Kde-hardware-devel] Review Request 121915: Add lidClosedChanged signal to org.kde.Solid.PowerManagement

2015-01-08 Thread Daniel Vrátil
d51ab19 Diff: https://git.reviewboard.kde.org/r/121915/diff/ Testing --- Tested with qdbus-monitor, signal is emitted when laptop lid is closed/opened. Thanks, Daniel Vrátil ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https

New framework: KXmlRpcClient

2015-01-08 Thread Daniel Vrátil
finally drop the bundled copy from DrKonqi ;-)). Cheers, Daniel -- Daniel Vrátil | dvra...@redhat.com | dvratil on #kde-devel, #kontact, #akonadi Software Engineer - KDE Desktop Team, Red Hat Inc. GPG Key: 0xC59D614F6F4AE348 Fingerprint: 4EC1 86E3 C54E 0B39 5FDD B5FB C59D 614F 6F4A E348

Re: KF 5.6 changelog

2015-01-08 Thread Daniel Vrátil
:-) Is there a plan to move it to Tier 3, or remove the dependency? Sorry if this was discussed before. Cheers, Dan -- Daniel Vrátil | dvra...@redhat.com | dvratil on #kde-devel, #kontact, #akonadi Software Engineer - KDE Desktop Team, Red Hat Inc. GPG Key: 0xC59D614F6F4AE348 Fingerprint: 4EC1 86E3 C54E 0B39

Re: [Kde-hardware-devel] Review Request 121700: Remove CMake modules that are now part of ECM

2014-12-30 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121700/#review72777 --- Ship it! Ship It! - Daniel Vrátil On Dec. 27, 2014, 6:43

Re: [Kde-hardware-devel] Review Request 121710: Avoid risk of starting two kscreen_launchers at the same having race conditions

2014-12-30 Thread Daniel Vrátil
the same DBus service name, and I don't want to run multiple backends in one process. - Daniel Vrátil On Dec. 28, 2014, 11:33 a.m., David Edmundson wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: [Kde-pim] kdepimlibs splitting done

2014-12-12 Thread Daniel Vrátil
--empty git read-tree --prefix qgpgme/src/ $GIT_COMMIT:qgpgme/src true ' master --all --tags ___ KDE PIM mailing list kde-...@kde.org https://mail.kde.org/mailman/listinfo/kde-pim KDE PIM home page at http://pim.kde.org/ -- Daniel Vrátil | dvra

Re: Review Request 121118: KEmoticons: Add a KEmoticonsIntegrationPlugin for KTextToHTML from KCoreAddons

2014-12-04 Thread Daniel Vrátil
PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121118/diff/ Testing --- Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121094: KCoreAddons: add KTextToHTML class for plaintext - HTML conversion

2014-12-03 Thread Daniel Vrátil
-CREATION Diff: https://git.reviewboard.kde.org/r/121094/diff/ Testing --- Unit test passes Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121094: KCoreAddons: add KTextToHTML class for plaintext - HTML conversion

2014-12-02 Thread Daniel Vrátil
/121094/diff/ Testing --- Unit test passes Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121175: KCodecs: Import KEmailAddress class from email address parsing and validation from KPimUtils

2014-12-01 Thread Daniel Vrátil
it, we could have renamed KCodecs to KStringUtils and moved the kcoreaddons text subdir there, but it's too late for that. So my suggestion would be to add KEmailAddress to kcoreaddons/src/lib/text. Daniel Vrátil wrote: There's a small problem with that, because KEmailAddress depends

Re: Review Request 121175: KCodecs: Import KEmailAddress class from email address parsing and validation from KPimUtils

2014-12-01 Thread Daniel Vrátil
/CMakeLists.txt 8b5c4a0 src/kemailaddress.h PRE-CREATION src/kemailaddress.cpp PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121175/diff/ Testing --- Test passes. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde

Re: Review Request 121094: KCoreAddons: add KTextToHTML class for plaintext - HTML conversion

2014-12-01 Thread Daniel Vrátil
Diff: https://git.reviewboard.kde.org/r/121094/diff/ Testing --- Unit test passes Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121175: KCodecs: Import KEmailAddress class from email address parsing and validation from KPimUtils

2014-11-25 Thread Daniel Vrátil
. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121175/#review70816 --- On Nov. 19, 2014, 2:23 p.m., Daniel Vrátil wrote

Re: Review Request 121118: KEmoticons: Add a KEmoticonsIntegrationPlugin for KTextToHTML from KCoreAddons

2014-11-25 Thread Daniel Vrátil
://git.reviewboard.kde.org/r/121118/diff/ Testing --- Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-19 Thread Daniel Vrátil
/#comment49426 The d pointers are actually accessed by implementations, so it has to be protected. - Daniel Vrátil On Nov. 18, 2014, 2:31 p.m., Daniel Vrátil wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-19 Thread Daniel Vrátil
/data/codec_x-kmime-rfc2231/basic-encode PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121047/diff/ Testing --- Builds, all unit-tests pass. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-19 Thread Daniel Vrátil
On Nov. 19, 2014, 1:45 p.m., Daniel Vrátil wrote: src/kcodecs.h, line 786 https://git.reviewboard.kde.org/r/121047/diff/1-2/?file=327084#file327084line786 The d pointers are actually accessed by implementations, so it has to be protected. Blah, this was supposed to be a comment

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-19 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121047/#review70616 --- On Nov. 18, 2014, 2:31 p.m., Daniel Vrátil wrote

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-19 Thread Daniel Vrátil
/121047/diff/ Testing --- Builds, all unit-tests pass. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 121175: KCodecs: Import KEmailAddress class from email address parsing and validation from KPimUtils

2014-11-19 Thread Daniel Vrátil
-CREATION Diff: https://git.reviewboard.kde.org/r/121175/diff/ Testing --- Test passes. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-18 Thread Daniel Vrátil
--- On Nov. 14, 2014, 1:41 p.m., Daniel Vrátil wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121047

Re: Review Request 121047: KCodecs: add MIME codecs, improve {encode, decode}RFC2047String()

2014-11-18 Thread Daniel Vrátil
pass. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121094: KCoreAddons: add KTextToHTML class for plaintext - HTML conversion

2014-11-18 Thread Daniel Vrátil
/lib/text/ktexttohtml.cpp PRE-CREATION src/lib/text/ktexttohtml_p.h PRE-CREATION src/lib/text/ktexttohtmlemoticonsinterface.h PRE-CREATION Diff: https://git.reviewboard.kde.org/r/121094/diff/ Testing --- Unit test passes Thanks, Daniel Vrátil

Re: Review Request 120155: Correctly load non-square icons in KIconLoader

2014-11-18 Thread Daniel Vrátil
://git.reviewboard.kde.org/r/120155/diff/ Testing --- QIcon icon = QIcon::fromTheme(QLatin1String(process-working)); QPixmap pix = icon.pixmap(22, 8 * 22); With this patch, the pix looks as expected. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde

Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

2014-11-14 Thread Daniel Vrátil
://git.reviewboard.kde.org/r/121095/diff/ Testing --- Tested with KTextToHTML code from /r/121094 in a QGuiApplication and it seems to work. Thanks, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman

Review Request 121118: KEmoticons: Add a KEmoticonsIntegrationPlugin for KTextToHTML from KCoreAddons

2014-11-14 Thread Daniel Vrátil
, Daniel Vrátil ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

<    1   2   3   >