Re: Review Request 119410: Document KDED dependency.

2014-07-23 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119410/#review62923 --- Ship it! Yes, KIO needs kded for favicons, cached passwords,

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread Marco Martin
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119409: framework part of the applet alternatives shooser

2014-07-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119409/ --- (Updated July 23, 2014, 9:18 a.m.) Review request for KDE Frameworks and

Re: Review Request 119410: Document KDED dependency.

2014-07-23 Thread Michael Palimaka
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119410/ --- (Updated July 23, 2014, 9:50 a.m.) Status -- This change has been

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread David Edmundson
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread Marco Martin
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119409: framework part of the applet alternatives shooser

2014-07-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119409/ --- (Updated July 23, 2014, 10:31 a.m.) Status -- This change has been

Re: KDELibs4Support fails to install on Windows if DocBookXML4 absent

2014-07-23 Thread Luigi Toscano
On Wednesday 23 of July 2014 00:27:18 Nicolás Alvarez wrote: Hello, Currently KDELibs4Support fails to install on Windows. It gives the following error: CMake Error at src/cmake_install.cmake:264 (file): file RELATIVE_PATH must be passed a full path to the file: searched_dir-NOTFOUND

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread David Edmundson
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread Marco Martin
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread Marco Martin
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread David Edmundson
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread Marco Martin
On July 22, 2014, 10:50 p.m., Eike Hein wrote: Two issues: - Existing Dialog instances don't get the corner mask applied correctly when switching from transparent to opague: ![Shot](http://wstaw.org/m/2014/07/23/cornermask.png) - Compiler warning:

Re: KDELibs4Support fails to install on Windows if DocBookXML4 absent

2014-07-23 Thread Nicolás Alvarez
El miércoles, 23 de julio de 2014, Luigi Toscano luigi.tosc...@tiscali.it escribió: On Wednesday 23 of July 2014 00:27:18 Nicolás Alvarez wrote: Hello, Currently KDELibs4Support fails to install on Windows. It gives the following error: CMake Error at src/cmake_install.cmake:264

Re: Review Request 119406: Always take the painter based path for composeOverBorder

2014-07-23 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119406/ --- (Updated July 23, 2014, 1:53 p.m.) Status -- This change has been

Re: Review Request 119377: Add highlighting commands

2014-07-23 Thread Christoph Rüßler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119377/ --- (Updated Juli 23, 2014, 5:08 nachm.) Review request for KDE Frameworks

Re: Review Request 119377: Add highlighting commands

2014-07-23 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119377/#review62977 --- Ship it! Ship It! - Christoph Cullmann On July 23, 2014,

Review Request 119435: Replacing also kaction.h header include

2014-07-23 Thread David Narváez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119435/ --- Review request for KDE Frameworks. Repository: kde-dev-scripts

KF5 Porting: Installing Icons

2014-07-23 Thread David Narvaez
Hi, The information at [0] seems to indicate I just need to worry about renaming files to match the new categories (btw, is there a script for that?) but what I get from CMake Warning (dev) at /usr/share/ECM/modules/ECMInstallIcons.cmake:205 (message): ecm_install_icons() with no ICONS

Re: KF5 Porting: Installing Icons

2014-07-23 Thread Jonathan Riddell
On Wed, Jul 23, 2014 at 06:08:42PM -0400, David Narvaez wrote: This warning is for project developers. Use -Wno-dev to suppress it. is that I need to update to which seems to me like I need to explicitly list all icons in the dir, and Yes you are advised to use the new install function as

Re: KF5 Porting: Installing Icons

2014-07-23 Thread David Narvaez
On Wed, Jul 23, 2014 at 6:20 PM, Jonathan Riddell j...@jriddell.org wrote: # The given icons, whose names must match the pattern:: # # size-group-name.ext seems to indicate I need to rename all of the files to remove the theme from the prefix. Keep the theme name, preferably use hicolor

Re: KF5 Porting: Installing Icons

2014-07-23 Thread Jonathan Riddell
On Wed, Jul 23, 2014 at 06:23:32PM -0400, David Narvaez wrote: On Wed, Jul 23, 2014 at 6:20 PM, Jonathan Riddell j...@jriddell.org wrote: # The given icons, whose names must match the pattern:: # # size-group-name.ext seems to indicate I need to rename all of the files to remove the

Review Request 119444: Replacing also kicon.h header include

2014-07-23 Thread David Narváez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119444/ --- Review request for KDE Frameworks. Repository: kde-dev-scripts

Review Request 119446: Add a simple test for meinproc

2014-07-23 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119446/ --- Review request for Documentation, KDE Frameworks and kdelibs.

Review Request 119447: meinproc: check both result of parsing and validation

2014-07-23 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119447/ --- Review request for Documentation, KDE Frameworks and kdelibs.

Re: Review Request 119447: meinproc: check both result of parsing and validation

2014-07-23 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119447/#review63024 --- +1, makes sense to me, I don't see a drawback to further