Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 710832 - kde-misc/basket[git] fails to compile with >=dev-libs/libgit2-0.99
Summary: kde-misc/basket[git] fails to compile with >=dev-libs/libgit2-0.99
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL: https://github.com/basket-notepads/ba...
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2020-02-25 18:48 UTC by Dan Goodliffe
Modified: 2020-03-11 18:08 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (info.txt,6.94 KB, text/plain)
2020-02-25 18:49 UTC, Dan Goodliffe
Details
build.log (build.log.xz,21.47 KB, application/x-xz)
2020-02-25 18:50 UTC, Dan Goodliffe
Details
Proposed fix (libgit2-0.99.patch,675 bytes, patch)
2020-02-25 19:13 UTC, Dan Goodliffe
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Goodliffe 2020-02-25 18:48:33 UTC
As per summary:

/var/tmp/portage/kde-misc/basket-2.49b/work/basket-2.49b/src/application.cpp: In constructor ‘Application::Application(int&, char**)’:
/var/tmp/portage/kde-misc/basket-2.49b/work/basket-2.49b/src/application.cpp:62:13: error: ‘git_threads_init’ was not declared in this scope; did you mean ‘git_rebase_init’?
   62 |             git_threads_init();
      |             ^~~~~~~~~~~~~~~~
      |             git_rebase_init
/var/tmp/portage/kde-misc/basket-2.49b/work/basket-2.49b/src/application.cpp: In destructor ‘virtual Application::~Application()’:
/var/tmp/portage/kde-misc/basket-2.49b/work/basket-2.49b/src/application.cpp:73:13: error: ‘git_threads_shutdown’ was not declared in this scope
   73 |             git_threads_shutdown();
      |             ^~~~~~~~~~~~~~~~~~~~
Comment 1 Dan Goodliffe 2020-02-25 18:49:15 UTC
Created attachment 615930 [details]
emerge --info
Comment 2 Dan Goodliffe 2020-02-25 18:50:22 UTC
Created attachment 615934 [details]
build.log
Comment 3 Dan Goodliffe 2020-02-25 18:59:06 UTC
I've opened an issue upstream https://github.com/basket-notepads/basket/issues/45 as this looks like it's just under-maintained. It also looks like a trivial fix, which I might take a swing at if I get chance later tonight.
Comment 4 Dan Goodliffe 2020-02-25 19:13:32 UTC
Created attachment 615936 [details, diff]
Proposed fix

Problem is actually around the libgit2 version detection and a presumed change in LIBGIT2_SOVERSION to be a char *, not a number.
Patch changes the check to be against LIBGIT2_VER_MINOR (99 >= 22)
Comment 5 Andreas Sturmlechner gentoo-dev 2020-02-26 10:04:17 UTC
Could you please submit your fix as a PR upstream?
Comment 6 Dan Goodliffe 2020-02-27 18:09:05 UTC
Done! https://github.com/basket-notepads/basket/pull/46
Comment 7 Larry the Git Cow gentoo-dev 2020-03-01 22:02:27 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=34f8f2b233bcf437aa08a2d48838645b50e77d16

commit 34f8f2b233bcf437aa08a2d48838645b50e77d16
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2020-03-01 22:01:05 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2020-03-01 22:02:11 +0000

    kde-misc/basket: Fix build against >=dev-libs/libgit2-0.99
    
    Thanks-to: Dan Goodliffe <gentoo@randomdan.homeip.net>
    Closes: https://bugs.gentoo.org/710832
    Package-Manager: Portage-2.3.90, Repoman-2.3.20
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 kde-misc/basket/basket-2.49b.ebuild                |  5 +++-
 .../files/basket-2.49b-libgit2-0.99-compat.patch   | 32 ++++++++++++++++++++++
 2 files changed, 36 insertions(+), 1 deletion(-)
Comment 8 Andreas Sturmlechner gentoo-dev 2020-03-11 18:08:28 UTC
Thanks again for your contribution.

This is just to notify you that basket repository was recently moved (back) to kde.org and libgit2-0.99 support was fixed over there, in a slightly different way:

https://cgit.kde.org/basket.git/commit/?id=090ac46912025f94aca710355e51a8154df36a56