Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
Hi Alex & Ben, On 26 May 2014, at 11:04 , Alex Merry wrote: > -DCMAKE_INSTALL_BUNDLEDIR=some/relative/path thanks to your hint I was able to insert a temporary workaround here on my CI system by supplying an additional configuration file for kconfig as this: --- $ cat ~/scripts/config/build/kco

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Marko Käning
On 27 May 2014, at 07:29 , Ben Cooksley wrote: > configurePlatformArgs=-DCMAKE_INSTALL_BUNDLEDIR="{installPrefix}/Applications” I ran into stg I had seen with my own trials to access the installPrefix variable via %(*)s: --- Traceback (most recent call last): File "tools/perform-build.py", lin

Re: Loading Qt 5 translations

2014-05-26 Thread Alexander Potashev
2014-05-26 18:17 GMT+04:00 Aurélien Gâteau : > Not sure where we could put this. Frameworks load their translations, I > believe it should be up to Qt to load its own translations. The only > framework where it could maybe make sense to add such feature is in > KI18n, but even there it feels a bit

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Marko Käning
On 27 May 2014, at 07:29 , Ben Cooksley wrote: > Please try the following syntax instead. > Note that I recommend you override this in > config/build/darwin-mavericks.cfg instead to ensure all CMake projects > on OS X are affected by it. > > configurePlatformArgs=-DCMAKE_INSTALL_BUNDLEDIR="{inst

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Ben Cooksley
On Mon, May 26, 2014 at 9:04 PM, Alex Merry wrote: > On Monday 26 May 2014 19:41:33 Ben Cooksley wrote: >> On Mon, May 26, 2014 at 7:38 PM, wrote: >> > On 26 May 2014, at 09:35 , Ben Cooksley wrote: >> >> For reasons stated above, application packages cannot reside within >> >> /Applications. >

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Ben Cooksley
On Tue, May 27, 2014 at 6:37 AM, wrote: > Hi Alex & Ben, Hi Marko, > > On 26 May 2014, at 11:04 , Alex Merry wrote: >> -DCMAKE_INSTALL_BUNDLEDIR=some/relative/path > > thanks to your hint I was able to insert a temporary workaround here on my CI > system > by supplying an additional configura

Review Request 118340: Allow the kactivitymanagerd daemon to be disabled.

2014-05-26 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118340/ --- Review request for KDE Frameworks, Plasma and Ivan Čukić. Repository: kac

Re: Review Request 118339: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Thomas Braxton
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118339/ --- (Updated May 27, 2014, 4:28 a.m.) Status -- This change has been mar

Re: Review Request 118339: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118339/#review58534 --- This review has been submitted with commit 4bea223cd2cab4e4ba

Re: Review Request 118339: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118339/#review58533 --- Ship it! Ship It! - Matthew Dawson On May 26, 2014, 11:36

Review Request 118339: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Thomas Braxton
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118339/ --- Review request for KDE Frameworks and Matthew Dawson. Repository: kconfig

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/ --- (Updated May 27, 2014, midnight) Review request for KDE Frameworks, KDE U

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/ --- (Updated May 26, 2014, 11:59 p.m.) Review request for KDE Frameworks, KDE

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/ --- (Updated May 26, 2014, 11:57 p.m.) Review request for KDE Frameworks, KDE

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
> On May 26, 2014, 10:18 a.m., Christoph Feck wrote: > > The original screen shot looks bad, because the bold font looks smaller > > than the normal font. This might be because of wrong hinting settings, or > > because of a bad font design. > > > > Regarding the scaling factor, CSS recommends

Re: Review Request 118332: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118332/#review58509 --- This review has been submitted with commit 8b88eaee8c2db96779

Re: Review Request 118332: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Thomas Braxton
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118332/ --- (Updated May 26, 2014, 7:59 p.m.) Status -- This change has been mar

Re: Review Request 118332: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118332/#review58506 --- Ship it! Ship It! - Matthew Dawson On May 26, 2014, 2:38 p

Review Request 118332: Update usage of QVariant::type() for Qt5.

2014-05-26 Thread Thomas Braxton
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118332/ --- Review request for KDE Frameworks and Matthew Dawson. Repository: kconfig

Re: Review Request 118216: Skip non-directory files inside the po/ directory

2014-05-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118216/#review58484 --- This review has been submitted with commit 93fe5c4acbb6498821

Re: Review Request 118216: Skip non-directory files inside the po/ directory

2014-05-26 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118216/ --- (Updated May 26, 2014, 2:18 p.m.) Status -- This change has been mar

Re: Loading Qt 5 translations

2014-05-26 Thread Aurélien Gâteau
On Sun, May 25, 2014, at 2:05, Albert Astals Cid wrote: > El Dissabte, 17 de maig de 2014, a les 18:21:16, Lasse Liehu va escriure: > > Hi > > > > Translations for frameworks are loaded automatically when they are > > used, but what about Qt 5 itself? Are those translations loaded and if > > not,

Jenkins build is back to stable : kio_master_qt5 #241

2014-05-26 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Christoph Feck
> On May 26, 2014, 10:18 a.m., Christoph Feck wrote: > > The original screen shot looks bad, because the bold font looks smaller > > than the normal font. This might be because of wrong hinting settings, or > > because of a bad font design. > > > > Regarding the scaling factor, CSS recommends

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/#review58454 --- The original screen shot looks bad, because the bold font look

Re: Review Request 118039: Simplify KConfig::readEntry/writeEntry

2014-05-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118039/#review58453 --- This review has been submitted with commit 5f4dc2973f10cd44b2

Re: Review Request 118039: Simplify KConfig::readEntry/writeEntry

2014-05-26 Thread Thomas Braxton
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118039/ --- (Updated May 26, 2014, 10:12 a.m.) Status -- This change has been ma

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/ --- (Updated May 26, 2014, 10 a.m.) Review request for KDE Frameworks, KDE Us

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
> On May 24, 2014, 1:57 a.m., Mark Gaiser wrote: > > src/ktitlewidget.cpp, line 44 > > > > > > Can you at least make this a bit less "magical"? Right now it's just > > "size * 1.3".. Why 1.3? What is the basis be

Re: Review Request 118289: Change titlewidget from bold to increased font size

2014-05-26 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118289/ --- (Updated May 26, 2014, 9:58 a.m.) Review request for KDE Frameworks and C

Re: Review Request 118276: give kwrite the proper dbus name in desktop file

2014-05-26 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118276/ --- (Updated May 26, 2014, 9:22 a.m.) Status -- This change has been mar

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Alex Merry
On Monday 26 May 2014 19:41:33 Ben Cooksley wrote: > On Mon, May 26, 2014 at 7:38 PM, wrote: > > On 26 May 2014, at 09:35 , Ben Cooksley wrote: > >> For reasons stated above, application packages cannot reside within > >> /Applications. > >> If they need to reside within a separate directory, we

Re: Review Request 118323: Improve metainfo.yaml for KWindowSystem

2014-05-26 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118323/#review58448 --- Ship it! Looks correct. I suggest running the validate_metain

Re: Review Request 118234: [frameworksintegration] Ensure the xcb connection gets flushed before the event dispatcher blocks

2014-05-26 Thread Martin Gräßlin
> On May 26, 2014, 10:11 a.m., Àlex Fiestas wrote: > > src/platformtheme/CMakeLists.txt, line 4 > > > > > > This workaround is quite important for 5.3.0 and older at least, maybe > > in those cases we should make

Re: Review Request 118234: [frameworksintegration] Ensure the xcb connection gets flushed before the event dispatcher blocks

2014-05-26 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118234/#review58445 --- src/platformtheme/CMakeLists.txt

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
Hi Allen, On 25 May 2014, at 00:43 , Allen Winter wrote: > I have local patches to commit yet that fixes some of the issues you mention > below. great! I am looking forward to hear from you. Peeking into Harald Fernengel’s Homebrew recipe for kauth [1] I found out that there seems to be no need

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
Hi Alex, On 24 May 2014, at 14:57 , Alex Merry wrote: > Where *is* kconfig_compiler_kf5 installed? thanks again for pointing that out! I’ve found it here: --- $ find /opt/kde -name kconfig_compiler_kf5.app $ find /Users/marko/WC/KDECI-builds/ -name kconfig_compiler_kf5.app /Users/marko/WC/KDECI

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
I see now that Harald installed in the application package below PREFIX/lib/kde5/libexec/ [1]. Here on the CI system I already do find an executable in an equivalent folder: --- $ ls -l /opt/kde/install/darwin/mavericks/clang/kf5-qt5/frameworks/kconfig/inst/lib/libexec/kf5 total 136 -rwxr-xr-x 1

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
Hi Ben, On 26 May 2014, at 09:03 , Ben Cooksley wrote: > This is because it was installed to /Applications/ within the > installation jail (located at > /Users/marko/WC/KDECI-builds/kconfig/local-inst in this instance). Yep. > I've no idea how OS X handles applications/executables - but a > solu

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
Hi Luigi, On 24 May 2014, at 17:10 , Luigi Toscano wrote: > Maybe, but there is a reason if it was discarded; it's not the "final" yes, because that patch doesn’t work anymore with the current file. But it shouldn’t be hard to come up with a new patch. Greets, Marko ___

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread mk-lists
On 26 May 2014, at 09:35 , Ben Cooksley wrote: > For reasons stated above, application packages cannot reside within > /Applications. > If they need to reside within a separate directory, we'll need to > arrange this - however it needs to be within the installation prefix. Well, sure, I don’t wan

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Ben Cooksley
On Mon, May 26, 2014 at 7:38 PM, wrote: > On 26 May 2014, at 09:35 , Ben Cooksley wrote: >> For reasons stated above, application packages cannot reside within >> /Applications. >> If they need to reside within a separate directory, we'll need to >> arrange this - however it needs to be within t

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Ben Cooksley
On Mon, May 26, 2014 at 7:32 PM, wrote: > Hi Ben, Hi Marko, > > On 26 May 2014, at 09:03 , Ben Cooksley wrote: >> This is because it was installed to /Applications/ within the >> installation jail (located at >> /Users/marko/WC/KDECI-builds/kconfig/local-inst in this instance). > Yep. > >> I'v

Review Request 118323: Improve metainfo.yaml for KWindowSystem

2014-05-26 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118323/ --- Review request for KDE Frameworks and Alex Merry. Repository: kwindowsyst

Re: Review Request 118252: Port KWindowSystem::minimizeWindow and unminizeWindow to XCB

2014-05-26 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118252/ --- (Updated May 26, 2014, 9:09 a.m.) Review request for KDE Frameworks, kwin

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-26 Thread Ben Cooksley
On Mon, May 26, 2014 at 3:59 AM, wrote: > Hi Alex, > > On 24 May 2014, at 14:57 , Alex Merry wrote: >> Where *is* kconfig_compiler_kf5 installed? > > thanks again for pointing that out! > > I’ve found it here: > --- > $ find /opt/kde -name kconfig_compiler_kf5.app > $ find /Users/marko/WC/KDECI-