Re: [Development] Please help me get my pending review count down

2015-08-05 Thread Thiago Macieira
On Tuesday 28 July 2015 19:06:59 Thiago Macieira wrote: > Deadline: 6 weeks before Qt 5.7 feature freeze. > > If we reach that date and the patches are still not integrated, I will have > to make a decision as Maintainer as to the future of them. It will be to > either apply them despite any brea

Re: Replacement for KDateTime

2015-08-05 Thread Dāvis Mosāns
2015-08-02 21:32 GMT+03:00 David Jarvie : > > Date-only KDateTime instances are not only used for Event start/end > timestamps. In KAlarm they are also used among other things for alarm snooze > times (independently of whether the event is date-only or not). So usage of > the date-only attribute is

Re: Replacement for KDateTime

2015-08-05 Thread Sandro Knauß
Hey John, > OK, I had a quick look at this and to do it in Qt and it wasn't actually > that hard. You can see the result at: > > https://codereview.qt-project.org/#/c/122750/ > > This should support the full set of VTIMEZONE rules, have a look and let me > know if not. > > I'm not sure if it wi

Re: Replacement for KDateTime

2015-08-05 Thread Dāvis Mosāns
2015-08-03 15:26 GMT+03:00 David Jarvie : > On Monday 03 Aug 2015 12:59:59 you wrote: > >> 2015-08-02 21:32 GMT+03:00 David Jarvie : > >> > > >> > Date-only KDateTime instances are not only used for Event start/end > >> > timestamps. In KAlarm they are also used among other things for alarm >> > sn

Re: Replacement for KDateTime

2015-08-05 Thread Dāvis Mosāns
2015-08-03 17:33 GMT+03:00 David Jarvie : > On Monday 03 Aug 2015 14:29:41 Dāvis Mosāns wrote: > >> 2015-08-03 15:26 GMT+03:00 David Jarvie : > >> > On Monday 03 Aug 2015 12:59:59 you wrote: > >> > > >> >> 2015-08-02 21:32 GMT+03:00 David Jarvie : > >> > > >> >> > > >> > > >> >> > Date-only KDateTi

Re: Replacement for KDateTime

2015-08-05 Thread Dāvis Mosāns
2015-08-03 20:09 GMT+03:00 Martin Klapetek : > On Mon, Aug 3, 2015 at 6:34 PM, Thiago Macieira wrote: >> >> On Monday 03 August 2015 08:33:54 Martin Klapetek wrote: >> > > If the format you're looking for requires support from translators, >> > > please >> > > add >> > > a new class to QtCore. >>

Re: Notes from the Phabricator BoF

2015-08-05 Thread Ben Cooksley
Hi all, Just clarifying a few things that have happened post discussion regarding how we would use Phabricator, in particular Projects. At this time Phabricator lacks sub-project functionality - which for certain projects is problematic (whether it will be or not will depend on your particular use

Plasma Applet for Audio Volume for kdereview

2015-08-05 Thread Jonathan Riddell
plasma-pa is a new volume manager and is intended to be a replacement for KMix in Plasma. We plan to ship it as a beta in Plasma 5.4 and it's currently in kdereview for your reviewing attention. https://projects.kde.org/projects/kdereview/plasma-pa Please check over it and consider if it passe

kwallet-pam for kdereview

2015-08-05 Thread Jonathan Riddell
kwallet-pam is a simple PAM module to open your wallet. We'd like to ship it in Plasma 5.4. It was previously in a scratch repository but shipped by distros such as Kubuntu and Fedora. It's about to go into kdereview and we'd appreciate your considered perusal. https://projects.kde.org/projects

Re: Plasma Applet for Audio Volume for kdereview

2015-08-05 Thread Jonathan Riddell
On Wed, Aug 05, 2015 at 02:51:25PM +0200, Ganton wrote: > Hi, Jonathan: > > > plasma-pa is a new volume manager and is intended to be a replacement for > > KMix in Plasma. > > > > We plan to ship it as a beta in Plasma 5.4 and it's currently in kdereview > > for your reviewing attention. > > > >

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-08-05 Thread Rex Dieter
> On April 7, 2015, 1:44 a.m., Aleix Pol Gonzalez wrote: > > This doesn't let the user change the path but the distributor, that's quite > > a different thing. Maybe it should be a runtime check? Ideally yes, but the approach taken here so far is at least an incremental improvement over the st

Re: Review Request 122987: Allow user to specify path to myspell dictionary files

2015-08-05 Thread Rex Dieter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122987/#review83458 --- Ship it! This is the obvious easy fix (I was about to submit

Re: Plasma Applet for Audio Volume for kdereview

2015-08-05 Thread Albert Astals Cid
El Dimecres, 5 d'agost de 2015, a les 12:45:01, Jonathan Riddell va escriure: > plasma-pa is a new volume manager and is intended to be a replacement for > KMix in Plasma. > > We plan to ship it as a beta in Plasma 5.4 and it's currently in kdereview > for your reviewing attention. > > https://pr