Re: Review Request 106300: add kimgio WebP image format plugin

2013-06-15 Thread Christoph Feck
> On May 18, 2013, 12:44 p.m., Commit Hook wrote: > > This review has been submitted with commit > > df845b06bc440fff5a43a9cdc149e8c88c29bc67 by Martin Koller to branch master. > > Christoph Feck wrote: > Just noticed this has been committed to kolourpaint instead of kdelibs. > Did I miss

Re: Review Request 111000: add KTextEditor::MessageInterface for KDE SC 4.11

2013-06-15 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111000/#review34403 --- Ship it! Yes, this is fine. I won't create trouble in KF5 sin

Re: Review Request 111042: Upstream SUSE specific changes and introduce a DISTRO switch to enable

2013-06-15 Thread Johannes Obermayr
> On June 15, 2013, 7:06 p.m., Albert Astals Cid wrote: > > Is this some kind of joke? > > > > Honestly upstreaming patches is good, but when they make sense, and adding > > billions of "If #SUSE" to our code does not make any sense. Please discard > > this review, break stuff in separate patc

Re: Review Request 111016: Set modified timestamp when moving resource from ftp -> file

2013-06-15 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111016/ --- (Updated June 15, 2013, 8:34 p.m.) Status -- This change has been mar

Re: Review Request 111016: Set modified timestamp when moving resource from ftp -> file

2013-06-15 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111016/#review34400 --- This review has been submitted with commit 4a75f8b6dbc36a858b5

Re: Review Request 111042: Upstream SUSE specific changes and introduce a DISTRO switch to enable

2013-06-15 Thread Luca Beltrame
> On June 15, 2013, 6:52 p.m., Luca Beltrame wrote: > > I would like to chime in, since I'm also a member of the openSUSE community > > KDE team. This patch and the idea behind it is *not* the official position > > of openSUSE, nor the official position of the openSUSE community KDE team. > >

Re: Review Request 111042: Upstream SUSE specific changes and introduce a DISTRO switch to enable

2013-06-15 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111042/#review34392 --- I would like to chime in, since I'm also a member of the openSU

Re: Review Request 111000: add KTextEditor::MessageInterface for KDE SC 4.11

2013-06-15 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111000/ --- (Updated June 15, 2013, 6:18 p.m.) Review request for Kate, kdelibs, Relea

Re: Review Request 111000: add KTextEditor::MessageInterface for KDE SC 4.11

2013-06-15 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111000/#review34390 --- For me it's ok to go on with this freeze exception since it's c

Re: Review Request 111042: Upstream SUSE specific changes and introduce a DISTRO switch to enable

2013-06-15 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111042/#review34388 --- Is this some kind of joke? Honestly upstreaming patches is goo

Review Request 111042: Upstream SUSE specific changes and introduce a DISTRO switch to enable

2013-06-15 Thread Johannes Obermayr
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111042/ --- Review request for kdelibs. Description --- Distributions should upst