Re: Review Request 113723: Fix KIO to build standalone, prepare for moving into its tier

2013-11-10 Thread Aleix Pol Gonzalez
--- On Nov. 11, 2013, 3:23 a.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113723

Re: Review Request 113723: Fix KIO to build standalone, prepare for moving into its tier

2013-11-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113723/#review43411 --- On Nov. 11, 2013, 3:23 a.m., Aleix Pol Gonzalez wrote

Re: Review Request 113723: Fix KIO to build standalone, prepare for moving into its tier

2013-11-11 Thread Aleix Pol Gonzalez
/KDocToolsConfig.cmake.in PRE-CREATION tier2/kdoctools/src/CMakeLists.txt 3940e98 Diff: http://git.reviewboard.kde.org/r/113723/diff/ Testing --- Builds, Installs, tests still pass; both modularized and monolithic kdelibs. Thanks, Aleix Pol Gonzalez

Review Request 113795: Cut dependency from KIO to KCoreAddons private headers

2013-11-11 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/113795/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113723: Fix KIO to build standalone, prepare for moving into its tier

2013-11-11 Thread Aleix Pol Gonzalez
/KDocToolsConfig.cmake.in PRE-CREATION tier2/kdoctools/src/CMakeLists.txt 3940e98 Diff: http://git.reviewboard.kde.org/r/113723/diff/ Testing --- Builds, Installs, tests still pass; both modularized and monolithic kdelibs. Thanks, Aleix Pol Gonzalez

Re: Review Request 113695: Fix KDEWebKit standalone build

2013-11-11 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/113695/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113694: Fix KNewStuff standalone build

2013-11-11 Thread Aleix Pol Gonzalez
/113694/diff/ Testing --- Builds and installs. All tests are commented out. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113693: Fix KNotifyConfig standalone build

2013-11-11 Thread Aleix Pol Gonzalez
Diff: http://git.reviewboard.kde.org/r/113693/diff/ Testing --- Builds, installs, the test seems to work. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Re: Review Request 113795: Cut dependency from KIO to KCoreAddons private headers

2013-11-11 Thread Aleix Pol Gonzalez
staging/kio/src/core/job_p.h 55c00fa staging/kio/src/core/jobclasses.h fb5051c Diff: http://git.reviewboard.kde.org/r/113795/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https

Re: Review Request 113805: Do not change the build types available with cmake

2013-11-11 Thread Aleix Pol Gonzalez
it explained somehow. Why is it bad to define such values? How will g++ calls compare? - Aleix Pol Gonzalez On Nov. 11, 2013, 10:16 p.m., Sune Vuorela wrote: --- This is an automatically generated e-mail. To reply, visit: http

Re: Review Request 113805: Do not change the build types available with cmake

2013-11-11 Thread Aleix Pol Gonzalez
On Nov. 12, 2013, 12:02 a.m., Aleix Pol Gonzalez wrote: Maybe I've missed something, but I would like to have it explained somehow. Why is it bad to define such values? How will g++ calls compare? Nicolás Alvarez wrote: In normal CMake, -DCMAKE_BUILD_TYPE=Debug builds without

Re: Review Request 113810: update kcookiejar manpage

2013-11-12 Thread Aleix Pol Gonzalez
to review the rest of documentation files. Also some guidelines might be useful as well. - Aleix Pol Gonzalez On Nov. 12, 2013, 10:10 a.m., Jonathan Riddell wrote: --- This is an automatically generated e-mail. To reply, visit: http

Re: Review Request 113818: Add Remaining Tier 2 Builds to Superbuild

2013-11-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113818/#review43527 --- Ship it! Ship It! - Aleix Pol Gonzalez On Nov. 12, 2013, 3

Re: Review Request 113506: Add a sb_all target and sb_$framework targets to make it easier to build frameworks standalone

2013-11-12 Thread Aleix Pol Gonzalez
by one is annoying. - Aleix Pol Gonzalez On Nov. 7, 2013, 2:42 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113506

Review Request 113821: Don't install kpagedialog_p.h

2013-11-12 Thread Aleix Pol Gonzalez
Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113683: Fix kdeclarative standalone build

2013-11-12 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113683/#review43555 --- Ship it! No, if it works just push it. - Aleix Pol Gonzalez

Re: Review Request 113821: Don't install kpagedialog_p.h

2013-11-13 Thread Aleix Pol Gonzalez
--- On Nov. 12, 2013, 6:46 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113821

Re: Review Request 113506: Add a sb_all target and sb_$framework targets to make it easier to build frameworks standalone

2013-11-13 Thread Aleix Pol Gonzalez
On Nov. 12, 2013, 5:11 p.m., Aleix Pol Gonzalez wrote: CMakeLists.txt, line 18 http://git.reviewboard.kde.org/r/113506/diff/5/?file=211419#file211419line18 I don't think we want that, superbuild shouldn't be used after the splitting but the kde build script. We will need

Re: Review Request 113506: Add a sb_all target and sb_$framework targets to make it easier to build frameworks standalone

2013-11-13 Thread Aleix Pol Gonzalez
On Nov. 12, 2013, 5:11 p.m., Aleix Pol Gonzalez wrote: CMakeLists.txt, line 18 http://git.reviewboard.kde.org/r/113506/diff/5/?file=211419#file211419line18 I don't think we want that, superbuild shouldn't be used after the splitting but the kde build script. We will need

Re: Review Request 113821: Don't install kpagedialog_p.h

2013-11-13 Thread Aleix Pol Gonzalez
builds, couldn't test since I didn't see any test. Tests still pass though. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113844: Move kspeech interface to tier3

2013-11-13 Thread Aleix Pol Gonzalez
, no? - Aleix Pol Gonzalez On Nov. 13, 2013, 5:16 p.m., Jeremy Whiting wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113844

Re: Review Request 113844: Move kspeech interface to tier3

2013-11-13 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113844/#review43621 --- Ship it! Ship It! - Aleix Pol Gonzalez On Nov. 13, 2013, 6

Re: Review Request 113867: Add knewstuff public dependencies to it's config.cmake file.

2013-11-14 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113867/#review43690 --- Ship it! Ship It! - Aleix Pol Gonzalez On Nov. 14, 2013, 6

Re: Review Request 113881: Standardise d-pointer classes of kmediaplayer

2013-11-15 Thread Aleix Pol Gonzalez
the Data pointer was before. It's only defined for one class but both have it as a member? It was used as a d-ptr only for some members? - Aleix Pol Gonzalez On Nov. 15, 2013, 12:20 p.m., Alex Merry wrote: --- This is an automatically

Re: Review Request 113882: Make constructors in KMediaPlayer explicit

2013-11-15 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113882/#review43728 --- Ship it! Ship It! - Aleix Pol Gonzalez On Nov. 15, 2013

Re: Review Request 113883: Rename translation catalogue kfileaudiopreview4 to kfileaudiopreview

2013-11-15 Thread Aleix Pol Gonzalez
been bumping to 5... - Aleix Pol Gonzalez On Nov. 15, 2013, 12:22 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113883

Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-15 Thread Aleix Pol Gonzalez
/diff/ Testing --- Nothing stopped building. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113863: Remove WINCE specific cmake code paths

2013-11-15 Thread Aleix Pol Gonzalez
. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113887: Deprecate KDE4_* KAuth macros

2013-11-18 Thread Aleix Pol Gonzalez
/kde4support/cmake/modules/KDE4Macros.cmake 6a63e5e Diff: http://git.reviewboard.kde.org/r/113887/diff/ Testing --- Nothing stopped building. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https

Re: Review Request 113863: Remove WINCE specific cmake code paths

2013-11-18 Thread Aleix Pol Gonzalez
/CMakeLists.txt 4ef0a9b tier3/kio/src/CMakeLists.txt 3800e85 tier3/kprintutils/src/CMakeLists.txt da0b7d5 tier4/kde4support/src/CMakeLists.txt 9046c96 Diff: http://git.reviewboard.kde.org/r/113863/diff/ Testing --- still builds. Thanks, Aleix Pol Gonzalez

Re: Review Request 113972: Add some autotests for KMediaPlayer

2013-11-20 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113972/#review44084 --- Ship it! Ship It! - Aleix Pol Gonzalez On Nov. 21, 2013

Review Request 113996: Install needed private headers for FrameworksIntegration

2013-11-21 Thread Aleix Pol Gonzalez
/CMakeLists.txt 9223ccf tier3/kiconthemes/src/CMakeLists.txt 4b3c978 Diff: http://git.reviewboard.kde.org/r/113996/diff/ Testing --- FrameworksIntegration builds modularized, monolithic kdelibs still build as well. Thanks, Aleix Pol Gonzalez ___ Kde

Review Request 114003: Install private headers needed by kde4support

2013-11-21 Thread Aleix Pol Gonzalez
--- Builds, tests still pass. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 114038: Integrate KAboutData and QCommandLineParser

2013-11-22 Thread Aleix Pol Gonzalez
kbuildsycoca5 use it and made sure it works. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 114003: Install private headers needed by kde4support

2013-11-22 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/114003/diff/ Testing --- Builds, tests still pass. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113975: Make some KSSL files public API

2013-11-25 Thread Aleix Pol Gonzalez
Diff: http://git.reviewboard.kde.org/r/113975/diff/ Testing --- Everything still builds. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 114003: Install private headers needed by kde4support

2013-11-25 Thread Aleix Pol Gonzalez
9a731fd tier4/kde4support/src/kio/kmimetyperesolver.h 515dcfb tier4/kde4support/src/kio/kmimetyperesolver.cpp 69e64fd tier4/kde4support/tests/CMakeLists.txt 7705381 Diff: http://git.reviewboard.kde.org/r/114003/diff/ Testing --- Builds, tests still pass. Thanks, Aleix Pol Gonzalez

Re: Review Request 113996: Install needed private headers for FrameworksIntegration

2013-11-25 Thread Aleix Pol Gonzalez
builds modularized, monolithic kdelibs still build as well. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 114038: Integrate KAboutData and QCommandLineParser

2013-11-25 Thread Aleix Pol Gonzalez
05019ad Diff: http://git.reviewboard.kde.org/r/114038/diff/ Testing --- Made kbuildsycoca5 use it and made sure it works. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org

Re: Review Request 114038: Integrate KAboutData and QCommandLineParser

2013-11-25 Thread Aleix Pol Gonzalez
--- On Nov. 22, 2013, 6:07 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114038

Re: Review Request 114272: Fix broken connect for signal finished in kcmultidialog

2013-12-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114272/#review45042 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 3, 2013, 2

Re: Review Request 114256: Make QFileDialog use KFileWidget from the KDE PlatformTheme

2013-12-04 Thread Aleix Pol Gonzalez
/diff/ Testing --- Created a test app that instantiates a QFileDialog, used a QFileDialog from systemsettings. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo

Re: Review Request 114185: Provide our own KFontDialog

2013-12-08 Thread Aleix Pol Gonzalez
/platformtheme/kdeplatformtheme.h 69f1f44 tier4/frameworkintegration/src/platformtheme/kdeplatformtheme.cpp 010eed4 Diff: http://git.reviewboard.kde.org/r/114185/diff/ Testing --- Works with the kfontrequestertest. Thanks, Aleix Pol Gonzalez ___ Kde

Re: Review Request 114356: Move KGlobalAccel from XmlGui into an own tier1 framework

2013-12-10 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/114356/#comment32420 Shouldn't that file be called KGlobalAccelConfig.cmake.in? We're not prefixing the rest of modules with KF5, right? - Aleix Pol Gonzalez On Dec. 10, 2013, 6:41 a.m., Martin Gräßlin wrote

Re: Review Request 112463: Port SMB kioslave to KF5/Qt5

2013-12-10 Thread Aleix Pol Gonzalez
to remove this comment- kioslave/smb/kio_smb_internal.cpp http://git.reviewboard.kde.org/r/112463/#comment32427 Does it really need QDir::separator()? It seems to me like everything should be '/'. - Aleix Pol Gonzalez On Dec. 5, 2013, 11:44 p.m., Mark Gaiser wrote

Re: Review Request 112463: Port SMB kioslave to KF5/Qt5

2013-12-10 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112463/#review45520 --- Ship it! - Aleix Pol Gonzalez On Dec. 10, 2013, 6:51 p.m

Re: Review Request 114447: Split Frameworks only definitions to a new file

2013-12-14 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114447/#review45683 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 14, 2013, 3

Re: Review Request 114501: Rename KDE4_ENABLE_EXCEPTIONS to KDE_ENABLE_EXCEPTIONS

2013-12-16 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114501/#review45802 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 16, 2013, 5

Re: Review Request 114502: KDE4_ENABLE_EXCEPTIONS = KDE_ENABLE_EXCEPTIONS

2013-12-16 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114502/#review45801 --- Ship it! Looks good to me. - Aleix Pol Gonzalez On Dec. 16

Re: Review Request 114683: Clean up target_link_libraries for KF5Plasma

2013-12-27 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114683/#review46193 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 27, 2013

Re: Review Request 114716: Separate author name from email addres in KAboutData::processCommandLine

2013-12-29 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114716/#review46422 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 29, 2013

Re: Review Request 114937: port kconfig_compiler_kf5 to QCommandLineParser

2014-01-10 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/114937/#comment33590 directoryName is not used now? - Aleix Pol Gonzalez On Jan. 10, 2014, 4:03 p.m., Bhushan Shah wrote: --- This is an automatically generated e-mail. To reply, visit: https

Review Request 115016: Make KJob usable from QML

2014-01-14 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/115016/diff/ Testing --- Everything still builds, shouldn't be a big deal. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 115016: Make KJob usable from QML

2014-01-14 Thread Aleix Pol Gonzalez
--- On Jan. 14, 2014, 8:16 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115016

Review Request 115030: Install public headers for KJsEmbed

2014-01-15 Thread Aleix Pol Gonzalez
the now defunct Kross KJS support). https://git.reviewboard.kde.org/r/115027/ Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 115016: Make KJob usable from QML

2014-01-16 Thread Aleix Pol Gonzalez
and check if it succeeded. Diffs - src/lib/jobs/kjob.h 24dbaec Diff: https://git.reviewboard.kde.org/r/115016/diff/ Testing --- Everything still builds, shouldn't be a big deal. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list

Re: Review Request 115049: Do not require ktranscript to be installed for the test to run

2014-01-16 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/115049/#comment33759 This limits the cwd. Maybe you can pass it as a preprocess definition? aka with add_definitions(-DKTRANSCRIPT_PATH=${blah}) ). - Aleix Pol Gonzalez On Jan. 16, 2014, 11:07 a.m., Aurélien Gâteau wrote

Re: Review Request 115030: Install public headers for KJsEmbed

2014-01-16 Thread Aleix Pol Gonzalez
., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115030/ --- (Updated Jan. 16, 2014, 2:33 p.m.) Review

Re: Review Request 114699: Use enums instead of ints in method types

2014-01-16 Thread Aleix Pol Gonzalez
the source compatibility, but then it improves the API considerably. - Aleix Pol Gonzalez On Jan. 16, 2014, 1:28 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r

Re: Review Request 115030: Install public headers for KJsEmbed

2014-01-16 Thread Aleix Pol Gonzalez
src/kjsembed/pointer.h e25a5e4 Diff: https://git.reviewboard.kde.org/r/115030/diff/ Testing --- I ported the share dataengine to use it (it was using the now defunct Kross KJS support). https://git.reviewboard.kde.org/r/115027/ Thanks, Aleix Pol Gonzalez

Re: Review Request 115077: Rename Macros in KF5DocTools to KDE5_

2014-01-18 Thread Aleix Pol Gonzalez
On Jan. 17, 2014, 6:51 p.m., Alex Merry wrote: KF5DocToolsMacros.cmake, lines 166-172 https://git.reviewboard.kde.org/r/115077/diff/1/?file=234284#file234284line166 These should *not* be renamed, as they are compatibility macros. However, they should probably be moved to

Review Request 115198: Fix KDE4Support compilation

2014-01-21 Thread Aleix Pol Gonzalez
5a7f4c8 Diff: https://git.reviewboard.kde.org/r/115198/diff/ Testing --- Builds. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 115202: Allow building KConfigWidgets on Windows

2014-01-21 Thread Aleix Pol Gonzalez
/#comment33948 What's the problem with finding documentation on windows? - Aleix Pol Gonzalez On Jan. 21, 2014, 10:26 p.m., Alexander Richardson wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 115199: Fix detection of shared-mime-info on Windows

2014-01-21 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115199/#review47959 --- Ship it! Looks good to me. - Aleix Pol Gonzalez On Jan

Review Request 115207: Improve integration QCommandLineParser - KAboutData

2014-01-21 Thread Aleix Pol Gonzalez
/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 115213: Remove KDE4_CREATE_HTML_HANDBOOK

2014-01-22 Thread Aleix Pol Gonzalez
ported because I ported them. OTOH, there will be non-ported applications, that's why we provide this warning. - Aleix Pol Gonzalez On Jan. 22, 2014, 7:01 a.m., David Narváez wrote: --- This is an automatically generated e-mail

Re: Review Request 115211: Mark target created by ecm_add_test as non GUI by default

2014-01-22 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115211/#review47996 --- Ship it! Ship It! - Aleix Pol Gonzalez On Jan. 22, 2014

Re: Review Request 115198: Fix KDE4Support compilation

2014-01-22 Thread Aleix Pol Gonzalez
it. I'm unsure what's the best way though. Diffs - src/kdeui/kapplication.cpp 5a7f4c8 Diff: https://git.reviewboard.kde.org/r/115198/diff/ Testing --- Builds. Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde

Re: Review Request 115213: Remove KDE4_CREATE_HTML_HANDBOOK

2014-01-22 Thread Aleix Pol Gonzalez
On Jan. 22, 2014, 11:32 a.m., Aleix Pol Gonzalez wrote: Well, but frameworks are not only for frameworks. They're all ported because I ported them. OTOH, there will be non-ported applications, that's why we provide this warning. Luigi Toscano wrote: You ported

Re: Review Request 115213: Remove KDE4_CREATE_HTML_HANDBOOK

2014-01-22 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115213/#review48003 --- Ship it! Ship It! - Aleix Pol Gonzalez On Jan. 22, 2014

Re: Review Request 115211: Mark target created by ecm_add_test as non GUI by default

2014-01-22 Thread Aleix Pol Gonzalez
On Jan. 22, 2014, 12:08 p.m., Alex Merry wrote: This seems sensible to me; however, I do wonder if ECM should also provide an ecm_mark_gui_executable function as well (I'm thinking of the case where most of the tests should be non-gui, but a handful want to display widgets). Well, we

Re: Review Request 115211: Mark target created by ecm_add_test as non GUI by default

2014-01-22 Thread Aleix Pol Gonzalez
Pol Gonzalez wrote: Well, we can't change the default, can we? Alex Merry wrote: I don't understand what you mean. If we have a /mark as non gui/ function is because executables are /gui executables/ by default. Having an ecm_mark_gui_executable() would make this weird I'd say. (TBH

Re: Review Request 115238: Bug in KDEPlatformFileDialogHelper(?) causes selectFile not to work in QFileDialog

2014-01-22 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/115238/#comment34038 no need for this debug. Thanks for improving the test, will have to look into the implementation. :) - Aleix Pol Gonzalez On Jan. 22, 2014, 10:26 p.m., Gregor Mi wrote: --- This is an automatically

Re: Review Request 115028: Allow the building of deprecated code to be disabled

2014-01-26 Thread Aleix Pol Gonzalez
they already are deprecated in kdelibs 4. I don't really like disabling compilation of deprecated symbols, especially in this case we're not winning that much. - Aleix Pol Gonzalez On Jan. 15, 2014, 1:56 p.m., Alex Merry wrote

Re: Review Request 114997: Improve KAuth README.md

2014-01-26 Thread Aleix Pol Gonzalez
/#comment34187 Need to use? link against? Also the (or similar) looks unsure. I would say: If you are using cmake, you can find KAuth by using: find_package(KF5Auth NO_MODULE) or finding KF5 with the Auth component, from your CMake scripts. - Aleix Pol Gonzalez On Jan. 26

Re: Review Request 115148: Add KWINDOWSYSTEM_ namespace to HAVE_FOO defines

2014-01-26 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115148/#review48326 --- Ship it! Makes sense to me. - Aleix Pol Gonzalez On Jan

Re: Review Request 115316: Add demo for KRecentFileList

2014-01-26 Thread Aleix Pol Gonzalez
the developer that he did it right, but I wouldn't see it as documentation. I would rename it to KRecentFilesActionTest - Aleix Pol Gonzalez On Jan. 25, 2014, 10:56 p.m., Gregor Mi wrote: --- This is an automatically generated e-mail

Re: Review Request 115207: Improve integration QCommandLineParser - KAboutData

2014-01-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115207/#review48347 --- On Jan. 21, 2014, 11:36 p.m., Aleix Pol Gonzalez wrote

Re: Review Request 115361: use renamed kmailservice5

2014-01-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115361/#review48489 --- Ship it! Ship It! - Aleix Pol Gonzalez On Jan. 28, 2014

Re: Review Request 115362: Do not explicitly link against libc

2014-01-28 Thread Aleix Pol Gonzalez
really ugly to have -lc so +1 from me. - Aleix Pol Gonzalez On Jan. 28, 2014, 4:41 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115362

Re: Review Request 115382: Remove unused dependencies

2014-01-29 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115382/#review48548 --- Ship it! Cool :) - Aleix Pol Gonzalez On Jan. 29, 2014, 3

Re: Review Request 115411: Remove obsolete CMake code from pre-splitting

2014-01-31 Thread Aleix Pol Gonzalez
frameworks, actually. All this checking if it's being built out of kdelibs doesn't make sense anymore. - Aleix Pol Gonzalez On Jan. 31, 2014, 12:03 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 115421: Clean up the CMakeLists.txt files

2014-02-03 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115421/#review48815 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 1, 2014, 1

Re: Review Request 115477: Remove commands that just set variables to their defaults

2014-02-04 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115477/#review48995 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 4, 2014, 4

Re: Review Request 115472: fix test for icon path

2014-02-04 Thread Aleix Pol Gonzalez
://git.reviewboard.kde.org/r/115472/#comment34595 You probably want endsWith(/.icons). - Aleix Pol Gonzalez On Feb. 4, 2014, 4:30 p.m., Jonathan Riddell wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 115472: fix test for icon path

2014-02-05 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115472/#review49019 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 5, 2014

Re: Review Request 115525: Deprecate slots in KCompletion and convert into slots the methods they call

2014-02-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115525/#review49161 --- Ship it! Looks good to me - Aleix Pol Gonzalez On Feb. 6

Re: Review Request 115530: Find Qt5::X11Extras only if X11 is found

2014-02-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115530/#review49177 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 7, 2014, 8

Re: Review Request 115616: Add platform to qt options in KCmdLineArgs

2014-02-10 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115616/#review49439 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 10, 2014

Re: Review Request 115629: Port DrKonqi to QCommandLineParser and QCommandLineOption

2014-02-10 Thread Aleix Pol Gonzalez
/#comment34891 You can instantiate QApplication in the stack, instead of calling new+delete. Also you probably want to create it in the beginning of the main function body. - Aleix Pol Gonzalez On Feb. 10, 2014, 4:06 p.m., Martin Gräßlin wrote

Re: Review Request 115629: Port DrKonqi to QCommandLineParser and QCommandLineOption

2014-02-10 Thread Aleix Pol Gonzalez
On Feb. 10, 2014, 4:14 p.m., Aleix Pol Gonzalez wrote: drkonqi/main.cpp, line 74 https://git.reviewboard.kde.org/r/115629/diff/1/?file=243089#file243089line74 You can instantiate QApplication in the stack, instead of calling new+delete. Also you probably want

Re: Review Request 115681: kquitapp - kquitapp5

2014-02-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115681/#review49609 --- Ship it! - Aleix Pol Gonzalez On Feb. 11, 2014, 9:04 p.m

Re: Review Request 115684: Generate local forwarding headers under a local subdir, to fix clash on Mac OS X.

2014-02-11 Thread Aleix Pol Gonzalez
it as a problem. This could break compilation on some projects other than KParts, will you be able to try the rest of the modules? Thanks for figuring it out! - Aleix Pol Gonzalez On Feb. 11, 2014, 10:15 p.m., David Faure wrote

Re: Review Request 115683: Install app desktop files to share/applications, not in a kde5 subdir

2014-02-11 Thread Aleix Pol Gonzalez
benefits of name-spacing the desktop files besides feeling different and special. :) - Aleix Pol Gonzalez On Feb. 11, 2014, 9:56 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 115695: Rework KNotification to work without KNotify daemon

2014-02-19 Thread Aleix Pol Gonzalez
On Feb. 19, 2014, 10:06 a.m., Aleix Pol Gonzalez wrote: src/knotificationmanager.cpp, line 180 https://git.reviewboard.kde.org/r/115695/diff/3/?file=243841#file243841line180 ? Martin Klapetek wrote: Yet once again the description xD - it's full of ... FIXMEs to indicate

Re: Review Request 115634: Add kconfig_compiler autotest that checks whether signals are emitted

2014-02-20 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115634/#review50388 --- Ship it! Ship It! - Aleix Pol Gonzalez On Feb. 20, 2014

Re: Review Request 115207: Improve integration QCommandLineParser - KAboutData

2014-02-21 Thread Aleix Pol Gonzalez
to duplicate information by passing it to the KAboutData _and_ the QApplication. Diffs (updated) - src/lib/kaboutdata.h c9e src/lib/kaboutdata.cpp c347521 Diff: https://git.reviewboard.kde.org/r/115207/diff/ Testing --- Thanks, Aleix Pol Gonzalez

Re: Review Request 115207: Improve integration QCommandLineParser - KAboutData

2014-02-21 Thread Aleix Pol Gonzalez
/kaboutdata.h c9e src/lib/kaboutdata.cpp c347521 Diff: https://git.reviewboard.kde.org/r/115207/diff/ Testing --- Thanks, Aleix Pol Gonzalez ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman

Re: Review Request 116462: Keep KSharedConfigPtr(kdeglobals) around in KHintSettings.

2014-03-02 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116462/#review51729 --- Ship it! Makes sense to me. - Aleix Pol Gonzalez On Feb

Re: Review Request 116540: Add configuration for ReviewBoard

2014-03-02 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116540/#review51731 --- Ship it! Ship It! - Aleix Pol Gonzalez On March 2, 2014

<    1   2   3   4   5   6   7   8   9   10   >