[Bug 1042760] New: Added CMake files to AqBanking and Gwenhywfar
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760 Bug ID: 1042760 Summary: Added CMake files to AqBanking and Gwenhywfar Classification: openSUSE Product: openSUSE Distribution Version: Leap 42.2 Hardware: Other OS: openSUSE 42.2 Status: NEW Severity: Enhancement Priority: P5 - None Component: KDE Applications Assignee: opensuse-kde-bugs@opensuse.org Reporter: joachim.langenbach@engsas.de QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- Backported CMake files (aqbanking-config.cmake, ...) to aqbanking 5.1 and gwenhywfar 4.13 to allow building kmymoney with kbanking plugin enabled. See https://build.opensuse.org/request/show/501167 for code changes -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c1
--- Comment #1 from Joachim Langenbach
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c2
Andreas Stieger
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c3
Wolfgang Bauer
And the ultimate goal is to enable the kbanking plugin for kmymoney via a Leap 42.2 maintenance update? (SR#501167 is for the devel package, but I assume this is for maintenance later?)
No. kmymoney 4.7.2 as shipped in Leap 42.2 does have the kbanking plugin enabled. This is about the latest 4.8.0, which is only available in KDE:Extra, that normally needs a newer aqbanking and gwenhywfar than shipped in Leap 42. (the ultimate goal probably is to update it for 42.3 then...) -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c4
--- Comment #4 from Andreas Stieger
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c5
--- Comment #5 from Wolfgang Bauer
I think it would be better if you could host the required library changes in KDE:Extra (for distribution target <= 42.2 that need it), rather than doing a maintenance update.
I suppose we could, yes. There's just one "problem": different versions are not co-installable. And aqbanking in particular is also used by other things in the distribution, e.g. gnucash. This leads to conflicts, i.e. you have to uninstall the other users if you want to install the latest version. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c6
--- Comment #6 from Wolfgang Bauer
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c7
Wolfgang Bauer
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c8
--- Comment #8 from Wolfgang Bauer
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c9
Wolfgang Bauer
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c10
--- Comment #10 from Joachim Langenbach
The modified kmymoney package is in KDE:Extra now.
This also means that there's no point in modifying aqbanking/gwenhywfar in/for Leap 42 any more.
Please excuse my delay. So much holidays in bavaria ;-) Thanks for your work and adding the missing definition switch for aqbanking version. I have tested it today and it seems to work. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c11
--- Comment #11 from Wolfgang Bauer
I have tested it today and it seems to work.
Great! But I noticed a possible mistake in the patch just now: +// following the gnucash code, aqbnkaing prior 5.2 does support +// identical functions, but per transaction type (sepa, debitnote, ...). +// Therefore I disabled debug output before aqbanking 5.2 +#ifdef AQBANKING_VERSION_GREATER_5_2_0 qDebug() << "SetTransaction: " << AB_Job_SetTransaction(abJob, abTransaction); +#endif Disabling this "debug output" also disables the call to AB_Job_SetTransaction(). Is this intended? Might this not cause problems? gnucash has this in gnc_ab_get_trans_job(): #ifdef AQBANKING_VERSION_GREATER_5_2_0 AB_Job_SetTransaction(job, ab_trans); #else switch (trans_type) { case SINGLE_DEBITNOTE: AB_JobSingleDebitNote_SetTransaction(job, ab_trans); break; case SINGLE_INTERNAL_TRANSFER: AB_JobInternalTransfer_SetTransaction(job, ab_trans); break; case SEPA_TRANSFER: AB_JobSepaTransfer_SetTransaction(job, ab_trans); break; case SEPA_DEBITNOTE: AB_JobSepaDebitNote_SetTransaction(job, ab_trans); break; case SINGLE_TRANSFER: default: AB_JobSingleTransfer_SetTransaction(job, ab_trans); break; }; #endif (https://code.gnucash.org/docs/MASTER/group__AqBanking.html#ga4f0bc157ef6c46c...) So I suppose the patch should call AB_JobSingleTransfer_SetTransaction(abJob, abTransaction) or AB_JobSepaTransfer_SetTransaction(abjob, abTransaction) if aqbanking <= 5.2.0? (see https://build.opensuse.org/package/rdiff/home:wolfi323:branches:KDE:Extra/kmymoney?opackage=kmymoney&oproject=KDE%3AExtra&rev=5 ) I saw that you mentioned a problem with sending Sepa transfers on the kmymoney mailinglist. That may very well be the reason... I did that change in the package here, maybe you could try it? http://download.opensuse.org/repositories/home:/wolfi323:/branches:/KDE:/Ext... -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c12
--- Comment #12 from Joachim Langenbach
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760
http://bugzilla.opensuse.org/show_bug.cgi?id=1042760#c13
--- Comment #13 from Wolfgang Bauer
I've tested your package today and transfers are working now. Many thanks!
Perfect. I'll submit this change too then. -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com