Outgoing e-mails from Gerrit review to uninterested people

2015-01-03 Thread Jan Kundrát
On Saturday, 3 January 2015 08:57:43 CEST, Aaron J. Seigo wrote: It would be nice if there was an opt-out for this. I receive a large number of emails from gerrit for reviews which I have been automatically subscribed to which I have absolutely zero interest in. Hi Aaron, sorry about that. D

Re: Changes to our Git infrastructure

2015-01-03 Thread Jan Kundrát
On Saturday, 3 January 2015 21:35:12 CEST, Jeff Mitchell wrote: On 3 Jan 2015, at 14:00, Jan Kundrát wrote: - Working on git trees, not patches. This directly translates into making the contributors familiar with our workflow, and therefore getting them "immersed" into what we're doing and hel

Re: Changes to our Git infrastructure

2015-01-03 Thread Albert Astals Cid
El Dissabte, 3 de gener de 2015, a les 15:31:26, Ben Cooksley va escriure: > Hi all, > > I've gone over the comments everyone has made thus far and came up > with the following community wishlist as it were. > It represents a combination of what everyone has said, in a fairly > distilled form. >

Re: Changes to our Git infrastructure

2015-01-03 Thread Jeff Mitchell
On 3 Jan 2015, at 14:00, Jan Kundrát wrote: - Working on git trees, not patches. This directly translates into making the contributors familiar with our workflow, and therefore getting them "immersed" into what we're doing and helping bridge the gap between maintainers and contributors. I agr

Re: Review Request 121717: libksysguard/processtable: Add new column "Relative Start Time"

2015-01-03 Thread Gregor Mi
> On Jan. 3, 2015, 12:35 p.m., Dominik Haumann wrote: > > The feature itself is nice, but in its current implementation, the > > translators won't be able to properly translate the strings into other > > languages, see detailed comments. > > > > Besides that, would be nice to get another revie

Re: Review Request 121717: libksysguard/processtable: Add new column "Relative Start Time"

2015-01-03 Thread Gregor Mi
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121717/ --- (Updated Jan. 3, 2015, 8:02 p.m.) Review request for KDE Base Apps and Jo

Re: Review Request 120202: [OS X] improvements to the kwallet/OSX keychain integration

2015-01-03 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120202/ --- (Updated Jan. 3, 2015, 8:41 p.m.) Review request for KDE Software on Mac

Re: Review Request 120202: [OS X] improvements to the kwallet/OSX keychain integration

2015-01-03 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120202/ --- (Updated Jan. 3, 2015, 8:42 p.m.) Review request for KDE Software on Mac

Re: Changes to our Git infrastructure

2015-01-03 Thread Jan Kundrát
On Saturday, 3 January 2015 03:31:26 CEST, Ben Cooksley wrote: Regrettably there were one or two items which conflicted. I sided with the option which kept the barrier to entry as low as possible as that seemed to be the greater consensus within the thread. Hi Ben, thanks for compiling a list.

Re: Changes to our Git infrastructure

2015-01-03 Thread Thiago Macieira
On Saturday 03 January 2015 12:39:42 Lydia Pintscher wrote: > On Sat, Jan 3, 2015 at 12:07 PM, Thiago Macieira wrote: > > For Gerrit: > I think before checking Ben's list against specific implementations we > should make sure the list is actually correct and complete. It's the > basis for an impor

Re: libkgeomap

2015-01-03 Thread Thomas Lübking
On Samstag, 3. Januar 2015 15:09:58 CEST, Tobias Leupold wrote: IMHO, the real question is, shouldn't that pointless wrapper be deprecated in favor of just using Marble directly? Can marble be used in an identical way as libkgeomap Afaiu and would expect, libkgeomap wraps marble, openstreetma

Re: Changes to our Git infrastructure

2015-01-03 Thread Matthew Dawson
On January 3, 2015 03:31:26 PM Ben Cooksley wrote: > Hi all, > > I've gone over the comments everyone has made thus far and came up > with the following community wishlist as it were. > It represents a combination of what everyone has said, in a fairly > distilled form. > > Regrettably there were

Review Request 121805: kconfig: fix building when KDE_INSTALL_DIRS_NO_DEPRECATED is set

2015-01-03 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121805/ --- Review request for kdelibs and Marko Käning. Repository: kconfig Descri

Re: libkgeomap

2015-01-03 Thread Tobias Leupold
> IMHO, the real question is, shouldn't that pointless wrapper be deprecated > in favor of just using Marble directly? Can marble be used in an identical way as libkgeomap, as a QWidget only displaying a map with not only coordinates but also photo thumbnails (grouping the coordinates when the z

Re: Review Request 121717: libksysguard/processtable: Add new column "Relative Start Time"

2015-01-03 Thread Dominik Haumann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121717/#review73001 --- The feature itself is nice, but in its current implementation,

Re: Changes to our Git infrastructure

2015-01-03 Thread Lydia Pintscher
On Sat, Jan 3, 2015 at 12:07 PM, Thiago Macieira wrote: > For Gerrit: I think before checking Ben's list against specific implementations we should make sure the list is actually correct and complete. It's the basis for an important decision so let's do this step-by-step? :) Cheers Lydia -- L

Re: Changes to our Git infrastructure

2015-01-03 Thread Thiago Macieira
For Gerrit: > - Code Reviews: > - CLI client to make changes to the code review system and to manage > the review (including retrieving the commits/patches) Check. It's ssh host gerrit > Client should have good documentation. > - System should automatically set the CC / Reviewers / etc

Re: Changes to our Git infrastructure

2015-01-03 Thread Ben Cooksley
On Sat, Jan 3, 2015 at 8:57 PM, Aaron J. Seigo wrote: > On Saturday, January 3, 2015 15.31:26 Ben Cooksley wrote: >> - System should automatically set the CC / Reviewers / etc for a >> review rather than the submitter needing to know who to set. > > It would be nice if there was an opt-out for t

Re: Changes to our Git infrastructure

2015-01-03 Thread Aaron J. Seigo
On Saturday, January 3, 2015 15.31:26 Ben Cooksley wrote: > - System should automatically set the CC / Reviewers / etc for a > review rather than the submitter needing to know who to set. It would be nice if there was an opt-out for this. I receive a large number of emails from gerrit for revie