Bug 831729

Summary:	app-office/kmymoney-5.1.2 build fix for >=kde-apps/akonadi-21.12.0
Product:	Gentoo Linux	Reporter:	George Shapovalov <gerrshapovalov>
Component:	Current packages	Assignee:	Gentoo KDE team <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	sam
Priority:	Normal	Keywords:	PATCH
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	https://invent.kde.org/office/kmymoney/-/commit/a06d2fe154f755c3af1e7f261cd631929b9da998
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	patch to fix the akonadi new API issue build.log

Description George Shapovalov 2022-01-21 13:08:30 UTC

Build of app-office/kmymoney fails after updating to the latest plasma. The specific culprit is akonadi, they changed something at the API level. The fix is a simple patch. However, looks like I have to upload it after opening the bug first, as I don't see the way to attach a patch here..

Reproducible: Always

Steps to Reproduce:
emerge kmymoney after having upgraded plasma.

Comment 1 George Shapovalov 2022-01-21 13:10:11 UTC

Created attachment 762998 [details, diff]
patch to fix the akonadi new API issue

The ebuild simply needs to reference the patch. Just add the:

PATCHES=( "${FILESDIR}/${P}-mymoneycontact.cpp.patch" )

line to the ebuild..

Comment 2 Sam James archtester

2022-01-21 13:44:22 UTC

I was going to ask if the patch is upstream but I see it is. Thanks!

for now, you can put it in /etc/portage/patches

Comment 3 Andreas Sturmlechner gentoo-dev

2022-01-21 13:52:39 UTC

Akonadi is not Plasma.

Where is your build.log?

Comment 4 George Shapovalov 2022-01-21 19:45:12 UTC

(In reply to Andreas Sturmlechner from comment #3)
> Akonadi is not Plasma.
Well, the build started breaking after that last major kde update, so I called it all "plasma" :), but yeah, to be precise, its some change in akonadi innards that needs these changes.

> Where is your build.log?
Yeah, sorry, I meant to attach that too, but had to get my mail system up and running again first - this kde upgrade apparently broke more things than just kmymoney..

Here it is, attached now. The specific error line is this one:
/var/tmp/portage/app-office/kmymoney-5.1.2/work/kmymoney-5.1.2/kmymoney/mymoney/mymoneycontact.cpp:13:10: fatal error: AkonadiCore/RecursiveItemFetchJob: No such file or directory
   13 | #include <AkonadiCore/RecursiveItemFetchJob>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Some API change moving AkonadiCore/RecursiveItemFetchJob around or something like that. But, as I can see, you found the original kde bug already..

Anyway, the patch above fixes it.

Comment 5 George Shapovalov 2022-01-21 19:47:16 UTC

Created attachment 763082 [details]
build.log

Actual log, mentioned in above comment.

Comment 6 Larry the Git Cow gentoo-dev

2022-02-10 20:10:10 UTC

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=972b3ec622b6c26162d4e557d1ae5d1bb8d5010d

commit 972b3ec622b6c26162d4e557d1ae5d1bb8d5010d
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2022-02-10 18:43:34 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2022-02-10 20:09:51 +0000

    app-office/kmymoney: Fix build with >=kde-apps/akonadi-21.12.0
    
    EAPI-8 bump.
    
    Closes: https://bugs.gentoo.org/831729
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 .../files/kmymoney-5.1.2-akonadi-21.12.0.patch     | 38 ++++++++++++++++++++++
 app-office/kmymoney/kmymoney-5.1.2.ebuild          |  6 ++--
 2 files changed, 42 insertions(+), 2 deletions(-)