Re: Plasma 5 Beta 2 tars
On Fri, Jun 06, 2014 at 11:24:50PM +0200, šumski wrote: Other then these issues present in tarball only (not git), there is an ongoing problem with plasma-desktop which blocks creating proper 32bit packages: http://paste.opensuse.org/25338349 I'm afraid I can't recreate this, it compiles both locally and in the build server find for i386 https://launchpad.net/~kubuntu-ppa/+archive/next/+build/6089974 Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Wednesday 11 June 2014, šumski wrote: Unfortunately, we cannot touch the buildhost, only sources which are built (and i'm running x86_64 OS). You might be right about kde_file, though seems to be a namespace clash/mixup. The attached patch resolves the failure here (basically renaming stat method to statFont) Might not be the best way to go forward, but might provide some clues what's wrong. Hrvoje, I think that patch is the way to go, can you push it? In brief: It seems to be a moc problem (that unfortunately i cannot reproduce the same fail in a simple test case) the file sys/stat.h in included that file redefines stat as stat64 when in 32 bits and LARGEFILE64_SOURCE is defined. moc, seeing there is a #define stat stat64 replaces it everywhere in the generated moc file, even where it shouldn't -- Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Thursday 12 of June 2014 14:53:57 Marco Martin wrote: On Wednesday 11 June 2014, šumski wrote: Unfortunately, we cannot touch the buildhost, only sources which are built (and i'm running x86_64 OS). You might be right about kde_file, though seems to be a namespace clash/mixup. The attached patch resolves the failure here (basically renaming stat method to statFont) Might not be the best way to go forward, but might provide some clues what's wrong. Hrvoje, I think that patch is the way to go, can you push it? Sure, i was only concerned about having two similar names, before where FontStat and stat methods, now would be FontStat and StatFont =) Cheers, Hrvoje signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
[CC:ing frameworks-devel, hoping for additional input] On Friday, June 06, 2014 23:24:50 šumski wrote: On Friday 06 of June 2014 21:32:48 Eric Hameleers wrote: On Fri, 6 Jun 2014, ?umski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. khotkeys also don't build, they have one ecm_optional_add_subdirectory(doc) too many ;-) Cheers, Hrvoje I got that error too in khelpcenter kinfocenter and kio-extras. That's where I stopped trying to compile this until the tarballs get fixed. Other then these issues present in tarball only (not git), there is an ongoing problem with plasma-desktop which blocks creating proper 32bit packages: http://paste.opensuse.org/25338349 build succeeds on 64bit builds. with our daily packages we've just used: sed -i 's|add_subdirectory( dbus )|#add_subdirectory( dbus )|g' kcms/kfontinst/CMakeLists.txt but this cannot go in distributions... It seems that the #defines in frameworks/kdelibs4support/src/kdecore/kde_file.h get mixed up, but I have to say I'm not portability guy enough to know which defines should trigger when. Input is most welcome here... Hrvoje, Jonathan: Could you perhaps try modifying it there locally, and see if the build succeeds then? (Should be enough to make sure the #else /* unix */ branch is evaluated, instead of the ones that point the KDE_lstat, KDE_stat, etc. to *64 versions. Cheers, -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Wednesday 11 of June 2014 17:07:04 Sebastian Kügler wrote: [CC:ing frameworks-devel, hoping for additional input] On Friday, June 06, 2014 23:24:50 šumski wrote: On Friday 06 of June 2014 21:32:48 Eric Hameleers wrote: On Fri, 6 Jun 2014, ?umski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. khotkeys also don't build, they have one ecm_optional_add_subdirectory(doc) too many ;-) Cheers, Hrvoje I got that error too in khelpcenter kinfocenter and kio-extras. That's where I stopped trying to compile this until the tarballs get fixed. Other then these issues present in tarball only (not git), there is an ongoing problem with plasma-desktop which blocks creating proper 32bit packages: http://paste.opensuse.org/25338349 build succeeds on 64bit builds. with our daily packages we've just used: sed -i 's|add_subdirectory( dbus )|#add_subdirectory( dbus )|g' kcms/kfontinst/CMakeLists.txt but this cannot go in distributions... It seems that the #defines in frameworks/kdelibs4support/src/kdecore/kde_file.h get mixed up, but I have to say I'm not portability guy enough to know which defines should trigger when. Input is most welcome here... Hrvoje, Jonathan: Could you perhaps try modifying it there locally, and see if the build succeeds then? (Should be enough to make sure the #else /* unix */ branch is evaluated, instead of the ones that point the KDE_lstat, KDE_stat, etc. to *64 versions. Unfortunately, we cannot touch the buildhost, only sources which are built (and i'm running x86_64 OS). You might be right about kde_file, though seems to be a namespace clash/mixup. The attached patch resolves the failure here (basically renaming stat method to statFont) Might not be the best way to go forward, but might provide some clues what's wrong. Cheers, Hrvoje Cheers, diff --git a/kcms/kfontinst/dbus/FontInst.cpp b/kcms/kfontinst/dbus/FontInst.cpp index 637d7a2..42413c1 100644 --- a/kcms/kfontinst/dbus/FontInst.cpp +++ b/kcms/kfontinst/dbus/FontInst.cpp @@ -157,7 +157,7 @@ void FontInst::list(int folders, int pid) emit fontList(pid, fonts); } -void FontInst::stat(const QString name, int folders, int pid) +void FontInst::statFont(const QString name, int folders, int pid) { KFI_DBUG name folders pid; diff --git a/kcms/kfontinst/dbus/FontInst.h b/kcms/kfontinst/dbus/FontInst.h index 1365efd..01183db 100644 --- a/kcms/kfontinst/dbus/FontInst.h +++ b/kcms/kfontinst/dbus/FontInst.h @@ -104,7 +104,7 @@ class KFONTINST_EXPORT FontInst : public QObject public Q_SLOTS: Q_NOREPLYvoidlist(int folders, int pid); -Q_NOREPLYvoidstat(const QString font, int folders, int pid); +Q_NOREPLYvoidstatFont(const QString font, int folders, int pid); Q_NOREPLYvoidinstall(const QString file, bool createAfm, bool toSystem, int pid, bool checkConfig); Q_NOREPLYvoiduninstall(const QString family, quint32 style, bool fromSystem, int pid, bool checkConfig); Q_NOREPLYvoiduninstall(const QString name, bool fromSystem, int pid, bool checkConfig); diff --git a/kcms/kfontinst/dbus/FontinstIface.cpp b/kcms/kfontinst/dbus/FontinstIface.cpp index e27cbbb..174ac51 100644 --- a/kcms/kfontinst/dbus/FontinstIface.cpp +++ b/kcms/kfontinst/dbus/FontinstIface.cpp @@ -1,8 +1,8 @@ /* - * This file was generated by qdbusxml2cpp version 0.7 + * This file was generated by qdbusxml2cpp version 0.8 * Command line was: qdbusxml2cpp -m -p FontinstIface org.kde.fontinst.xml -i Family.h * - * qdbusxml2cpp is Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies). + * qdbusxml2cpp is Copyright (C) 2014 Digia Plc and/or its subsidiary(-ies). * * This is an auto-generated file. * This file may have been hand-edited. Look for HAND-EDIT comments diff --git a/kcms/kfontinst/dbus/FontinstIface.h b/kcms/kfontinst/dbus/FontinstIface.h index 2a59cbe..cb1b9ce 100644 --- a/kcms/kfontinst/dbus/FontinstIface.h +++ b/kcms/kfontinst/dbus/FontinstIface.h @@ -1,15 +1,15 @@ /* - * This file was generated by qdbusxml2cpp version 0.7 + * This file was generated by qdbusxml2cpp version 0.8 * Command line was: qdbusxml2cpp -m -p FontinstIface org.kde.fontinst.xml -i Family.h * - * qdbusxml2cpp is Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies). + * qdbusxml2cpp is Copyright (C) 2014 Digia Plc and/or its subsidiary(-ies). * * This is an auto-generated file. * Do not edit! All changes made to it will be lost. */ -#ifndef FONTINSTIFACE_H -#define FONTINSTIFACE_H +#ifndef FONTINSTIFACE_H_1402519768 +#define FONTINSTIFACE_H_1402519768 #include QtCore/QObject #include QtCore/QByteArray @@ -40,56 +40,56 @@ public Q_SLOTS: // METHODS
Re: Plasma 5 Beta 2 tars
Last minute update to fix issues in the translation files. Alas I found no fix for the reported problem compiling on i386 http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/plasma-desktop-4.97.0.tar.xz fefbfa73eace6fe85c0f401605bcb9695389b73ac03a9558cff1cbb9d04e8155 plasma-desktop-4.97.0.tar.xz appearing on depot soon and should be available for download shortly after Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Tuesday, June 10, 2014 22:10:19 Jonathan Riddell wrote: Last minute update to fix issues in the translation files. Alas I found no fix for the reported problem compiling on i386 Could you send a log here of the build failure on i386? -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Tue, Jun 10, 2014 at 11:14:26PM +0200, Sebastian Kügler wrote: On Tuesday, June 10, 2014 22:10:19 Jonathan Riddell wrote: Last minute update to fix issues in the translation files. Alas I found no fix for the reported problem compiling on i386 Could you send a log here of the build failure on i386? See earlier e-mail in thread from hrvoje.sen...@gmail.com http://paste.opensuse.org/25338349 Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
New tars to fix cmake issue 5676822dd981f2b8035868cd6f6b90162ede52ac300d2cfbf3810fb7957e7851 khotkeys-4.97.0.tar.xz 6e75432104f34b9538a99461fa29bc10d0e6f17bf5fa835ac2d02c5718836f29 khelpcenter-4.97.0.tar.xz 2c6298b94f5a98982cb43779c3d1843b88b561050b60324a83975909cef8f8c5 kinfocenter-4.97.0.tar.xz a150e5279301cb787190e20d9256d4739846267e46a847108406345f2f405a9a kio-extras-4.97.0.tar.xz Up at http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/ and coming soon to depot. Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
Zitat von Martin Gräßlin mgraess...@kde.org: can we revert that change in plasma-workspace for the release? I would not want to go back further as plasma-workspace also has quite some changes which would be good to be tested. E.g. the new lockscreen UI. Hi, okay, go ahead and revert till after the release. My attempts to prophylactically fix build due to my change caused it to fail. Sorry about that :( Cheers, Kai Uwe ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Fri, Jun 06, 2014 at 11:11:05AM +0200, Mario Fux wrote: Why appending some packages with -kf5 and some with 5? Seems quite confusing. And baloo and milou are not really (or not yet for baloo) KDE Frameworks. See if it all they should be appended with a 5 and the term kf5 should be kept for KDE Frameworks. Just the convention I've been using, it may well not make sense. kfilemetadata source becomes kfilemetadata5 because it's a library and 5 is the new soversion. baloo is more than a library so I added kf5 because it's the version of baloo which uses KF5. Same for milou. Thinking about it now I think milou isn't expected to be co-installable any more than Plasma Desktop so I don't think it needs a rename. Baloo isn't co-installable except the libraries so there's no reason it should be treated differently than kfilemetadata. I'll make new tars. Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
New tars up at http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/ changes: plasma-workspace has changed API commit reverted milou-kf5 tar now milou baloo-kf5 tar now baloo5 oxygen-fonts now using correct branch plasma-workspace;master;6903e1e94f4b5a0ab15a19bfe77d6b362b976280;plasma-workspace-4.97.0.tar.xz;b6e433d71eb4ac3453d1f6df42c7e275 milou;frameworks;5599a7dabb28f048af700f0202096c281ff3098d;milou-4.97.0.tar.xz;173f4ece0cf4d5832a4c7f74472baae8 baloo;frameworks;5b454c0f5662e9e5c1b669c45367bbe73c546aac;baloo5-4.97.0.tar.xz;529b4ecdcaea2501aeffceda1759d194 oxygen-fonts;release-0.4;556f299a8c562e4beaf72b60bd8e55e0647c8616;oxygen-fonts-0.3.97.tar.xz;1ecdc2af7be15425f76fa9b3308b85b5 Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Friday 06 of June 2014 07:33:09 Martin Gräßlin wrote: On Thursday 05 June 2014 20:19:55 šumski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. Plasma-workspace doesn't build. It is adjusted for new Solid API, which is not in KF5 4.100.0. (error: 'const class Solid::Battery' has no member named 'isPresent') can we revert that change in plasma-workspace for the release? I would not want to go back further as plasma-workspace also has quite some changes which would be good to be tested. E.g. the new lockscreen UI. I imagine similar issues with todays changes in kdeclarative and plasma- framework. Maybe it would be best to tag from yesterday evening? no that should be fine. We had only changes in kdeclarative and plasma- framework and they should not have propagated further down. ok. in this case i meant on runtime, as i noticed changes in some plasmoids, e.g. notifications, and some other one. Cheers, Hrvoje Cheers Martin signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. khotkeys also don't build, they have one ecm_optional_add_subdirectory(doc) too many ;-) Cheers, Hrvoje signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Fri, 6 Jun 2014, ?umski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. khotkeys also don't build, they have one ecm_optional_add_subdirectory(doc) too many ;-) Cheers, Hrvoje I got that error too in khelpcenter kinfocenter and kio-extras. That's where I stopped trying to compile this until the tarballs get fixed. Cheers, Eric -- Eric Hameleers al...@slackware.com Jabber: al...@jabber.xs4all.nl ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Friday 06 of June 2014 21:32:48 Eric Hameleers wrote: On Fri, 6 Jun 2014, ?umski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. khotkeys also don't build, they have one ecm_optional_add_subdirectory(doc) too many ;-) Cheers, Hrvoje I got that error too in khelpcenter kinfocenter and kio-extras. That's where I stopped trying to compile this until the tarballs get fixed. Other then these issues present in tarball only (not git), there is an ongoing problem with plasma-desktop which blocks creating proper 32bit packages: http://paste.opensuse.org/25338349 build succeeds on 64bit builds. with our daily packages we've just used: sed -i 's|add_subdirectory( dbus )|#add_subdirectory( dbus )|g' kcms/kfontinst/CMakeLists.txt but this cannot go in distributions... Cheers, Hrvoje signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Plasma 5 Beta 2 tars
Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. Appearing here soon http://download.kde.org/unstable/plasma/4.97.0/src/ Temporarily here in the mean time http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/ md5sums http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/4.97.0-release-data I've renamed baloo and milou to have -kf5 in the tar name as you may well want the kdelibs4 versions around. Also kfilemetadata as kfilemetadata5. Jonathan ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On June 5, 2014 03:47:59 PM Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I've renamed baloo and milou to have -kf5 in the tar name as you may well want the kdelibs4 versions around. Also kfilemetadata as kfilemetadata5. Regarding this, for source based distributions baloo/kfilemetadata can't be co-installed with there kdelibs4 version as the development files conflict, the main problem being the CMake files. I posted an RR here: https://git.reviewboard.kde.org/r/118512/ to try and fix that. Is this acceptable? Otherwise Plasma Next and the rest of KDE SC 4.x can't really be co-installed (especially KDEPIM!) Thanks, -- Matthew smime.p7s Description: S/MIME cryptographic signature ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Plasma 5 Beta 2 tars
On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. Plasma-workspace doesn't build. It is adjusted for new Solid API, which is not in KF5 4.100.0. (error: 'const class Solid::Battery' has no member named 'isPresent') I imagine similar issues with todays changes in kdeclarative and plasma- framework. Maybe it would be best to tag from yesterday evening? Cheers, Hrvoje Appearing here soon http://download.kde.org/unstable/plasma/4.97.0/src/ Temporarily here in the mean time http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/ md5sums http://starsky.19inch.net/~jr/tmp/plasma-4.97.0/4.97.0-release-data I've renamed baloo and milou to have -kf5 in the tar name as you may well want the kdelibs4 versions around. Also kfilemetadata as kfilemetadata5. Jonathan ___ Kde-packager mailing list kde-packa...@kde.org https://mail.kde.org/mailman/listinfo/kde-packager signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Re: Plasma 5 Beta 2 tars
On Thursday 05 June 2014 20:19:55 šumski wrote: On Thursday 05 of June 2014 16:47:59 Jonathan Riddell wrote: Tars are up for Plasma 5 Beta 2, please try them out and let me know of problems. I'd especially like to know if translations successfully install as I noticed some not doing so last time. Plasma-workspace doesn't build. It is adjusted for new Solid API, which is not in KF5 4.100.0. (error: 'const class Solid::Battery' has no member named 'isPresent') can we revert that change in plasma-workspace for the release? I would not want to go back further as plasma-workspace also has quite some changes which would be good to be tested. E.g. the new lockscreen UI. I imagine similar issues with todays changes in kdeclarative and plasma- framework. Maybe it would be best to tag from yesterday evening? no that should be fine. We had only changes in kdeclarative and plasma- framework and they should not have propagated further down. Cheers Martin signature.asc Description: This is a digitally signed message part. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel