from:"Marko Käning"

[kmymoney4] [Bug 372453] KMyMoney 4.8 crashes on opening some files

2016-11-16 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=372453

--- Comment #9 from Marko Käning <mk-li...@mailbox.org> ---
(In reply to Ralf Habacker from comment #8)
> I guess checking the related functionality is not covered by a unit 
> test running on the CI.

Sorry for the noise, then. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 370227] Crash on exit

2016-11-15 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #14 from Marko Käning <mk-li...@mailbox.org> ---
*** Bug 364725 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 364725] Crashes when quitting the application while online-banking update dialog is open

2016-11-15 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=364725

Marko Käning <mk-li...@mailbox.org> changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from Marko Käning <mk-li...@mailbox.org> ---


*** This bug has been marked as a duplicate of bug 370227 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 372453] KMyMoney 4.8 crashes on opening some files

2016-11-15 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=372453

Marko Käning <mk-li...@mailbox.org> changed:

   What|Removed |Added

 CC||mk-li...@mailbox.org

--- Comment #7 from Marko Käning <mk-li...@mailbox.org> ---
KDE's CI system takes care of those checks. (Or, did I misunderstand you?)

-- 
You are receiving this mail because:
You are the assignee for the bug.

Re: Review Request 129393: Use Qt's plugin system instead of KService

2016-11-14 Thread Marko Käning



> On Nov. 14, 2016, 7:17 a.m., Marko Käning wrote:
> > Why do you migrate away from KService to something Qt'ish?
> 
> Christian David wrote:
> Because there is no use of KService for us. Also this is way simpler. The 
> new system is recommended by the Frameworks developers (if KService is not 
> needed).
> 
> The on demand plugins still use KService to find plugins.
> 
> Btw: This patch includes the KF5/Qt5 port of the plugin load system.

Thanks for clarifying this, Christian!


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129393/#review100826
---


On Nov. 13, 2016, 9:50 p.m., Christian David wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129393/
> ---
> 
> (Updated Nov. 13, 2016, 9:50 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> ---
> 
> Hi,
> 
> this is my long overdue change on the plugin system. It actually started in 
> Randa.
> 
> Sorry for such a long review request! I think it is ready for publishing, I 
> made this review request only to give everybody a chance to ask questions :)
> 
> 
> Commit 1
> 
> 
> This is a major change. Instead of a KPluginFactory based plugin
> loading, Qt's plugin system is used. The docu was adopeted to reflect
> this.
> 
> Commit 2
> 
> 
> The plugins for onlineTasks are now loaded with QPluginLoader on demand.
> The necessary information is encoded in Qt's json plugin header.
> 
> Commit 3
> 
> 
> Moved plugin loader out of kmm_plugin and put it into the executable
> 
> The plugin shared library is for plugins now and does not include the
> loading of plugins (which was kind of wired). All loading operations of
> standard plugins is done in KMyMoneyApp now.
> 
> Therefor the interface system for plugins needed do be reworked to
> great degree. Some overly complex operations were replaced by simpler
> ones. All plugins were adopted to that. Also it is more type safe now.
> 
> Now the user configuration which plugins should be loaded or not is
> respected (standard plugins only).
> 
> An old bug was fixed: If the configuration in the plugin settings
> changed the plugins are loaded/unloaded after the user accepts the
> dialog (perviously this was done when the checkbox was clicked).
> 
> Todos
> =
> 
> The new system is still not finished. However, it is a great step in
> the right direction. An uncomplete list of FIXMEs (or bugs?):
> 
> 1. pluginloader.{h,cpp} are gui only now and should be merged into the
> settings dialog.
> 2. The UI for plugins still has some issues, e.g. the online banking
> plugins which are loaded on demand (and are not deactivatable) have a
> checkbox for loading and unloading which has no function.
> 3. Unloading plugins may not work correctly (I think it never did).
> 4. The payeeIdentifiers are not listet in the plugin settings page.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt 09970dd5626e308c443ea20ee20ecf8503b21209 
>   kmymoney/CMakeLists.txt acbe6a3c91a62a15871a0061486a7d0771b3a5e6 
>   kmymoney/dialogs/konlinetransferform.h 
> e4761904d68a92471c0b05cc9e5b2bcd1461511d 
>   kmymoney/dialogs/konlinetransferform.cpp 
> 07e73ab17f93cd8fd804b9d11ede2b049f76eefe 
>   kmymoney/dialogs/settings/ksettingsplugins.cpp 
> 007a59d4be08fe779245dd5d52f664ebfcee447d 
>   kmymoney/kmymoney.h 8d10023b5e35be8f4449973ea16e0ec0cc0acca2 
>   kmymoney/kmymoney.cpp 043342dd87cbc03557c5e2b29eb3b9dfcd1083dd 
>   kmymoney/mymoney/onlinejobadministration.h 
> e72d1a6e46c7c8bcc7cf05193e04bdc8bd9103f5 
>   kmymoney/mymoney/onlinejobadministration.cpp 
> 52658747a44bc5477ccd75a531aef471b4553b19 
>   kmymoney/pluginloader.h PRE-CREATION 
>   kmymoney/pluginloader.cpp PRE-CREATION 
>   kmymoney/plugins/CMakeLists.txt fd7cad75256c0a43d7d1fd3a23b37cef7fab8bf9 
>   kmymoney/plugins/csvexport/CMakeLists.txt 
> 9f2e29fd43e32cd3c6acc7f597c45d522eac4eaf 
>   kmymoney/plugins/csvexport/csvexport.json.in PRE-CREATION 
>   kmymoney/plugins/csvexport/csvexporterplugin.h 
> f163ade2d59ae1b6eb32958a317384e77a658523 
>   kmymoney/plugins/csvexport/csvexporterplugin.cpp 
> 31027c65111edbad8ca78b00f5b7df572056bb37 
>   kmymoney/plugins/csvexport/kmm_csvexport.desktop 
> 957a9edc4101504e0db08378392e29358327b1c6 
>   kmymoney/plugins/csvimport/CMakeLists.txt 
&g

Re: Review Request 129393: Use Qt's plugin system instead of KService

2016-11-13 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129393/#review100826
---



Why do you migrate away from KService to something Qt'ish?

- Marko Käning


On Nov. 13, 2016, 9:50 p.m., Christian David wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129393/
> ---
> 
> (Updated Nov. 13, 2016, 9:50 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> ---
> 
> Hi,
> 
> this is my long overdue change on the plugin system. It actually started in 
> Randa.
> 
> Sorry for such a long review request! I think it is ready for publishing, I 
> made this review request only to give everybody a chance to ask questions :)
> 
> 
> Commit 1
> 
> 
> This is a major change. Instead of a KPluginFactory based plugin
> loading, Qt's plugin system is used. The docu was adopeted to reflect
> this.
> 
> Commit 2
> 
> 
> The plugins for onlineTasks are now loaded with QPluginLoader on demand.
> The necessary information is encoded in Qt's json plugin header.
> 
> Commit 3
> 
> 
> Moved plugin loader out of kmm_plugin and put it into the executable
> 
> The plugin shared library is for plugins now and does not include the
> loading of plugins (which was kind of wired). All loading operations of
> standard plugins is done in KMyMoneyApp now.
> 
> Therefor the interface system for plugins needed do be reworked to
> great degree. Some overly complex operations were replaced by simpler
> ones. All plugins were adopted to that. Also it is more type safe now.
> 
> Now the user configuration which plugins should be loaded or not is
> respected (standard plugins only).
> 
> An old bug was fixed: If the configuration in the plugin settings
> changed the plugins are loaded/unloaded after the user accepts the
> dialog (perviously this was done when the checkbox was clicked).
> 
> Todos
> =
> 
> The new system is still not finished. However, it is a great step in
> the right direction. An uncomplete list of FIXMEs (or bugs?):
> 
> 1. pluginloader.{h,cpp} are gui only now and should be merged into the
> settings dialog.
> 2. The UI for plugins still has some issues, e.g. the online banking
> plugins which are loaded on demand (and are not deactivatable) have a
> checkbox for loading and unloading which has no function.
> 3. Unloading plugins may not work correctly (I think it never did).
> 4. The payeeIdentifiers are not listet in the plugin settings page.
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt 09970dd5626e308c443ea20ee20ecf8503b21209 
>   kmymoney/CMakeLists.txt acbe6a3c91a62a15871a0061486a7d0771b3a5e6 
>   kmymoney/dialogs/konlinetransferform.h 
> e4761904d68a92471c0b05cc9e5b2bcd1461511d 
>   kmymoney/dialogs/konlinetransferform.cpp 
> 07e73ab17f93cd8fd804b9d11ede2b049f76eefe 
>   kmymoney/dialogs/settings/ksettingsplugins.cpp 
> 007a59d4be08fe779245dd5d52f664ebfcee447d 
>   kmymoney/kmymoney.h 8d10023b5e35be8f4449973ea16e0ec0cc0acca2 
>   kmymoney/kmymoney.cpp 043342dd87cbc03557c5e2b29eb3b9dfcd1083dd 
>   kmymoney/mymoney/onlinejobadministration.h 
> e72d1a6e46c7c8bcc7cf05193e04bdc8bd9103f5 
>   kmymoney/mymoney/onlinejobadministration.cpp 
> 52658747a44bc5477ccd75a531aef471b4553b19 
>   kmymoney/pluginloader.h PRE-CREATION 
>   kmymoney/pluginloader.cpp PRE-CREATION 
>   kmymoney/plugins/CMakeLists.txt fd7cad75256c0a43d7d1fd3a23b37cef7fab8bf9 
>   kmymoney/plugins/csvexport/CMakeLists.txt 
> 9f2e29fd43e32cd3c6acc7f597c45d522eac4eaf 
>   kmymoney/plugins/csvexport/csvexport.json.in PRE-CREATION 
>   kmymoney/plugins/csvexport/csvexporterplugin.h 
> f163ade2d59ae1b6eb32958a317384e77a658523 
>   kmymoney/plugins/csvexport/csvexporterplugin.cpp 
> 31027c65111edbad8ca78b00f5b7df572056bb37 
>   kmymoney/plugins/csvexport/kmm_csvexport.desktop 
> 957a9edc4101504e0db08378392e29358327b1c6 
>   kmymoney/plugins/csvimport/CMakeLists.txt 
> c3e13285944f3624d69f7cc7cde0d4d9014a2a2c 
>   kmymoney/plugins/csvimport/csvimport.json.in PRE-CREATION 
>   kmymoney/plugins/csvimport/csvimporterplugin.h 
> e66f9cf8e7bfdf9a431d7c57db2e905f5e89ec66 
>   kmymoney/plugins/csvimport/csvimporterplugin.cpp 
> 9a28dca3efd54b82882ef71595439d34b09a5177 
>   kmymoney/plugins/csvimport/kmm_csvimport.desktop 
> 52dfc6ef5a425f70f4caba42e92378665926c774 
>   kmymoney/plugins/icalendarexport/CMakeLists.txt 
> b85f29282114b5c8eb4d6

Re: Review Request 129371: [kmymoney/4.8] Use GpgME++ if possible, QGpgme from kdepimlibs-4 as fallback

2016-11-11 Thread Marko Käning



> On Nov. 11, 2016, 9:38 p.m., Marko Käning wrote:
> > Cool, does this mean, that one won't need kdepimlibs anymore when building 
> > KMM? :-)
> 
> Andreas Sturmlechner wrote:
> Yes, please give it some testing ;)

Can't promise when, but will give KMM4 some testing on OSX/macOS. Thanks for 
implementing this!


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129371/#review100789
---


On Nov. 10, 2016, 7:13 p.m., Andreas Sturmlechner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129371/
> ---
> 
> (Updated Nov. 10, 2016, 7:13 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> ---
> 
> Gives kmymoney-4 some extended life on systems that upgrade to KDE 
> Applications 16.12 and are required to upgrade to kdepimlibs-conflicting 
> GpgME-1.7.0 by the latest version of KDE PIM.
> 
> Remove bogus qgpgme include.
> Make kdepimlibs optional (based on b5de50ccaa1571c4feb0c17ea8016554e3b822d7 
> from master)
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt 22515185dcb2bdb6a9027101270ff97d445008b6 
>   kmymoney/dialogs/settings/ksettingsschedules.cpp 
> da138a7ab138d2f5ef1e8f5b7ad636f9a1964343 
>   kmymoney/kmymoney.cpp a4251f031cee280983aa67c55447d3f97865dcbd 
>   libkdchart/kdablibfakes/src/KDABLibFakes.h 
> f47ec24423a2ca61b04a98069351539e738fe572 
>   libkgpgfile/CMakeLists.txt a41a6a408e3da8769308dae75d4f514aa969dc87 
>   libkgpgfile/kgpgfile.cpp b1870be92edb833ed30f369e3e0ca0f320fe147b 
> 
> Diff: https://git.reviewboard.kde.org/r/129371/diff/
> 
> 
> Testing
> ---
> 
> Built successfully (so far only tested with GpgME++ 1.7.1 and without 
> kdepimlibs-4).
> Also built fine on a system with Gpgme 1.7.1 (no Gpgmepp bindings) and 
> kdepimlibs-4 installed.
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

Re: Review Request 129371: [kmymoney/4.8] Use GpgME++ if possible, QGpgme from kdepimlibs-4 as fallback

2016-11-11 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129371/#review100789
---



Cool, does this mean, that one won't need kdepimlibs anymore when building KMM? 
:-)

- Marko Käning


On Nov. 10, 2016, 7:13 p.m., Andreas Sturmlechner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129371/
> ---
> 
> (Updated Nov. 10, 2016, 7:13 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> ---
> 
> Gives kmymoney-4 some extended life on systems that upgrade to KDE 
> Applications 16.12 and are required to upgrade to kdepimlibs-conflicting 
> GpgME-1.7.0 by the latest version of KDE PIM.
> 
> Remove bogus qgpgme include.
> Make kdepimlibs optional (based on b5de50ccaa1571c4feb0c17ea8016554e3b822d7 
> from master)
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt 22515185dcb2bdb6a9027101270ff97d445008b6 
>   kmymoney/dialogs/settings/ksettingsschedules.cpp 
> da138a7ab138d2f5ef1e8f5b7ad636f9a1964343 
>   kmymoney/kmymoney.cpp a4251f031cee280983aa67c55447d3f97865dcbd 
>   libkdchart/kdablibfakes/src/KDABLibFakes.h 
> f47ec24423a2ca61b04a98069351539e738fe572 
>   libkgpgfile/CMakeLists.txt a41a6a408e3da8769308dae75d4f514aa969dc87 
>   libkgpgfile/kgpgfile.cpp b1870be92edb833ed30f369e3e0ca0f320fe147b 
> 
> Diff: https://git.reviewboard.kde.org/r/129371/diff/
> 
> 
> Testing
> ---
> 
> Built successfully (so far only tested with GpgME++ 1.7.1 and without 
> kdepimlibs-4).
> Also built fine on a system with Gpgme 1.7.1 (no Gpgmepp bindings) and 
> kdepimlibs-4 installed.
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

[kmymoney4] [Bug 370227] Crash on exit

2016-11-09 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #12 from Marko Käning <mk-li...@mailbox.org> ---
So, what are we going to do about this crash now?

Which delete()'s exactly should I replace by deleteLater()?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 370227] Crash on exit

2016-10-30 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #10 from Marko Käning <mk-li...@mailbox.org> ---
(In reply to Thomas Baumgart from comment #6)
> AqBanking/Gwenhywfar still had a dialog open somewhere. That is causing the
> crash. I wonder, if you can reproduce it by leaving an AqBanking dialog open
> and then exit from KMyMoney.

Yes, I could provoke it by letting the online-update dialog open when exiting
the application. That triggered the exact same backtrace as posted here.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 370227] Crash on exit

2016-10-30 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #9 from Marko Käning <mk-li...@mailbox.org> ---
(In reply to RJVB from comment #8)
> Qt provides the QObject::deleteLater() method for that.

I remember this from some patches you submitted a long time ago. Wasn't it in
KDevelop?
Yeah, I guess this needs some investigation. Will do what Thomas asked me to,
i.e. close KMM with an AqBanking-window still open.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Re: libalkimia 4.3.2 support in kmm 4.8

2016-10-30 Thread Marko Käning

Hi David,

On 30 Oct 2016, at 11:08 , Christian Dávid  wrote:
> the information that alkimia <= 6.0.0 fetches Qt 5 is wrong!
> 
> Alkimia <= 5 is Qt 4 based, it is fetched indirectly over the line
> 
> find_package(KDE4 4.6.0 REQUIRED)
> 
> As written in another thread, the issue is that alkimia 4.x shippes a find 
> module which just finds any version of alkimia. Maybe there are other bugs in 
> the cmake files, too. However, most of these issues should be fixed in 
> version 5.

thanks for clearing this up!

It would have been good to have a wiki page somewhere which explains for 
packagers what should be used with which version.

OK, I’ll try to make kmymoney4-devel use libalkimia version 5 then.

Greets,
Marko

[kmymoney4] [Bug 370227] Crash on exit

2016-10-30 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #5 from Marko Käning <mk-li...@mailbox.org> ---
No, it's definitely built without KF5, as this is from a standard MacPorts
install of port kmymoney4-devel which doesn't know anything about the KF5 world
up to now. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 370227] Crash on exit

2016-10-30 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

Marko Käning <mk-li...@mailbox.org> changed:

   What|Removed |Added

 CC||rjvber...@gmail.com

--- Comment #3 from Marko Käning <mk-li...@mailbox.org> ---
René, do you have an idea why there is a libkdeui.5.dylib in a qt4-mac-build of
kmymoney4-devel? Is the 5 really an indication for KF5, I am really not sure
about that now!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[kmymoney4] [Bug 370227] Crash on exit

2016-10-30 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=370227

--- Comment #2 from Marko Käning <mk-li...@mailbox.org> ---
No, this is a purely non-KF5'ish installation (still)! :-)

Don't know why libkdeui has version number 5 though.

I guess this needs to be figured out.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Re: Review Request 129265: Port from KF5Gpgme to GpgME

2016-10-26 Thread Marko Käning



> On Oct. 26, 2016, 9:44 p.m., Marko Käning wrote:
> > I am wondering which files would your submitted approach pick on 
> > OSX/MacPorts with these KF5 libs installed:
> > 
> > ```
> > $ port installed kf5-gpgmepp 
> > The following ports are currently installed:
> >   kf5-gpgmepp @16.08.2_0+qt5kde (active)
> > $ port contents kf5-gpgmepp 
> > Port kf5-gpgmepp contains:
> >   /opt/local/include/KF5/gpgme++/assuanresult.h
> >   /opt/local/include/KF5/gpgme++/configuration.h
> >   /opt/local/include/KF5/gpgme++/context.h
> >   /opt/local/include/KF5/gpgme++/data.h
> >   /opt/local/include/KF5/gpgme++/decryptionresult.h
> >   /opt/local/include/KF5/gpgme++/defaultassuantransaction.h
> >   /opt/local/include/KF5/gpgme++/editinteractor.h
> >   /opt/local/include/KF5/gpgme++/encryptionresult.h
> >   /opt/local/include/KF5/gpgme++/engineinfo.h
> >   /opt/local/include/KF5/gpgme++/error.h
> >   /opt/local/include/KF5/gpgme++/eventloopinteractor.h
> >   /opt/local/include/KF5/gpgme++/exception.h
> >   /opt/local/include/KF5/gpgme++/global.h
> >   /opt/local/include/KF5/gpgme++/gpgadduserideditinteractor.h
> >   /opt/local/include/KF5/gpgme++/gpgagentgetinfoassuantransaction.h
> >   /opt/local/include/KF5/gpgme++/gpgmefw.h
> >   /opt/local/include/KF5/gpgme++/gpgmepp_export.h
> >   /opt/local/include/KF5/gpgme++/gpgsetexpirytimeeditinteractor.h
> >   /opt/local/include/KF5/gpgme++/gpgsetownertrusteditinteractor.h
> >   /opt/local/include/KF5/gpgme++/gpgsignkeyeditinteractor.h
> >   /opt/local/include/KF5/gpgme++/importresult.h
> >   /opt/local/include/KF5/gpgme++/interfaces/assuantransaction.h
> >   /opt/local/include/KF5/gpgme++/interfaces/dataprovider.h
> >   /opt/local/include/KF5/gpgme++/interfaces/passphraseprovider.h
> >   /opt/local/include/KF5/gpgme++/interfaces/progressprovider.h
> >   /opt/local/include/KF5/gpgme++/key.h
> >   /opt/local/include/KF5/gpgme++/keygenerationresult.h
> >   /opt/local/include/KF5/gpgme++/keylistresult.h
> >   /opt/local/include/KF5/gpgme++/notation.h
> >   /opt/local/include/KF5/gpgme++/result.h
> >   /opt/local/include/KF5/gpgme++/scdgetinfoassuantransaction.h
> >   /opt/local/include/KF5/gpgme++/signingresult.h
> >   /opt/local/include/KF5/gpgme++/trustitem.h
> >   /opt/local/include/KF5/gpgme++/verificationresult.h
> >   /opt/local/include/KF5/gpgme++/vfsmountresult.h
> >   /opt/local/include/KF5/gpgmepp_version.h
> >   /opt/local/include/KF5/qgpgme/dataprovider.h
> >   /opt/local/include/KF5/qgpgme/qgpgme_eventloopinteractor.h
> >   /opt/local/include/KF5/qgpgme/qgpgme_export.h
> >   /opt/local/lib/cmake/KF5Gpgmepp/KF5GpgmeppConfig.cmake
> >   /opt/local/lib/cmake/KF5Gpgmepp/KF5GpgmeppConfigVersion.cmake
> >   /opt/local/lib/cmake/KF5Gpgmepp/KF5GpgmeppTargets-macports.cmake
> >   /opt/local/lib/cmake/KF5Gpgmepp/KF5GpgmeppTargets.cmake
> >   /opt/local/lib/libKF5Gpgmepp-pthread.5.3.2.dylib
> >   /opt/local/lib/libKF5Gpgmepp-pthread.5.dylib
> >   /opt/local/lib/libKF5Gpgmepp-pthread.dylib
> >   /opt/local/lib/libKF5Gpgmepp.5.3.2.dylib
> >   /opt/local/lib/libKF5Gpgmepp.5.dylib
> >   /opt/local/lib/libKF5Gpgmepp.dylib
> >   /opt/local/lib/libKF5QGpgme.5.3.2.dylib
> >   /opt/local/lib/libKF5QGpgme.5.dylib
> >   /opt/local/lib/libKF5QGpgme.dylib
> >   /opt/local/share/qt5/mkspecs/modules/qt_QGpgme.pri
> > ```
> > as well as those installed in the MacPorts installation itself via port 
> > gpgme:
> > ```
> > $ port installed gpgme +universal 
> > The following ports are currently installed:
> >   gpgme @1.7.0_0 (active)
> > $ port contents gpgme 
> > Port gpgme contains:
> >   /opt/local/bin/gpgme-config
> >   /opt/local/bin/gpgme-tool
> >   /opt/local/include/gpgme++/configuration.h
> >   /opt/local/include/gpgme++/context.h
> >   /opt/local/include/gpgme++/data.h
> >   /opt/local/include/gpgme++/decryptionresult.h
> >   /opt/local/include/gpgme++/defaultassuantransaction.h
> >   /opt/local/include/gpgme++/editinteractor.h
> >   /opt/local/include/gpgme++/encryptionresult.h
> >   /opt/local/include/gpgme++/engineinfo.h
> >   /opt/local/include/gpgme++/error.h
> >   /opt/local/include/gpgme++/eventloopinteractor.h
> >   /opt/local/include/gpgme++/exception.h
> >   /opt/local/include/gpgme++/global.h
> >   /opt/local/include/gpgme++/gpgadduserideditinteractor.h
> >   /opt/local/include/gpgme++/gpgagentgetinfoassuantransaction.h
> >   /opt/local/include/gpgme++/gpgmefw.h
> >   /opt/local/include/gpgme++/gpgmepp_export.h
> >   /opt/local/

Re: Review Request 129265: Port from KF5Gpgme to GpgME

2016-10-26 Thread Marko Käning

/include/gpgme++/signingresult.h
  /opt/local/include/gpgme++/tofuinfo.h
  /opt/local/include/gpgme++/trustitem.h
  /opt/local/include/gpgme++/verificationresult.h
  /opt/local/include/gpgme++/vfsmountresult.h
  /opt/local/include/gpgme.h
  /opt/local/include/gpgmepp_version.h
  /opt/local/lib/cmake/Gpgmepp/GpgmeppConfig.cmake
  /opt/local/lib/cmake/Gpgmepp/GpgmeppConfigVersion.cmake
  /opt/local/lib/libgpgme-pthread.11.dylib
  /opt/local/lib/libgpgme-pthread.a
  /opt/local/lib/libgpgme-pthread.dylib
  /opt/local/lib/libgpgme.11.dylib
  /opt/local/lib/libgpgme.a
  /opt/local/lib/libgpgme.dylib
  /opt/local/lib/libgpgmepp.6.dylib
  /opt/local/lib/libgpgmepp.a
  /opt/local/lib/libgpgmepp.dylib
  /opt/local/share/aclocal/gpgme.m4
  /opt/local/share/common-lisp/source/gpgme/gpgme-package.lisp
  /opt/local/share/common-lisp/source/gpgme/gpgme.asd
  /opt/local/share/common-lisp/source/gpgme/gpgme.lisp
  /opt/local/share/doc/gpgme/AUTHORS
  /opt/local/share/doc/gpgme/COPYING
  /opt/local/share/doc/gpgme/COPYING.LESSER
  /opt/local/share/doc/gpgme/ChangeLog
  /opt/local/share/doc/gpgme/INSTALL
  /opt/local/share/doc/gpgme/NEWS
  /opt/local/share/doc/gpgme/README
  /opt/local/share/doc/gpgme/THANKS
  /opt/local/share/doc/gpgme/TODO
  /opt/local/share/info/gpgme.info
  /opt/local/share/info/gpgme.info-1
  /opt/local/share/info/gpgme.info-2
```
We actually enforced the use of `kdepimlibs`' version of GpgME during building 
of `kmymoney4-devel` with this patch:
```
*** kmymoney-4.7.2/CMakeLists.txt   Sat Apr 25 19:20:16 2015
--- kmymoney-4.7.2/CMakeLists.txt   Wed Oct  5 23:14:44 2016
***
*** 70,75 
--- 70,82 
  endif (WIN32)
  # needed by libkgpgfile
  find_package(QGpgme REQUIRED)
+
+ # This is specific for MacPorts, as it ships a newer version of gpgme (1.7.0)
+ if (QGPGME_FOUND)
+   set(GPGME_INCLUDES "%PREFIX%/include/KDE4")
+   include_directories(${GPGME_INCLUDES})
+ endif (QGPGME_FOUND)
+
  find_package(KdepimLibs REQUIRED)
  find_package(SharedMimeInfo REQUIRED)
```
I can't test this with a `frameworks` version of KMyMoney so far, as there is 
still no offical release of the KF5 frameworks available on OSX/MacPorts at 
this point in time.

So, I am just curious, but I believe your CMake magic would like to pick the 
latter set of files which would make port `kf5-gpgmepp` obsolete. Is that 
intended?

- Marko Käning


On Oct. 26, 2016, 5:41 p.m., Andreas Sturmlechner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129265/
> ---
> 
> (Updated Oct. 26, 2016, 5:41 p.m.)
> 
> 
> Review request for KMymoney.
> 
> 
> Repository: kmymoney
> 
> 
> Description
> ---
> 
> Gpgmepp is part of GpgME since 1.7.0.
> 
> REVIEW: 129265
> 
> 
> Diffs
> -
> 
>   CMakeLists.txt c535faba80db64b0a30fc5fdfb09a7a2590ab8a4 
>   config-kmymoney.h.cmake f024db61af25e291cde2eef392fe161485c82e5d 
>   kmymoney/main.cpp 694ffa28ae67ad0051b893fbdcf9b7b3624f0525 
>   libkgpgfile/CMakeLists.txt 4e366ceab3c12728276feaaec78319315fb9ea6d 
>   libkgpgfile/kgpgfile.cpp 88aac42de83d28be45ad8d1516bd94eb240ed811 
> 
> Diff: https://git.reviewboard.kde.org/r/129265/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

Re: alkimia yet again - on KF5

2016-10-25 Thread Marko Käning

Hi Christian,

On 25 Oct 2016, at 20:08 , Cristian Oneț  wrote:
> Yes, build libalkimia [1] and use it instead of the one provided by your 
> system.
> 
> [1] https://quickgit.kde.org/?p=alkimia.git

there is still the old location at kde-apps, which also lead to confusion for 
me on MacPorts:
---
$ port info libalkimia | grep Homepage
Homepage: 
http://kde-apps.org/content/show.php/libalkimia?content=137323
---
Only when I updated my kmymoney4-devel port recently I ran into this…

I’ve completely forgotten to change that to quickgit, which should have 
happened long ago, I guess.

Well, even the latest git version of KMM on branch 4.8 is still building and 
running with this! :-o

OK, I hope you (will soon) have a wiki page somewhere which keeps track of 
which libalkimia version is needed for which version of KMM on KDE4 and KF5...

Greets,
Marko

Re: More libalkimia problems/questions was: problems compiling 4.8 on system with both qt4 and qt5

2016-10-25 Thread Marko Käning

On 25 Oct 2016, at 19:20 , Ralf Habacker  wrote:
> ECM is part of the former only Qt5 supporting alkimia build system.
> There are several ecm related macros used:
> 
> autotests/CMakeLists.txt:5:ecm_mark_as_test(alkvaluetest)
> autotests/CMakeLists.txt:11:ecm_mark_as_test(alkquoteitemtest)
> autotests/CMakeLists.txt:17:ecm_mark_as_test(alkcompanytest)
> src/CMakeLists.txt:50:ecm_setup_version(${ALKIMIA_LIB_VERSION}
> VARIABLE_PREFIX ALKIMIA
> src/CMakeLists.txt:55:ecm_configure_package_config_file()

Ok, I see.


> To remove the ecm dependency they need to be implemented locally. The
> question is if it is worth to reimplement them.

No, it’s fine, I see now that we on MacPorts also have a qt4 variant of it:
---
$ port info kde-extra-cmake-modules
kde-extra-cmake-modules @5.11.0 (kde, kf5, devel)
Variants: debug, docs, qt4, qt5
---

I forgot. (It’s too long ago that I dealt with KF5/CI on OSX…)

Thanks for pointing this out.

Re: More libalkimia problems/questions was: problems compiling 4.8 on system with both qt4 and qt5

2016-10-25 Thread Marko Käning

Hi Ralf,

On 25 Oct 2016, at 14:44 , timothy  wrote:
> However find_package(ECM 0.0.11 REQUIRED NO_MODULE). I downloaded ECM
> 0.0.11 and built it. But it also has qt5 dependencies.

is ECM needed for Qt4? I always thought that it would only be needed for 
Qt5/KF5…

Greets,
Marko

[kmymoney4] [Bug 364725] New: Crashes when quitting the application while online-banking update dialog is open

2016-06-24 Thread Marko Käning via KDE Bugzilla

https://bugs.kde.org/show_bug.cgi?id=364725

Bug ID: 364725
   Summary: Crashes when quitting the application while
online-banking update dialog is open
   Product: kmymoney4
   Version: git master
  Platform: MacPorts Packages
OS: OS X
Status: UNCONFIRMED
  Severity: crash
  Priority: NOR
 Component: onlinebanking
  Assignee: kmymoney-devel@kde.org
  Reporter: mk-li...@mailbox.org

KMM crashes when I quit the application while the online-banking update-dialog
is still open.

For some reason it is possible to bring the app's main window into foreground
with the update-dialog in background and then the app reacts to an application
close command. This tries to finish the application, but when doing that
crashes it.

---

Application: KMyMoney (kmymoney), signal: Segmentation fault: 11
(lldb) process attach --pid 710
Process 710 stopped
Executable module set to
"/Applications/MacPorts/KDE4/kmymoney.app/Contents/MacOS/kmymoney".
Architecture set to: x86_64-apple-macosx.
(lldb) set set term-width 200
(lldb) thread info
thread #1: tid = 0x39be, 0x7fff8aeb8e20 libsystem_kernel.dylib`__wait4 + 8,
queue = 'com.apple.main-thread', stop reason = signal SIGSTOP

(lldb) bt all
* thread #1: tid = 0x39be, 0x7fff8aeb8e20 libsystem_kernel.dylib`__wait4 +
8, queue = 'com.apple.main-thread', stop reason = signal SIGSTOP
  * frame #0: 0x7fff8aeb8e20 libsystem_kernel.dylib`__wait4 + 8
frame #1: 0x000106fac9be libkdeui.5.dylib`KCrash::startProcess(int,
char const**, bool) + 286
frame #2: 0x000106fabae9
libkdeui.5.dylib`KCrash::defaultCrashHandler(int) + 1209
frame #3: 0x7fff89fed5aa libsystem_platform.dylib`_sigtramp + 26
frame #4: 0x0001135d4f42 libgwenhywfar.60.dylib`GWEN_List1_GetFirst +
16
frame #5: 0x0001135d315f libgwenhywfar.60.dylib`GWEN_Inherit_FindData +
52
frame #6: 0x000108f1e06a dir.so`GWEN_ConfigMgrDir_LockGroup + 76
frame #7: 0x0001136d0754
libaqbanking.35.dylib`AB_Banking_LockSharedConfig + 70
frame #8: 0x0001136fc7a5 libaqbanking.35.dylib`AB_Gui_WriteDialogPrefs
+ 207
frame #9: 0x00011364df85 libgwenhywfar.60.dylib`GWEN_Dialog_free + 89
frame #10: 0x000113652144
libgwenhywfar.60.dylib`GWEN_Gui_Internal_ProgressEnd + 999
frame #11: 0x0001136cbb3e libaqbanking.35.dylib`AB_Banking_ExecuteJobs
+ 2257
frame #12: 0x00011311dad2
kmm_kbanking.so`KMyMoneyBanking::executeQueue(AB_IMEXPORTER_CONTEXT*) + 82
frame #13: 0x00011311d9e5
kmm_kbanking.so`KBankingPlugin::executeQueue() + 165
frame #14: 0x00011311d589
kmm_kbanking.so`KBankingPlugin::updateAccount(MyMoneyAccount const&, bool) +
2089
frame #15: 0x000104854ae4
kmymoney`KMyMoneyApp::slotAccountUpdateOnline() + 468
frame #16: 0x000104864bcb
kmymoney`KMyMoneyApp::qt_static_metacall(QObject*, QMetaObject::Call, int,
void**) + 2107
frame #17: 0x00010845506b QtCore`QMetaObject::activate(QObject*,
QMetaObject const*, int, void**) + 2363
frame #18: 0x0001073df6a9 QtGui`QAction::activate(QAction::ActionEvent)
+ 233
frame #19: 0x000107769004 QtGui`QAbstractButtonPrivate::click() + 84
frame #20: 0x000107769de8
QtGui`QAbstractButton::mouseReleaseEvent(QMouseEvent*) + 88
frame #21: 0x0001078357cf
QtGui`QToolButton::mouseReleaseEvent(QMouseEvent*) + 15
frame #22: 0x00010743ca1c QtGui`QWidget::event(QEvent*) + 716
frame #23: 0x000107769cf3 QtGui`QAbstractButton::event(QEvent*) + 179
frame #24: 0x000107835c7b QtGui`QToolButton::event(QEvent*) + 171
frame #25: 0x0001073e87eb
QtGui`QApplicationPrivate::notify_helper(QObject*, QEvent*) + 251
frame #26: 0x0001073ea775 QtGui`QApplication::notify(QObject*, QEvent*)
+ 2949
frame #27: 0x00010843c186
QtCore`QCoreApplication::notifyInternal(QObject*, QEvent*) + 118
frame #28: 0x0001073e91fb
QtGui`QApplicationPrivate::sendMouseEvent(QWidget*, QMouseEvent*, QWidget*,
QWidget*, QWidget**, QPointer&, bool) + 475
frame #29: 0x000107395902 QtGui`qt_mac_handleMouseEvent(NSEvent*,
QEvent::Type, Qt::MouseButton, QWidget*, bool) + 1298
frame #30: 0x7fff8c9ed145 AppKit`-[NSWindow sendEvent:] + 781
frame #31: 0x00010738cde1 QtGui`-[QCocoaWindow sendEvent:] + 113
frame #32: 0x7fff8c98e5d4 AppKit`-[NSApplication sendEvent:] + 2021
frame #33: 0x000107391d21 QtGui`-[QNSApplication sendEvent:] + 97
frame #34: 0x7fff8c7de9f9 AppKit`-[NSApplication run] + 646
frame #35: 0x00010739b16b
QtGui`QEventDispatcherMac::processEvents(QFlags)
+ 2027
frame #36: 0x0001084394ff
QtCore`QEventLoop::exec(QFlags) + 495
frame #37: 0x00010843c737 QtCore`QCoreApplication::exec() + 199
frame #38: 0x000104801f01 kmymoney`runKMyMoney(KApplication*,
KStartupLogo*) + 2753
frame #39: 0x00010480105e kmymoney`main + 9374
frame #40:

Re: Master is closed for KDE4 based development

2016-06-16 Thread Marko Käning

On 16 Jun 2016, at 23:16 , Thomas Baumgart  wrote:
> 
> No problemo, it's good that someone is watching us and doing some QA from the
> outside.
> 

:-)

Wish I was there…

In that case I’d try to get the gwenhywfar4 fixed. :-)

Please see my other post on thread "KMyMoney 4.8.0 (the Randa release) is out” 
regarding that issue [1].

It had been annoying me in April, but up to now I have had no time whatsoever 
to investigate this or even come up with a fix.

Greets to the whole Randa team working on KMM as well as on all the other KDE 
software!!!
Have fun hacking!
Marko

[1] https://mail.kde.org/pipermail/kmymoney/2016-June/002576.html

signature.asc
Description: Message signed with OpenPGP using GPGMail

Re: Master is closed for KDE4 based development

2016-06-16 Thread Marko Käning

On 16 Jun 2016, at 22:53 , Cristian Oneț  wrote:
> 
> Yes.

Thanks, Cristian, for responding.

I should have checked build.kde.org before posting, then I’d have seen that all 
had been done perfectly yesterday morning.

Or course, as you’re still all assembled in Randa these days! ;)

Sorry for the noise, thus!

Re: Master is closed for KDE4 based development

2016-06-16 Thread Marko Käning

Hi Christian,

is KDE’s CI (meaning Scarlett) informed about this change?

Greets,
Marko



On 15 Jun 2016, at 10:32 , Cristian Oneț  wrote:

> Frameworks was merged into master, this means that the frameworks
> branch has reached the end of its life and was deleted.
> 
> Cristian

Re: Suddenly building trouble on OSX for kmymoney's KDE4 master

2016-04-17 Thread Marko Käning

Hi Christian,

On 17 Apr 2016, at 11:10 , Christian David  wrote:

> The missing ...config.cmake files were added recently. Maybe they were not 
> added to the port (I do not know how macports work)?!

I am pulling the source tgz’s from upstream directly. So, these files should be 
in there, no?
But perhaps they’re generated and OSX has been forgotten up to now…

OK, will direct my question to AqBanking’s ML.

Greets,
Marko

Re: Suddenly building trouble on OSX for kmymoney's KDE4 master

2016-04-16 Thread Marko Käning

Hi Christian,

I am not mixing versions of gwenhywfar. Have installed either one or the other. 
Right now I installed 4.15.2beta even - which has the same problem.

But well, with 4.14.0 KMM wouldn’t build - as described earlier. I haven’t 
checked up to now whether the older version still shipped said file.

I am using downloads directly from upstream, how come this surfaces only for me?

Greets,
Marko



$ port contents gwenhywfar4-devel | grep -- -cpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/api.h
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppdialog.hpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppgui.hpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppwidget.hpp
  /opt/local/lib/libgwengui-cpp.0.dylib
  /opt/local/lib/libgwengui-cpp.a
  /opt/local/lib/libgwengui-cpp.dylib

Re: plugin development help

2016-04-16 Thread Marko Käning

Hi Przemysław,

what exactly are you planning for this plugin?

Have you somewhere some outline about what the plugin would actually allow to 
do with Lua and KMM?

Greets,
Marko

Re: Suddenly building trouble on OSX for kmymoney's KDE4 master

2016-04-16 Thread Marko Käning

Ah, I see, the pc file for cpp is missing:

---
$ port contents gwenhywfar4-devel | grep -- -cpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/api.h
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppdialog.hpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppgui.hpp
  /opt/local/include/gwenhywfar4/gwen-gui-cpp/cppwidget.hpp
  /opt/local/lib/libgwengui-cpp.0.dylib
  /opt/local/lib/libgwengui-cpp.a
  /opt/local/lib/libgwengui-cpp.dylib

$ port contents gwenhywfar4-devel | grep -- -qt4
  /opt/local/include/gwenhywfar4/gwen-gui-qt4/qt4_gui.hpp
  /opt/local/lib/cmake/gwengui-qt4-4.14/gwengui-qt4-config-version.cmake
  /opt/local/lib/cmake/gwengui-qt4-4.14/gwengui-qt4-config.cmake
  /opt/local/lib/libgwengui-qt4.0.dylib
  /opt/local/lib/libgwengui-qt4.a
  /opt/local/lib/libgwengui-qt4.dylib
  /opt/local/lib/pkgconfig/gwengui-qt4.pc
---

That’s an upstream issue then.

Don’t know whether this file exists for version 4.13.0...

Suddenly building trouble on OSX for kmymoney's KDE4 master

2016-04-16 Thread Marko Käning

Hi devs,

last time I built KMM for MacPorts in January all was fine, but today I ran 
into this:
---
:info:configure -- Found LibOfx: /opt/local/lib
:info:configure CMake Warning at CMakeLists.txt:168 (find_package):
:info:configure   Could not find a package configuration file provided by 
"gwengui-cpp"
:info:configure   (requested version 4.13.0) with any of the following names:
:info:configure
:info:configure gwengui-cppConfig.cmake
:info:configure gwengui-cpp-config.cmake
:info:configure
:info:configure   Add the installation prefix of "gwengui-cpp" to 
CMAKE_PREFIX_PATH or set
:info:configure   "gwengui-cpp_DIR" to a directory containing one of the above 
files.  If
:info:configure   "gwengui-cpp" provides a separate development package or SDK, 
be sure it
:info:configure   has been installed.
:info:configure
:info:configure
:info:configure -- Gwenhywfar 4.14.0.0 found at /opt/local/include/gwenhywfar4 
(/opt/local/lib/libgwenhywfar.60.dylib)
:info:configure -- AqBanking 5.5.1 found at /opt/local/include/aqbanking5 
(/opt/local/lib/libaqbanking.dylib;/opt/local/lib/libgwenhywfar.60.dylib)
:info:configure CMake Error at CMakeLists.txt:176 (message):
:info:configure   KBanking requirements not met
---

Is this because I am having a newer gwenhywfar version installed already?

Greets,
Marko

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-06-11 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #21 from Marko Käning mk-li...@email.de ---
(In reply to harry bennett from comment #20)
 For what it's worth, I was finally able to import what I needed after an the
 upgrade discussed upthread. ( https://bugs.kde.org/show_bug.cgi?id=346495#c7
 )

Good to know. :)


 There is however (still?) an issue with widget focus and sizing during the
 import process. I have to stretch a window or two to fully reveal buttons.

Well, I can imagine. widget focus and size issues unfortunately do happen at
times, but I don't know what to do about those. If you could prepare some
screenshots - in case you've got more issues than the one in the currently
attached image - please upload them to this ticket.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-28 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #19 from Marko Käning mk-li...@email.de ---
Allen, we had a discussion about this hiding of dialogs on OSX a while ago on
KDE-MAC...

https://mail.kde.org/pipermail/kde-mac/2014-July/001175.html

Hoping that this discussion helps to figure it out.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-26 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #11 from Marko Käning mk-li...@email.de ---
It is there and fully functional also on MacPorts' OSX version of KMM.
(Tested for current port kmymoney4-devel.)

I'll try to update port kmymoney4 wrt KMM's latest release now...

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-26 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #12 from Marko Käning mk-li...@email.de ---
Harry, you may also try to install kmymoney4-devel!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [Kmymoney] KMyMoney 4.7.2 released

2015-04-26 Thread Marko Käning

Hi Cristian,

On 26 Apr 2015, at 13:04 , Cristian Oneț onet.crist...@gmail.com wrote:
 KMyMoney 4.7.2 was released, you can read the release accouncement here [1].
 the Windows installer is ready, the OSX package will be ready (I'm Marko will 
 let us know), the rest depends on your distributions.

I don’t think you _are_ Marko... ;-)

... but you can be sure that I’ll let you know once it is up and running on 
MacPorts. :-)

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-26 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #14 from Marko Käning mk-li...@email.de ---
Check this wiki page: https://trac.macports.org/wiki/KDE

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] KMyMoney 4.7.2 released

2015-04-26 Thread Marko Käning

Hi devs and users,

so, the 4.7.2 is out on OSX/MacPorts

- http://trac.macports.org/changeset/135573

The builders of the 4 most recent OSX versions 10.[7-10] succeeded!
(The 10.6.8 builder seems to be down at the moment.)

...

BTW, there is an ongoing discussion on Bugzilla

- https://bugs.kde.org/show_bug.cgi?id=346495

which made clear to me, that KMyMoney users on OSX/MacPorts need to consider 
this
wiki page

- https://trac.macports.org/wiki/KDE

in order to make sure that they run kbuildsycoca4 after the installation of 
ports
kmymoney4 or kmymoney4-devel.

...

Also I want to mention that that same wiki page has useful hints regarding the
installation of ports 

kde4-workspace  qtcurve

which will help avoiding trouble caused by KDE’s default “Macintosh (Aqua)” 
style.

...

Another recent bug which might hit people trying to install KMyMoney’s ports on
MacPorts is discussed here

- https://trac.macports.org/ticket/47496

and concerns a old, but nasty flaw in meinproc4 when generating documentation at
build time. But I hope that this - finally really reproducible crash - will help
to solve this mystery.

I’ll investigate that next… Stay tuned.

Greets,
Marko




P.S. for KMM’s developers:
   Perhaps it would be good to mention https://trac.macports.org/wiki/KDE
   on KMyMoney’s website under HOME/MacPorts or something like that?!
   I’ve just created a task for this on KMM’s kanboard in column “Backlog”

   - https://todo.kde.org/?controller=taskaction=showtask_id=1232
   :)

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] : Use DATA_INSTALL_DIR to install appdata

2015-04-26 Thread Marko Käning

Hi Jack,

On 25 Apr 2015, at 15:39 , Jack ostrof...@frontier.com wrote:
 Should that variable appear in the general environment, or does it only need 
 to be within the make environment when configuring and compiling?

I think you should consult KDE’s KF5 porting recommendations or so…

Greets,
Mark
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] KMyMoney 4.7.2 released

2015-04-26 Thread Marko Käning

A correction:

On 26 Apr 2015, at 20:10 , Marko Käning mk-li...@email.de wrote:
 The builders of the 4 most recent OSX versions 10.[7-10] succeeded!
 (The 10.6.8 builder seems to be down at the moment.)

It is Lion’s build bot which is down, not Snow Leopard’s! :-)


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-26 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #13 from Marko Käning mk-li...@email.de ---
I've just tried to run the CSVWizard and have to say, that in my case the
current git master, as well as the release version 4.7.2 SEEMED fail to run the
wizard. 

Already when clicking the Select File button for step 1. Start the
application seemingly hung up, but in *truth* only the Adding profile name
dialog was _hidden_ behind the wizard's dialog window without me noticing it.
This stuff happens on OSX sometimes, which is why I looked out for it and saw
it after all... :(

Same happens then when I want to select the CSV file.
That file open dialog is kept behind the CSVWizard dialog as well. The latter
acts actually system modal and stay atop of all windows!!

When reaching step 3 I *do* see buttons here on my end!!!

---

One question: Did you run

  $ kbuildsycoca4

(once) as the user running KMyMoney (i.e. not using sudo) before actually
starting the application for the first time?

If not, please do so and restart KMM and the wizard!

This tool should normally be run on a regular basis by a launch agent installed
by MacPorts, which can or must be enabled by

 $ launchctl load -w /Library/LaunchAgents/org.macports.kdecache.plist 

but this doesn't help, if you install the port and don't wait for this
regularly called agent to do its job first.

So, that is why it is always a good idea calling it directly on the command
line after every sudo port install WHATEVERKDEAPPLICATION.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-26 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #17 from Marko Käning mk-li...@email.de ---
Yes, sorry, I meant to point out in my yesterday's posts, that indeed I was
talking about a sample CSV data file, of course.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] Bugzilla doesn't know about the latest release 4.7.2

2015-04-26 Thread Marko Käning

BTW, Bugzilla does not (yet) know about the latest version 4.7.2...
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney] kmymoney: Use DATA_INSTALL_DIR to install appdata

2015-04-25 Thread Marko Käning

Hi Thomas,

On 25 Apr 2015, at 09:02 , Thomas Baumgart t...@net-bembel.de wrote:
 Maybe, but that ${KDE_INSTALL_APPDIR} does not exist in my KDE 4.x 
 environment.

OK, so this should be kept in mind for the KF5 version then.

Thanks for pointing that out.

Greets,
Marko


signature.asc
Description: Message signed with OpenPGP using GPGMail
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney] kmymoney: Use DATA_INSTALL_DIR to install appdata

2015-04-24 Thread Marko Käning

Hi devs,

this should - for making it more cross-platform - rather use ECM's

${KDE_INSTALL_APPDIR}

instead of 

${DATA_INSTALL_DIR}/appdata/

right!?

Greets,
Marko



On 24 Apr 2015, at 19:45 , Heiko Becker heire...@exherbo.org wrote:

 Git commit 9b51544d9fc8188fec231eb4006cde2ca0c200ee by Heiko Becker.
 Committed on 24/04/2015 at 15:44.
 Pushed by heikobecker into branch 'master'.
 
 Use DATA_INSTALL_DIR to install appdata
 
 M  +1-1kmymoney/CMakeLists.txt
 
 http://commits.kde.org/kmymoney/9b51544d9fc8188fec231eb4006cde2ca0c200ee
 
 diff --git a/kmymoney/CMakeLists.txt b/kmymoney/CMakeLists.txt
 index aed99d5..7febf05 100644
 --- a/kmymoney/CMakeLists.txt
 +++ b/kmymoney/CMakeLists.txt
 @@ -108,7 +108,7 @@ install(PROGRAMS kmymoney.desktop
   DESTINATION ${XDG_APPS_INSTALL_DIR}
   )
 install(PROGRAMS kmymoney.appdata.xml
 -  DESTINATION share/appdata/
 +  DESTINATION ${DATA_INSTALL_DIR}/appdata/
   )
 
 install(FILES x-kmymoney.xml
 

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-23 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

--- Comment #4 from Marko Käning mk-li...@email.de ---
Can you supply a test file please, so that I can try to reproduce what you're
seeing?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 346495] Mac cvs import does not allow completion of import.

2015-04-23 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=346495

Marko Käning mk-li...@email.de changed:

   What|Removed |Added

 CC||mk-li...@email.de

--- Comment #3 from Marko Käning mk-li...@email.de ---
You're experiencing issues with KMM from MacPorts?

As I am maintaining that port, please tell me which port and version are you
using?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Build failed in Jenkins: kmymoney_master_qt5 #127

2015-01-21 Thread Marko Käning

Trying to address this with


http://commits.kde.org/kde-build-metadata/dad6f4f6d6f38795a763e9541a0353053361b861

Will restart build in 20 mins (to give the CI system time to pull the 
dependencies anew).
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

On 14 Jan 2015, at 20:54 , Cristian Oneț onet.crist...@gmail.com wrote:
 Thanks for the fastest feedback :) I've seen,
:)


 I'll look at those issues but it could take a couple of days.
Cool.

 I guess you'll follow the commits ;).
Yes, I do. I am a semi-automatic CI system, you know. ;-)
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

On 14 Jan 2015, at 21:05 , Cristian Oneț onet.crist...@gmail.com wrote:
 This time you were actually faster then the automatic build server
 [1].

Yes, because my OSX/CI system is faster than the KDE one.


 Btw: any ideas why the build was not triggered there?

Oh, didn’t it?
Will check.


 [1] http://build.kde.org/job/kmymoney_frameworks_qt5/

Strange, yes, it didn’t start. Just did so manually now.

Have seen this today for other projects as well… Need to ask Ben about it.

Thanks for mentioning this.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning


On 14 Jan 2015, at 21:05 , Cristian Oneț onet.crist...@gmail.com wrote:

 Btw: any ideas why the build was not triggered there?

Turns our the “Poll SCM” option wasn’t checked in Jenkins’ job configuration.

Fixed now.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

Hi Ben,

On 14 Jan 2015, at 22:22 , Ben Cooksley bcooks...@kde.org wrote:
 Poll SCM isn't the option needed, Trigger builds remotely should be
 sufficient.

I was mislead by “kmymoney_master”, where that option was set.


 Have you verified that the build metadata is correct for KMyMoney here?

That should be correct.


 Also, merge commits may not trigger the hooks in some cases.

Could be that this is the reason for it.

Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

Hi Cristian,

On 14 Jan 2015, at 20:39 , Cristian Oneț onet.crist...@gmail.com wrote:

 Git commit 90c52e5effb5714042e1e8110f729b681b28dff8 by Cristian Oneț.
 Committed on 14/01/2015 at 19:25.
 Pushed by conet into branch 'frameworks'.
 
 Merge remote-tracking branch 'origin/master' into frameworks

thanks for taking care of this.

I’ve found a couple of problems on OSX/CI.

See the attached build log, please.

Regards,
Marko


kmymoney.log.gz
Description: GNU Zip compressed data
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

 This might sound backwards, but the job needs to be called
 kmymoney_master_qt5 in order for it to work.
 It should now trigger properly.

Yes, indeed, I am puzzled about this now.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [kmymoney/frameworks] /: Merge remote-tracking branch 'origin/master' into frameworks

2015-01-14 Thread Marko Käning

Hi Ben,

I had to manually start a build for this again…

Don’t understand why it is not doing this on its own, especially now that 
the last two commits went directly into frameworks and not as merges.

Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Master branch is open for development

2015-01-06 Thread Marko Käning

Hi Cristian,

On 05 Jan 2015, at 14:01 , Cristian Oneț onet.crist...@gmail.com wrote:
 After the merge is done I
 will try to revive the frameworks branch by merging master into it.
 That way the onlinebanking part will also be ported to frameworks.

that would be really nice, so that we can have finally a look at the
actually required frameworks dependencies.

Happy New Year,
Marko 
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] KMyMoney's KF5 dependencies for Jenkins master

2014-12-13 Thread Marko Käning

Hi devs,

now that kdepimlibs has been split up in a multitude of various frameworks
we should be able to limit the dependencies for KMyMoney, right?!


This is what dependency-data-kf5-qt5 defines: 
---
# KDE PIM Dependencies
kde/kdepimlibs: kdesupport/akonadi
kde/kdepimlibs: kdesupport/prison
kde/kdepimlibs: kde/pim/kcontacts
kde/kdepimlibs: kde/pim/kxmlrpcclient
kde/kdepimlibs: kde/pim/kblog
kde/kdepimlibs: kde/pim/kcalcore
kde/kdepimlibs: kde/pim/kcalutils
kde/kdepimlibs: kde/pim/kholidays
kde/kdepimlibs: kde/pim/kimap
kde/kdepimlibs: kde/pim/kldap
kde/kdepimlibs: kde/pim/kidentitymanagement
kde/kdepimlibs: kde/pim/kmbox
kde/kdepimlibs: kde/pim/kmime
kde/kdepimlibs: kde/pim/kontactinterface
kde/kdepimlibs: kde/pim/kpimtextedit
kde/kdepimlibs: kde/pim/ktnef
kde/kdepimlibs: kde/pim/syndication
kde/kdepimlibs: kde/pim/gpgmepp

# Extragear Office
extragear/office/kmymoney: extragear/office/alkimia
extragear/office/kmymoney: kde/kdepimlibs
---

and I think that could be stripped down to something less, like
---
# Extragear Office
extragear/office/kmymoney: kdesupport/akonadi
extragear/office/kmymoney: kdesupport/kcontacts
extragear/office/kmymoney: extragear/office/alkimia
extragear/office/kmymoney: kde/pim/kholidays
extragear/office/kmymoney: kde/pim/kidentitymanagement
extragear/office/kmymoney: kde/pim/gpgmepp
---


Strangely enough - although KMM builds fine - I see this in the log:
---
-- Could NOT find KF5Abc: found neither KF5AbcConfig.cmake nor 
kf5abc-config.cmake
-- Could NOT find KF5 (missing:  Holidays Gpgmepp QGpgme PimIdentities Akonadi 
Abc)
---

These frameworks don’t have a matching name to above, which results after all 
in 
---
 KMyMoney 4.100.0 

Configure results (user options):

KDE PIM holidays:no

KDE PIM encryption:  no

KDE PIM addressbook: no
---


Does this mean, that the porting of the required frameworks isn’t complete yet?
Or does it simply mean that KMyMoney still needs to adapt to the new naming?

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] kf5 based version

2014-12-01 Thread Marko Käning

Hi Jeremy,

On 01 Dec 2014, at 22:16 , Jeremy Whiting jpwhit...@kde.org wrote:
 Today I tried to build the frameworks branch of kmymoney, but it fails at 
 cmake time looking for KF5PimIdentitiesConfig.cmake. I have a whole 
 frameworks build with kdesrc-build here, but don't see PimIdentities 
 anywhere. It's mentioned in kdepimlibs (and in the kdepimlibs description on 
 projects.kde.org) but kdepimlibs master branch doesn't create that cmake file 
 somehow. Is there a certain branch of kdepimlibs that the kf5 port of kmm 
 depends on ?

well, on OSX/CI I have removed kdepimlibs altogether from KMM’s dependencies, 
thus avoiding the whole issue!
That’s the only way how I can build its ‘frameworks' branch successfully for 
KF5 at the moment on OSX.

Cristian will surely give you more information for Linux...

Greets,
Marko___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 121192: Support Page, PlasmaMenu and Home Button

2014-11-20 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121192/#review70708
---


I wonder why 3 out of 4 diffs cannot be displayed here in ReviewBoard...

- Marko Käning


On Nov. 20, 2014, 9:20 p.m., Lars Meiendresch wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/121192/
 ---
 
 (Updated Nov. 20, 2014, 9:20 p.m.)
 
 
 Review request for KMymoney.
 
 
 Repository: kmymoney-org
 
 
 Description
 ---
 
 New Support Page, Bugs removed in the PlasmaMenu and Home Button refer 
 you to the index.
 
 
 Diffs
 -
 
   documentation.php ba6067f 
   faq.php 85aaec8 
   plasmaMenu.inc 109ab08 
   support.php PRE-CREATION 
 
 Diff: https://git.reviewboard.kde.org/r/121192/diff/
 
 
 Testing
 ---
 
 There will be bugs - im sure. Sorry for that. Im tried my best.
 
 
 Thanks,
 
 Lars Meiendresch
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kmymoney4] [Bug 340956] Duplicated statements in KMM-4.7.1 when importing transactions

2014-11-16 Thread Marko Käning

https://bugs.kde.org/show_bug.cgi?id=340956

Marko Käning mk-li...@email.de changed:

   What|Removed |Added

 CC||mk-li...@email.de

--- Comment #11 from Marko Käning mk-li...@email.de ---
(In reply to Dr.Edgar Alwers from comment #0)
 It looks like the files generated by KMM-4.6.4 are not fully compatible with
 KMM 4.7.1.

I also see this every now and then - when there has been a change on client or
server side!

I usually clean up the overlapping transactions bit by bit and then go on
downloading transactions with the latest aqbanking library from then on.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Theming issue?

2014-10-31 Thread Marko Käning

On 30 Oct 2014, at 23:52 , allan agande...@gmail.com wrote:
 Isn't it just the Home view and Reports that use a different colour, and 
 aren't they HTML based?  Perhaps that allows/causes the difference.

Hmm, I see now that it is not only

“Scheduled transactions” and “Legders” 

but also in

“Payees”, “Reports” and “Forecast” view when I really look carefully 
enough...


So it is characteristic for all scroll bar backgrounds in all views.

Why is this only happening for scroll bar backgrounds and no other control??

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] Theming issue?

2014-10-30 Thread Marko Käning

Hi,

I just realised that there might be a problem with respect to theming and fill 
styles in KMyMoney for vertical scrollbars.

The “Home view shows a scrollbar on the right, which has the same grey 
background colour like the whole application. However, “Scheduled transactions” 
and “Legders” views have scrollbars which have *white* background. (See 
attached image snippets for “Home” and “Ledgers”.)

I make use of the qtcurve theme on OSX/MacPorts, but this also happens for the 
Oxygen theme, for instance.

Shall I file a ticket for this on b.k.o.?

Greets,
Marko


 


 ___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Theming issue?

2014-10-30 Thread Marko Käning

Hi Cristian,

On 30 Oct 2014, at 23:18 , Cristian Oneț onet.crist...@gmail.com wrote:
 Just check the Use system colors checkbox in the Color settings.

that solved it.

Hmm… Wondering why this is happening the way it is…

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [Kmymoney] KMyMoney 4.7.0, the first release candidate for 4.8, has been released

2014-10-03 Thread Marko Käning

Hi Cristian,

On 03 Oct 2014, at 07:29 , Cristian Oneț onet.crist...@gmail.com wrote:
 The old site is no longer maintained.

should we install a forward there, in order to avoid confusion for those
who reach the site via google or through an old bookmark?

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [Kmymoney] KMyMoney 4.7.0, the first release candidate for 4.8, has been released

2014-10-03 Thread Marko Käning

On 03 Oct 2014, at 09:28 , Cristian Oneț onet.crist...@gmail.com wrote:
 As I've said http://kmymoney2.sourceforge.net/ redirects to
 http://www.kmymoney.org/. If this does not happen please clear your
 cache.

Ah, ok, I missed that. I thought you were talking there about
index-home.html as well...


 http://kmymoney2.sourceforge.net/index-home.html is still there to
 access the old site. Thomas will take care of this properly, later.

So then my browser isn’t misbehaving re redirect. Good to know.


OK, I think the latter needs a link to the new site and a large message
telling that those pages still exist only for historical reasons, no?
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [Kmymoney] KMyMoney 4.7.0, the first release candidate for 4.8, has been released

2014-10-03 Thread Marko Käning

On 03 Oct 2014, at 09:38 , Cristian Oneț onet.crist...@gmail.com wrote:
 I guess so, I'll add that later. Meanwhile the new site is already
 third in the search results list [1].
 
 [1] https://www.google.com/#q=kmymoney
 ___

Hmm, for me - in Germany - it is the 1st hit!

:-/
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] [Kmymoney] KMyMoney 4.7.0, the first release candidate for 4.8, has been released

2014-10-03 Thread Marko Käning

Sorry, I have to correct myself!

http://kmymoney2.sourceforge.net/ 

is indeed the 1st, but it redirects, of course.

:-)



On 03 Oct 2014, at 09:48 , Marko Käning mk-li...@email.de wrote:

 On 03 Oct 2014, at 09:38 , Cristian Oneț onet.crist...@gmail.com wrote:
 I guess so, I'll add that later. Meanwhile the new site is already
 third in the search results list [1].
 
 [1] https://www.google.com/#q=kmymoney
 ___
 
 Hmm, for me - in Germany - it is the 1st hit!

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] OSX/CI: building kmymoney failed on branch frameworks

2014-09-03 Thread Marko Käning

Linking CXX executable kmymoney.app/Contents/MacOS/kmymoney
Undefined symbols for architecture x86_64:
  KGPGFile::keyExpires(QString const), referenced from:
  KMyMoneyApp::slotUpdateConfiguration() in kmymoney.cpp.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [kmymoney/kmymoney.app/Contents/MacOS/kmymoney] Error 1
make[1]: *** [kmymoney/CMakeFiles/kmymoney.dir/all] Error 2
make: *** [all] Error 2

KDE Continuous Integration Build
== Building Project: kmymoney - Branch frameworks

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Randa wrap-up

2014-08-19 Thread Marko Käning

Hi Cristian,

On 19 Aug 2014, at 22:11 , Cristian Oneț onet.crist...@gmail.com wrote:
 From what I've seen cross platform availability is taken pretty
 seriously which means that the work done by the Windows and the OSX
 teams are pretty important for Frameworks.

OK, good to hear.
Thibaut told me that he was the only OSX developer at Randa...
Well, I guess, the only thing I can do is to try to keep going until more
OSX devs come onboard. Right now we’ve got 2 to 3… :)


 I think you know that setting up the CI system on these platforms would be
 the most important step towards improving these platform's status in the
 community.

That’s why I exclusively worked on that front and didn’t care about getting
on with my one and only review request for KMyMoney during the last months!
:)


Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] OSX/CI: building kmymoney failed on branch frameworks

2014-08-19 Thread Marko Käning

Hi Cristian,

I rebuilt KF5 from scratch today and ran into this:

---

[ 50%] Built target mymoneyfinancialcalculatortest
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/reports/reportstestcommon.cpp:24:10:
 fatal error: 'kdeversion.h' file not found
#include kdeversion.h
 ^
[ 50%] [ 50%] Building CXX object 
kmymoney/mymoney/CMakeFiles/mymoneybudgettest.dir/mymoneybudgettest_automoc.cpp.o
Building CXX object 
kmymoney/mymoney/CMakeFiles/mymoneymoneytest.dir/mymoneymoneytest_automoc.cpp.o
[ 50%] Building CXX object 
kmymoney/mymoney/CMakeFiles/mymoneyobjecttest.dir/mymoneyobjecttest_automoc.cpp.o
Linking CXX executable mymoneybudgettest.app/Contents/MacOS/mymoneybudgettest
1 error generated.
make[2]: *** 
[kmymoney/mymoney/CMakeFiles/mymoneyforecasttest.dir/__/reports/reportstestcommon.cpp.o]
 Error 1
make[1]: *** [kmymoney/mymoney/CMakeFiles/mymoneyforecasttest.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs
Linking CXX executable mymoneymoneytest.app/Contents/MacOS/mymoneymoneytest
[ 50%] Built target mymoneybudgettest
Linking CXX executable mymoneyobjecttest.app/Contents/MacOS/mymoneyobjecttest
[ 50%] Built target mymoneymoneytest
[ 50%] Built target mymoneyobjecttest
Linking CXX executable mymoneyfiletest.app/Contents/MacOS/mymoneyfiletest
[ 50%] Built target mymoneyfiletest
make: *** [all] Error 2

---

Greets,
Marko





kmymoney.log.gz
Description: GNU Zip compressed data
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] OSX/CI system: KF5-kmymoney build broken on branch frameworks

2014-07-23 Thread Marko Käning

Hi Cristian,

On 23 Jul 2014, at 08:51 , Cristian Oneț onet.crist...@gmail.com wrote:
 The application exits because of a failed assertion [1] so this is
 intended.
ok, true. I could have seen it if I had checked on the relevant line of code, 
yes.


 Let me know if KMyMoney works on Mac if the session bus is running.
Same thing happens:
---
$ 
/opt/kde/install/darwin/mavericks/clang/kf5-qt5/extragear/office/kmymoney/inst/Applications/KF5/kmymoney.app/Contents/MacOS/kmymoney
 
Could not find drkonqi at 
/opt/kde/install/darwin/mavericks/clang/kf5-qt5/frameworks/kcrash/inst/lib/libexec/drkonqi
kmymoney: set session bus address to 
unix:path=/tmp/launch-F2kEmm/unix_domain_listener
Session bus not found 
To circumvent this problem try the following command (with Linux and bash) 
export $(dbus-launch)
Abort trap: 6
---

But well, I think this all might really be due to the fact that my 
QStandardPaths aren’t yet set correctly.

No problem.

:)

Thanks so far!
Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] OSX/CI system: KF5-kmymoney build broken on branch frameworks

2014-07-22 Thread Marko Käning

Hi,

On 22 Jul 2014, at 19:21 , Cristian Oneț onet.crist...@gmail.com wrote:
 This should be fixed now, please give it another try. Btw, the build
 of frameworks on Windows works already and the tests run successfully.

your change 67c3fe7dc0078766d2f59a44802bee40fc9bbb3f regarding GMP causes new 
problems for me:
---
Linking CXX static library libkgpgfile.a
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/storage/mymoneystoragexml.cpp:19:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/storage/mymoneystoragexml.h:32:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/storage/imymoneyserialize.h:37:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneyaccount.h:32:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneymoney.h:52:
/opt/kde/install/darwin/mavericks/clang/kf5-qt5/extragear/office/alkimia/inst/include/alkimia/alkvalue.h:23:10:
 fatal error: 'gmpxx.h' file not found
#include gmpxx.h // krazy:exclude=camelcase
 ^
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/plugins/interfaces/kmmimportinterface.cpp:29:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/kmymoney.h:35:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneyaccount.h:32:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneymoney.h:52:
/opt/kde/install/darwin/mavericks/clang/kf5-qt5/extragear/office/alkimia/inst/include/alkimia/alkvalue.h:23:10:
 fatal error: 'gmpxx.h' file not found
#include gmpxx.h // krazy:exclude=camelcase
 ^
[ 11%] In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/kmymoneyglobalsettings.cpp:34:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/widgets/kmymoneymvccombo.h:40:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneyschedule.h:32:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneytransaction.h:33:
In file included from 
/Users/marko/WC/KDECI-builds/kmymoney/kmymoney/mymoney/mymoneymoney.h:52:
/opt/kde/install/darwin/mavericks/clang/kf5-qt5/extragear/office/alkimia/inst/include/alkimia/alkvalue.h:23:10:
 fatal error: 'gmpxx.h' file not found
#include gmpxx.h // krazy:exclude=camelcase
 ^
Built target kgpgfile
---

Looks like you missed this somehow.

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] OSX/CI system: KF5-kmymoney build broken on branch frameworks

2014-07-22 Thread Marko Käning

Hi Cristian,

On 22 Jul 2014, at 21:22 , Cristian Oneț onet.crist...@gmail.com wrote:
 Could you update libalkimia and after that give this another try?

thanks to your efforts my build failure from before vanished after your change 
to alkimia:
---
MVM2:scripts marko$ ./install.sh kmymoney
--
Project: kmymoney
Calling prepare script
Calling build script
BUILD FAILED
==
MVM2:scripts marko$ ./install.sh alkimia
--
Project: alkimia
Calling prepare script
Calling build script
==
MVM2:scripts marko$ ./install.sh kmymoney rebuild
--
Project: kmymoney
Calling prepare script
Calling build script
==
---




___

 That means, KMyMoney’s KF5 port now SUCCESSFULLY BUILT FOR THE 1ST 
TIME ON OSX!!! :-D

___




You’re just amazing! Thanks for all your work on the frameworks port!

Greets,
Marko




P.S.: Yes, I also think it is very good that we now have 3 different compilers 
building KMM’s sources independently on different OSes! That increases our 
chances to catch things early, so one can hope at least. :)
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] OSX/CI system: KF5-kmymoney build broken on branch frameworks

2014-07-22 Thread Marko Käning

Hi Cristian,
Hi Ian,

On 22 Jul 2014, at 21:59 , Marko Käning mk-li...@email.de wrote:
 That means, KMyMoney’s KF5 port now SUCCESSFULLY BUILT FOR THE 1ST TIME ON 
 OSX!!! :-D

To be fully honest:

Well, unfortunately I cannot run it successfully:

---
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   libsystem_kernel.dylib  0x7fff85d44866 __pthread_kill + 10
1   libsystem_pthread.dylib 0x7fff87cac35c pthread_kill + 92
2   libsystem_c.dylib   0x7fff8c638b1a abort + 125
3   libQt5Core_debug.5.dylib0x000110c5aaa8 
qt_message_fatal(QtMsgType, QMessageLogContext const, QString const) + 24
4   libQt5Core_debug.5.dylib0x000110c5f038 
qt_message_output(QtMsgType, QMessageLogContext const, QString const) + 72 
(qlogging.cpp:1379)
5   libKF5KDELibs4Support.5.dylib   0x000112dfd9ce QDebug::~QDebug() + 
222 (qdebug.h:92)
6   libKF5KDELibs4Support.5.dylib   0x000112df8b85 QDebug::~QDebug() + 
21 (qdebug.h:94)
7   libKF5KDELibs4Support.5.dylib   0x000112edee28 
KApplicationPrivate::init(bool) + 456 (kapplication.cpp:416)
8   libKF5KDELibs4Support.5.dylib   0x000112edec02 
KApplication::KApplication(bool) + 258 (kapplication.cpp:311)
9   libKF5KDELibs4Support.5.dylib   0x000112edeaf5 
KApplication::KApplication(bool) + 37 (kapplication.cpp:311)
10  0x00010fa82613 main + 4579 
(main.cpp:118)
11  libdyld.dylib   0x7fff859d75fd start + 1
---

but this doesn’t surprise me at all, as other KF5 applications have similar 
problems!
Check out my bug report for kate [1].
There I also mention khelpcenter and konversation which manage to exit 
*gracefully* _without_ crashing.

KMM still belongs to the crashing apps, since it is silently assuming that dbus 
is present!

This all is very likely caused by the way my CI system separates all frameworks 
and applications in their own jails.
It looks as if the access to the dbus socket isn’t properly done, or whatever 
else… I haven’t investigated that yet.

But it may well be that I only need to finally adapt QStandardsPaths, to avoid 
other hassle, including the dbus issue.

That will be my next endeavour! :)

But, perhaps you can try to fix KMM so that it doesn’t rely blindly on dbus 
without checking (as kate obviously does) and bring it thus to a state like 
khelpcenter or konversation.

Qt's apps (Assistant.app etc.) start without problems, but when I try to start 
qdbusviewer.app it also says Error: Cannot connect to D-Bus: without more 
details! [1] This might be a not so hard nut for Ian to crack...

Anyway, I am very happy that we got this far already.

Greets,
Marko



[1] https://bugs.kde.org/show_bug.cgi?id=337140#c5






kmymoney_2014-07-22-220222_MVM2.crash.gz
Description: GNU Zip compressed data
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] OSX/CI system: KF5-kmymoney build broken on branch frameworks

2014-07-21 Thread Marko Käning

Hi Cristian,

now that kdepimlibs can be removed as a necessary dependency I end up in a 
problem with KDChart:
---
[ 12%] [ 12%] [ 12%] [ 12%] [ 12%] [ 12%] [ 12%] Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractCartesianDiagram.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAreaBase.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractArea.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/ChartGraphicsItem.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAxis.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAreaWidget.cpp.o
Building CXX object 
libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractCoordinatePlane.cpp.o
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractCartesianDiagram.cpp.o]
 Error 1
make[2]: *** Waiting for unfinished jobs
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAreaBase.cpp.o] Error 
1
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAxis.cpp.o] Error 1
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractArea.cpp.o] Error 1
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractAreaWidget.cpp.o] 
Error 1
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/ChartGraphicsItem.cpp.o] Error 1
clang: error: unknown argument: '-fno-reorder-blocks' 
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning) in 
the future
make[2]: *** 
[libkdchart/src/CMakeFiles/kmm_kdchart.dir/KDChartAbstractCoordinatePlane.cpp.o]
 Error 1
make[1]: *** [libkdchart/src/CMakeFiles/kmm_kdchart.dir/all] Error 2
make: *** [all] Error 2
---

Any idea what can be done about this?

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119350: Add konversation to kf5-qt5

2014-07-18 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119350/#review62622
---



dependency-data-kf5-qt5
https://git.reviewboard.kde.org/r/119350/#comment43418

I guess, I can omit [frameworks] here, can't I?


- Marko Käning


On July 18, 2014, 7:29 a.m., Marko Käning wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119350/
 ---
 
 (Updated July 18, 2014, 7:29 a.m.)
 
 
 Review request for KMymoney, Ben Cooksley, Eike Hein, and Michael Pyne.
 
 
 Repository: kde-build-metadata
 
 
 Description
 ---
 
 Add KF5-konversation to CI system
 
 
 Diffs
 -
 
   dependency-data-kf5-qt5 4753582339538b8235c64d56ec7ee91965d40006 
 
 Diff: https://git.reviewboard.kde.org/r/119350/diff/
 
 
 Testing
 ---
 
 Builds fine (including qca master)
 
 konversation doesn't run successfully with qca2, yet, though.
 
 
 Thanks,
 
 Marko Käning
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-18 Thread Marko Käning

Hi Cristian,

I see that Jenkins finally picks up KMM on KF5! :)

Weird, that it can’t find alkimia though:

06:21:24 In file included from 
/srv/jenkins/workspace/kmymoney_frameworks_qt5/kmymoney/mymoney/mymoneytransaction.h:33:0,
06:21:24  from 
/srv/jenkins/workspace/kmymoney_frameworks_qt5/kmymoney/mymoney/mymoneyschedule.h:32,
06:21:24  from 
/srv/jenkins/workspace/kmymoney_frameworks_qt5/kmymoney/widgets/kmymoneymvccombo.h:40,
06:21:24  from 
/srv/jenkins/workspace/kmymoney_frameworks_qt5/kmymoney/kmymoneyglobalsettings.cpp:34:
06:21:24 
/srv/jenkins/workspace/kmymoney_frameworks_qt5/kmymoney/mymoney/mymoneymoney.h:51:30:
 fatal error: alkimia/alkvalue.h: No such file or directory
06:21:24  #include alkimia/alkvalue.h
06:21:24   ^
06:21:24 compilation terminated.
06:21:24 make[2]: *** 
[kmymoney/CMakeFiles/kmm_config.dir/kmymoneyglobalsettings.cpp.o] Error 1

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-18 Thread Marko Käning

On 18 Jul 2014, at 08:40 , Cristian Oneț onet.crist...@gmail.com wrote:
 Yeah, I know, I'm working on that.

I see.

Good luck!

Bye for now.
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119350: Add konversation to kf5-qt5

2014-07-18 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119350/
---

(Updated July 18, 2014, 8:53 a.m.)


Review request for KMymoney, Ben Cooksley, Eike Hein, and Michael Pyne.


Changes
---

It should work like this now. But I guess I am waiting for Ben to ok this.


Repository: kde-build-metadata


Description
---

Add KF5-konversation to CI system


Diffs (updated)
-

  dependency-data-kf5-qt5 4753582339538b8235c64d56ec7ee91965d40006 

Diff: https://git.reviewboard.kde.org/r/119350/diff/


Testing
---

Builds fine (including qca master)

konversation doesn't run successfully with qca2, yet, though.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119350: Add konversation to kf5-qt5

2014-07-18 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119350/
---

(Updated July 18, 2014, 6:57 a.m.)


Status
--

This change has been marked as submitted.


Review request for KMymoney, Ben Cooksley, Eike Hein, and Michael Pyne.


Repository: kde-build-metadata


Description
---

Add KF5-konversation to CI system


Diffs
-

  dependency-data-kf5-qt5 4753582339538b8235c64d56ec7ee91965d40006 

Diff: https://git.reviewboard.kde.org/r/119350/diff/


Testing
---

Builds fine (including qca master)

konversation doesn't run successfully with qca2, yet, though.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-18 Thread Marko Käning

Hi Cristian,

On 18 Jul 2014, at 19:26 , Cristian Oneț onet.crist...@gmail.com wrote:
 So... the first successful build is available [1]. This is a good
 starting point for the port, there are only 1097 warnings to fix [2]
 :).

wow, what an achievement! Congratulations to you!!!


 Btw, could we set up an email notification for build failures to this
 list? I saw that other builds have something similar.

Well, perhaps Ben can give us information how that can be achieved.

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning



 On July 17, 2014, 6:39 a.m., Ben Cooksley wrote:
  dependency-data-kf5-qt5, line 617
  https://git.reviewboard.kde.org/r/119305/diff/2/?file=290658#file290658line617
 
  Please use frameworks/kf5umbrella - which includes all Frameworks 
  (incl. KActivities and the former parts of kdelibs - but not kdepimlibs)

Isn't this already covered by the following generic dep defined right after 
kf5umbrella:

# The generic dependency on all frameworks
kde/*: frameworks/kf5umbrella
extragear/*: frameworks/kf5umbrella

???

So, I guess simply deleting 2 lines should be it.


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/#review62540
---


On July 16, 2014, 11:05 p.m., Marko Käning wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119305/
 ---
 
 (Updated July 16, 2014, 11:05 p.m.)
 
 
 Review request for KMymoney, Ben Cooksley and Michael Pyne.
 
 
 Repository: kde-build-metadata
 
 
 Description
 ---
 
 Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
 also missing on the linux system for KF5...
 
 
 Diffs
 -
 
   dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 
 
 Diff: https://git.reviewboard.kde.org/r/119305/diff/
 
 
 Testing
 ---
 
 It doesn't build yet, since I don't know how to handle the optional deps to 
 frameworks Holidays, Gpgmepp and QGpgme.
 
 Alkimia is still not building on OSX, though. (Its logical dependency 
 structure already exists.) 
 
 I'll post the problem on kmymoney-devel.
 
 
 Thanks,
 
 Marko Käning
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

(Updated July 17, 2014, 9:04 a.m.)


Review request for KMymoney, Ben Cooksley and Michael Pyne.


Repository: kde-build-metadata


Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
also missing on the linux system for KF5...


Diffs (updated)
-

  dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 

Diff: https://git.reviewboard.kde.org/r/119305/diff/


Testing
---

It doesn't build yet, since I don't know how to handle the optional deps to 
frameworks Holidays, Gpgmepp and QGpgme.

Alkimia is still not building on OSX, though. (Its logical dependency structure 
already exists.) 

I'll post the problem on kmymoney-devel.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning



 On July 17, 2014, 6:39 a.m., Ben Cooksley wrote:
  dependency-data-kf5-qt5, line 617
  https://git.reviewboard.kde.org/r/119305/diff/2/?file=290658#file290658line617
 
  Please use frameworks/kf5umbrella - which includes all Frameworks 
  (incl. KActivities and the former parts of kdelibs - but not kdepimlibs)
 
 Marko Käning wrote:
 Isn't this already covered by the following generic dep defined right 
 after kf5umbrella:
 
 # The generic dependency on all frameworks
 kde/*: frameworks/kf5umbrella
 extragear/*: frameworks/kf5umbrella
 
 ???
 
 So, I guess simply deleting 2 lines should be it.
 
 Ben Cooksley wrote:
 Indeed. All that should be needed is the dependency on alkimia.

AND kdepimlibs.


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/#review62540
---


On July 17, 2014, 9:04 a.m., Marko Käning wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119305/
 ---
 
 (Updated July 17, 2014, 9:04 a.m.)
 
 
 Review request for KMymoney, Ben Cooksley and Michael Pyne.
 
 
 Repository: kde-build-metadata
 
 
 Description
 ---
 
 Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
 also missing on the linux system for KF5...
 
 
 Diffs
 -
 
   dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 
 
 Diff: https://git.reviewboard.kde.org/r/119305/diff/
 
 
 Testing
 ---
 
 It doesn't build yet, since I don't know how to handle the optional deps to 
 frameworks Holidays, Gpgmepp and QGpgme.
 
 Alkimia is still not building on OSX, though. (Its logical dependency 
 structure already exists.) 
 
 I'll post the problem on kmymoney-devel.
 
 
 Thanks,
 
 Marko Käning
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

(Updated July 17, 2014, 9:31 a.m.)


Review request for KMymoney, Ben Cooksley and Michael Pyne.


Changes
---

Can this be committed as is? Then we might get at least a successful Linux 
build of KMM...


Repository: kde-build-metadata


Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
also missing on the linux system for KF5...


Diffs
-

  dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 

Diff: https://git.reviewboard.kde.org/r/119305/diff/


Testing (updated)
---

It doesn't build yet, since alkimia is still not building on OSX. (It's being 
dealt with by the KMM devs atm.)


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

(Updated July 17, 2014, 9:20 p.m.)


Status
--

This change has been marked as submitted.


Review request for KMymoney, Ben Cooksley and Michael Pyne.


Repository: kde-build-metadata


Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
also missing on the linux system for KF5...


Diffs
-

  dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 

Diff: https://git.reviewboard.kde.org/r/119305/diff/


Testing
---

It doesn't build yet, since alkimia is still not building on OSX. (It's being 
dealt with by the KMM devs atm.)


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] [kde-build-metadata] /: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-17 Thread Marko Käning

Git commit 22f893701945c04cc5b0405d1a17eff948668d59 by Marko Käning.
Committed on 17/07/2014 at 21:16.
Pushed by kaning into branch 'master'.

Add kmymoney to dependency data for kf5-qt5 branch group

Herewith it is intended to bring kmymoney to the OSX/CI system, but it
is still also missing on Linux' CI system for KF5.

CCMAIL: kmymoney-devel@kde.org
REVIEW: 119305

M  +4-0dependency-data-kf5-qt5

http://commits.kde.org/kde-build-metadata/22f893701945c04cc5b0405d1a17eff948668d59

diff --git a/dependency-data-kf5-qt5 b/dependency-data-kf5-qt5
index 6819771..4753582 100644
--- a/dependency-data-kf5-qt5
+++ b/dependency-data-kf5-qt5
@@ -609,3 +609,7 @@ extragear/base/kscreen[frameworks]: 
extragear/libs/libkscreen[frameworks]
 
 # kdeedu
 kde/kdeedu/kalgebra: kde/kdeedu/analitza
+
+# kmymoney
+extragear/office/kmymoney: extragear/office/alkimia
+extragear/office/kmymoney: kde/kdepimlibs
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-17 Thread Marko Käning

Hi Cristian,

On 17 Jul 2014, at 18:16 , Cristian Oneț onet.crist...@gmail.com wrote:
 Done
 http://commits.kde.org/alkimia/7fe269fae63d0b5786ee8953460e8e0faf564fb5

that was easy. Cool!

alkimia now builds also on OSX!

THANKS!!!

Greets,
Marko
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] Review Request 119350: Add konversation to kf5-qt5

2014-07-17 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119350/
---

Review request for KMymoney, Ben Cooksley, Eike Hein, and Michael Pyne.


Repository: kde-build-metadata


Description
---

Add KF5-konversation to CI system


Diffs
-

  dependency-data-kf5-qt5 4753582339538b8235c64d56ec7ee91965d40006 

Diff: https://git.reviewboard.kde.org/r/119350/diff/


Testing
---

Builds fine (including qca master)

konversation doesn't run successfully with qca2, yet, though.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-16 Thread Marko Käning

On July 16, 2014, 2:52 a.m., Michael Pyne wrote:
dependency-data-kf5-qt5, line 616
https://git.reviewboard.kde.org/r/119305/diff/1/?file=290515#file290515line616

I'm assuming by Holidays you're referring to kdepimlibs, so that can
be specified here (and appears to be a required, not optional, dependency):

extragear/office/kmymoney: kde/kdepimlibs

This would also take care of QGpgme.

As far as Gpgmepp, that is a non-KDE dependency, isn't it? We only
track KDE projects within this metadata file, external dependencies are
required to be installed separately within the CI system.

Marko Käning wrote:
Ok, I thought that those 3 deps would be separate FWs.

Cristian Oneț wrote:
Actually Gpgmepp is a KDE dependency since it's part of kdepimlibs
https://projects.kde.org/projects/kde/kdepimlibs/repository/revisions/frameworks/entry/gpgme++/CMakeLists.txt#L54
so if kdepimlibs is really added as a dependency (btw I don see kdepimlibs
here http://build.kde.org/job/kmymoney_frameworks_qt5/1/console) then it
should build fine.

I hope the 'make the dependency optional' part will come but plese note
that it's not yet committed. Anyway, even when it will it's still good to
build with kdepimlibs present when possible.

Yes, I follow the discussion on KMYMONEY-DEVEL and am aware of that it his in
question whether the use of an optional dependency is going to be allowed.

See https://git.reviewboard.kde.org/r/119308/ where I have added the logical
dependency information for KF5-kdepimlibs.

- Marko

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/#review62454
---

On July 16, 2014, 2:32 a.m., Marko Käning wrote:

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

(Updated July 16, 2014, 2:32 a.m.)

Review request for KMymoney, Ben Cooksley and Michael Pyne.

Repository: kde-build-metadata

Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still
also missing on the linux system for KF5...

Diffs
-

dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e

Diff: https://git.reviewboard.kde.org/r/119305/diff/

Testing
---

It doesn't build yet, since I don't know how to handle the optional deps to
frameworks Holidays, Gpgmepp and QGpgme.

Alkimia is still not building on OSX, though. (Its logical dependency
structure already exists.)

I'll post the problem on kmymoney-devel.

Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-16 Thread Marko Käning

Hi Cristian,

On 16 Jul 2014, at 06:53 , Cristian Oneț onet.crist...@gmail.com wrote:
 Those are libraries from kdepimlibs and should be handled by adding
 kdepimlibs[frameworks] as a dependency.

yes, I know. I’ve taken care of this in [1] just now.


 ... I hope you can help to fix a problem I met on OSX with the alkimia build 
 system
 
 Ben pointed this out:
 
 ---
 
 On 15 Jul 2014, at 12:50 , Ben Cooksley bcooks...@kde.org wrote:
 This is a fairly usual split-installation-prefix build failure then :)
 You need to ask the libalkimia developers to adjust the CMake code to
 ensure the GMP include paths are added to include_directories()
 appropriately.


Cristian, any idea about this ^^^ ?

Greets,
Marko




[1] https://git.reviewboard.kde.org/r/119308/
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-16 Thread Marko Käning

Hi Cristian,

On 16 Jul 2014, at 09:21 , Cristian Oneț onet.crist...@gmail.com wrote:
 Thanks,

you are welcome. I am happy about YOUR efforts to bring KMM to KF5!


 I'll fix this, I thought [1] is enough (add gmp to
 target_link_libraries) but I guess that it's true only for frameworks.
 
 [1] 
 https://projects.kde.org/projects/extragear/office/alkimia/repository/revisions/master/entry/libalkimia/CMakeLists.txt#L67

Well, it looks like that line is not enough ON branch *frameworks* where I have 
to build this...


 AFAIK adding used frameworks to the include directories is not
 necessary since this is deduced from target_link_libraries in cmake 3,
 please correct me if this is wrong.


I have no idea - atm - since I am (admittedly) more or less blindly just trying 
to set up
the OSX/CI system in such a way that it handles KMM’s dependencies the way it 
handles all
the other deps needed for the projects beyond KF5, which got eventually 
successfully build [2].

Greets,
Marko


[2] 
https://trac.macports.org/wiki/KDEProblems/KDEMacPortsCI/Status#KF5softwarebeyondthebasicframeworks
[3] https://git.reviewboard.kde.org/r/119308/
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-16 Thread Marko Käning

Ahh, well, kdepimlibs doesn’t build anyway atm [1]!

Greets,
Marko



[1] http://build.kde.org/job/kdepimlibs_frameworks_qt5/50/console
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-16 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

(Updated July 16, 2014, 11:05 p.m.)


Review request for KMymoney, Ben Cooksley and Michael Pyne.


Changes
---

Adapted dependency-data in order to allow building kmymoney on the CI system. 
(Can't test whether this works, as alkimia currently doesn't build on OSX.)


Repository: kde-build-metadata


Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
also missing on the linux system for KF5...


Diffs (updated)
-

  dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 

Diff: https://git.reviewboard.kde.org/r/119305/diff/


Testing
---

It doesn't build yet, since I don't know how to handle the optional deps to 
frameworks Holidays, Gpgmepp and QGpgme.

Alkimia is still not building on OSX, though. (Its logical dependency structure 
already exists.) 

I'll post the problem on kmymoney-devel.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-16 Thread Marko Käning



 On July 16, 2014, 2:52 a.m., Michael Pyne wrote:
  dependency-data-kf5-qt5, line 614
  https://git.reviewboard.kde.org/r/119305/diff/1/?file=290515#file290515line614
 
  You shouldn't need to specify a branch manually here (the part in and 
  including the brackets) since the appropriate branch should be able to be 
  found in logical-module-structure. This was only really required for the 
  old/deprecated dependency-data file which mixed KDE4 and KF5 dependencies.
  
  I just checked and kmymoney does indeed already include that 
  information (and so does alkimia).
 
 Marko Käning wrote:
 Ah, ok, so that is already working then...

OK, I've basically copied the entries from dependency-data into 
dependency-data-kf5-qt5, which should add the missing stuff for the CI system.


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/#review62454
---


On July 16, 2014, 11:05 p.m., Marko Käning wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119305/
 ---
 
 (Updated July 16, 2014, 11:05 p.m.)
 
 
 Review request for KMymoney, Ben Cooksley and Michael Pyne.
 
 
 Repository: kde-build-metadata
 
 
 Description
 ---
 
 Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
 also missing on the linux system for KF5...
 
 
 Diffs
 -
 
   dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 
 
 Diff: https://git.reviewboard.kde.org/r/119305/diff/
 
 
 Testing
 ---
 
 It doesn't build yet, since I don't know how to handle the optional deps to 
 frameworks Holidays, Gpgmepp and QGpgme.
 
 Alkimia is still not building on OSX, though. (Its logical dependency 
 structure already exists.) 
 
 I'll post the problem on kmymoney-devel.
 
 
 Thanks,
 
 Marko Käning
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-16 Thread Marko Käning

On 16 Jul 2014, at 14:17 , Ben Cooksley bcooks...@kde.org wrote:
 As long as we have one recent successful build, we can proceed.

Oh, we do have one in [1]!

Cool.



[1] http://build.kde.org/job/kdepimlibs_frameworks_qt5/
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

[Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-15 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/
---

Review request for KMymoney, Ben Cooksley and Michael Pyne.


Repository: kde-build-metadata


Description
---

Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
also missing on the linux system for KF5...


Diffs
-

  dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 

Diff: https://git.reviewboard.kde.org/r/119305/diff/


Testing
---

It doesn't build yet, since I don't know how to handle the optional deps to 
frameworks Holidays, Gpgmepp and QGpgme.

Alkimia is still not building on OSX, though. (Its logical dependency structure 
already exists.) 

I'll post the problem on kmymoney-devel.


Thanks,

Marko Käning

___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Frameworks

2014-07-15 Thread Marko Käning

Hi Cristian  Alvaro,

KMyMoney was built today for the first time as KF5 on jenkins.

What’s also a bit odd: it is still missing alkimia, which I added as a 
dependency in [2].

KMM itself doesn't build anyway, since I don't know _yet_ how to handle its 
optional dependencies (frameworks Holidays, Gpgmepp and QGpgme) on the CI 
system...


Apart from that I hope you can help to fix a problem I met on OSX with the 
alkimia build system as discussed in [3].

Ben pointed this out:

---

On 15 Jul 2014, at 12:50 , Ben Cooksley bcooks...@kde.org wrote:
 This is a fairly usual split-installation-prefix build failure then :)
 You need to ask the libalkimia developers to adjust the CMake code to
 ensure the GMP include paths are added to include_directories()
 appropriately.


---

Greets,
Marko

[1] http://build.kde.org/job/kmymoney_frameworks_qt5/1/console
[2] https://git.reviewboard.kde.org/r/119305/
[3] http://mail.kde.org/pipermail/kde-frameworks-devel/2014-July/017583.html
___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119305: Add kmymoney to dependency data for kf5-qt5 branch group

2014-07-15 Thread Marko Käning



 On July 16, 2014, 2:52 a.m., Michael Pyne wrote:
 

So, basically, you're saying that this whole RR can be discarded, because 
everything is already defined in file dependency-data, right!?
Good news. :)


 On July 16, 2014, 2:52 a.m., Michael Pyne wrote:
  dependency-data-kf5-qt5, line 614
  https://git.reviewboard.kde.org/r/119305/diff/1/?file=290515#file290515line614
 
  You shouldn't need to specify a branch manually here (the part in and 
  including the brackets) since the appropriate branch should be able to be 
  found in logical-module-structure. This was only really required for the 
  old/deprecated dependency-data file which mixed KDE4 and KF5 dependencies.
  
  I just checked and kmymoney does indeed already include that 
  information (and so does alkimia).

Ah, ok, so that is already working then...


 On July 16, 2014, 2:52 a.m., Michael Pyne wrote:
  dependency-data-kf5-qt5, line 616
  https://git.reviewboard.kde.org/r/119305/diff/1/?file=290515#file290515line616
 
  I'm assuming by Holidays you're referring to kdepimlibs, so that can 
  be specified here (and appears to be a required, not optional, dependency):
  
  extragear/office/kmymoney: kde/kdepimlibs
  
  This would also take care of QGpgme.
  
  As far as Gpgmepp, that is a non-KDE dependency, isn't it? We only 
  track KDE projects within this metadata file, external dependencies are 
  required to be installed separately within the CI system.

Ok, I thought that those 3 deps would be separate FWs.


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119305/#review62454
---


On July 16, 2014, 2:32 a.m., Marko Käning wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119305/
 ---
 
 (Updated July 16, 2014, 2:32 a.m.)
 
 
 Review request for KMymoney, Ben Cooksley and Michael Pyne.
 
 
 Repository: kde-build-metadata
 
 
 Description
 ---
 
 Herewith I intended to bring kmymoney to the OSX/CI system, but it is still 
 also missing on the linux system for KF5...
 
 
 Diffs
 -
 
   dependency-data-kf5-qt5 6819771be9eeddd37586570dd63b5fe9115c554e 
 
 Diff: https://git.reviewboard.kde.org/r/119305/diff/
 
 
 Testing
 ---
 
 It doesn't build yet, since I don't know how to handle the optional deps to 
 frameworks Holidays, Gpgmepp and QGpgme.
 
 Alkimia is still not building on OSX, though. (Its logical dependency 
 structure already exists.) 
 
 I'll post the problem on kmymoney-devel.
 
 
 Thanks,
 
 Marko Käning
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119234: Transform KHolidays into an optional dependency.

2014-07-12 Thread Marko Käning



 On July 11, 2014, 9:58 p.m., Christian David wrote:
  Is this only used until the pim libs are KF5 ready? Because I think this 
  function is so important that it justifies having the pim libs a obligatory 
  dependency. There will be many bug reports if a package maintainer forgets 
  to compile with kdepimlibs.
 
 Cristian Oneț wrote:
 No, I would like to see this as a permanent change. I'm not that sure 
 about which of the features are so important that they must be a mandatory 
 dependency. I like the idea of having the freedom to build without kdepimlibs 
 if necessary and if that can be obtained easily why not have it? I'm not 
 sugesting that packagers should do this but for example in source based 
 distributions users would be able to have a choice, which is nice.
 
 Alvaro Soliverez wrote:
 I'm against this being a permanent change. It affects how schedules are 
 handled, which impacts all over the application.
 And I certainly don't want to track down a bug reported by an unsupecting 
 user just to figure out later a packager left the dependency out because kde 
 pim libs was too big.
 
 Cristian Oneț wrote:
 I hate to disagree but I don't see it that way. The only effect to the 
 way schedules are handled is that processing dates are not obtained from the 
 holliday region thus the old (and documented) behavior of considering only 
 weekends as non-processing days will be in effect.
 
 If you ask me is this feature really that crucial that we should make 
 everyone build kdepimlibs in oder to use kmymoney I would say no. Again, I'm 
 not removing the feature, just making it optional.
 
 The feature isn't even documented so I'm not sure if we would start 
 receiving bug reports about it. But just to be on the safe side we could 
 finish the feature in main.cpp:69 which would provide a list of optional 
 features that were built (or found as plugins) in the about data.
 
 If it's something I've learned from and appreacited in the KF5 model is 
 that the take everything or leave it approach is wrong. Think about it, 
 kdepimlibs has 22 libraries plus their dependecies , of which we only use 4 
 (QGpgme, KHolidays, PIMIdentities, Akonadi) for pretty slim features. Knowing 
 this wouldn't it be wiser to allow those who don't need those features to be 
 able to use the application without these dependencies?

Cristian, I am very much in favour for what you're stating here. This is 
something which one wouldn't be too worried about on Linux, but on OSX as on 
Windows in your case it is a real show-stopper to be forced into having far too 
many dependencies for seemingly little functionality gain.

Thus, I also vote for making things optional!


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119234/#review62153
---


On July 11, 2014, 9:58 p.m., Cristian Oneț wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119234/
 ---
 
 (Updated July 11, 2014, 9:58 p.m.)
 
 
 Review request for KMymoney.
 
 
 Repository: kmymoney
 
 
 Description
 ---
 
 If KDE PIM libraries will not be found during configuration all days will be 
 considered processing days. Note that this review request dependes on the 
 cmake changes in https://git.reviewboard.kde.org/r/119207/
 
 
 Diffs
 -
 
   kmymoney/dialogs/settings/ksettingsschedules.cpp 96bfb84 
   kmymoney/kmymoney.cpp 687187a 
 
 Diff: https://git.reviewboard.kde.org/r/119234/diff/
 
 
 Testing
 ---
 
 Build and run the application with and without HAVE_KDEPIMLIBS defined.
 
 
 Thanks,
 
 Cristian Oneț
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119207: Refactor the contact fetching code into a single class.

2014-07-10 Thread Marko Käning



 On July 10, 2014, 7:47 a.m., Marko Käning wrote:
  Hi Cristian, can you supply also a diff for branch frameworks, or is that 
  still too early to ask for? Greets, Marko
 
 Cristian Oneț wrote:
 I posted this review request against master since I think that master 
 should also contain this improvement. If it will be pushed then I'll pick it 
 into frameworks without going trough the review process.

Ah, ok.

Things like KF5QGpgme are going to follow later, I figure, right?


- Marko


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119207/#review62037
---


On July 10, 2014, 7:24 a.m., Cristian Oneț wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119207/
 ---
 
 (Updated July 10, 2014, 7:24 a.m.)
 
 
 Review request for KMymoney.
 
 
 Repository: kmymoney
 
 
 Description
 ---
 
 Refactor the contact fetching code into a single class.
 
 This avoids code duplication and makes it easier to transform
 the kdepimlibs dependency into an optional one. If kdepimlibs
 is not detected the user will not be able to load contact data
 from the address book (big deal...).
 
 It also makes it easier to fetch contact data from other parts of the
 code, like for a payee.
 
 I'm not sure about putting this into the kmm_mymoney library but since
 it only contains a data source and we don't have another lib for these
 kind of objects it ended up here.
 
 
 Diffs
 -
 
   kmymoney/wizards/newuserwizard/knewuserwizard.cpp 
 7fd250cfc46db505dd8a1b595a2179c91e0183d9 
   kmymoney/wizards/newuserwizard/knewuserwizard_p.h 
 adb4ca564cdd4b70e8cb4c4f1d053c0bd6335c38 
   kmymoney/mymoney/CMakeLists.txt 62f29e88017fae38b1da8f2a18cb35c5b627b573 
   kmymoney/mymoney/mymoneycontact.h PRE-CREATION 
   kmymoney/mymoney/mymoneycontact.cpp PRE-CREATION 
   config-kmymoney.h.cmake 5f3845a46413259d60302ba1b921a266ee3bb1f9 
   CMakeLists.txt dacdf606a9320c1926c4d848d0917708f8cdd44f 
   kmymoney/dialogs/knewfiledlg.h 92004714c8bedf8b1cf15bfc1e0f20edcab3a66c 
   kmymoney/dialogs/knewfiledlg.cpp 5b6e8e960184235e3f6bc61439432ea48855a4f4 
 
 Diff: https://git.reviewboard.kde.org/r/119207/diff/
 
 
 Testing
 ---
 
 Tested the Load from addressbook feature, it works. Also compiled the 
 module without finding the kdepimlibs module, it worked.
 
 
 Thanks,
 
 Cristian Oneț
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Re: [Kmymoney-devel] Review Request 119207: Refactor the contact fetching code into a single class.

2014-07-09 Thread Marko Käning


---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119207/#review62037
---


Hi Cristian, can you supply also a diff for branch frameworks, or is that still 
too early to ask for? Greets, Marko

- Marko Käning


On July 10, 2014, 7:24 a.m., Cristian Oneț wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/119207/
 ---
 
 (Updated July 10, 2014, 7:24 a.m.)
 
 
 Review request for KMymoney.
 
 
 Repository: kmymoney
 
 
 Description
 ---
 
 Refactor the contact fetching code into a single class.
 
 This avoids code duplication and makes it easier to transform
 the kdepimlibs dependency into an optional one. If kdepimlibs
 is not detected the user will not be able to load contact data
 from the address book (big deal...).
 
 It also makes it easier to fetch contact data from other parts of the
 code, like for a payee.
 
 I'm not sure about putting this into the kmm_mymoney library but since
 it only contains a data source and we don't have another lib for these
 kind of objects it ended up here.
 
 
 Diffs
 -
 
   kmymoney/wizards/newuserwizard/knewuserwizard.cpp 
 7fd250cfc46db505dd8a1b595a2179c91e0183d9 
   kmymoney/wizards/newuserwizard/knewuserwizard_p.h 
 adb4ca564cdd4b70e8cb4c4f1d053c0bd6335c38 
   kmymoney/mymoney/CMakeLists.txt 62f29e88017fae38b1da8f2a18cb35c5b627b573 
   kmymoney/mymoney/mymoneycontact.h PRE-CREATION 
   kmymoney/mymoney/mymoneycontact.cpp PRE-CREATION 
   config-kmymoney.h.cmake 5f3845a46413259d60302ba1b921a266ee3bb1f9 
   CMakeLists.txt dacdf606a9320c1926c4d848d0917708f8cdd44f 
   kmymoney/dialogs/knewfiledlg.h 92004714c8bedf8b1cf15bfc1e0f20edcab3a66c 
   kmymoney/dialogs/knewfiledlg.cpp 5b6e8e960184235e3f6bc61439432ea48855a4f4 
 
 Diff: https://git.reviewboard.kde.org/r/119207/diff/
 
 
 Testing
 ---
 
 Tested the Load from addressbook feature, it works. Also compiled the 
 module without finding the kdepimlibs module, it worked.
 
 
 Thanks,
 
 Cristian Oneț
 


___
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

1 2 3 4 >

1 - 100 of 329 matches

Mail list logo