Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 548544 - media-sound/clementine-9999: requires dev-libs/crypto++
Summary: media-sound/clementine-9999: requires dev-libs/crypto++
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Nikoli
URL:
Whiteboard:
Keywords:
Depends on: 548542
Blocks:
  Show dependency tree
 
Reported: 2015-05-03 23:31 UTC by Fat-Zer
Modified: 2018-05-02 07:51 UTC (History)
2 users (show)

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


Attachments
A propoused patch (clementine-9999.ebuild.patch,331 bytes, patch)
2015-05-03 23:37 UTC, Fat-Zer
Details | Diff
alternative patch to fix the bug without bug #548542 (clementine-9999.ebuild.patch,712 bytes, patch)
2015-05-22 08:00 UTC, Fat-Zer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fat-Zer 2015-05-03 23:31:11 UTC
Also it requires it to provide a pkg-config metadate which is not so in gentoo yet.

Reproducible: Always
Comment 1 Fat-Zer 2015-05-03 23:37:24 UTC
Created attachment 402582 [details, diff]
A propoused patch
Comment 2 Leonardo Amaral 2015-05-17 22:08:58 UTC
I confirm this issue.

CMake Error at CMakeLists.txt:277 (message):
  Either crypto++ must be available or the non-GPL Spotify code must be
  compiled in
Comment 3 Leonardo Amaral 2015-05-21 10:05:19 UTC
This bug in clementine requires the both fix in report https://bugs.gentoo.org/show_bug.cgi?id=548542

Compilation confirmed.
Comment 4 Ian Delaney (RETIRED) gentoo-dev 2015-05-22 02:23:03 UTC
1. Can maintainer possibly acquire patches and confirm all? Once again we have fixes ready to go and it doesn't involve the actual maintainer. 
2. This bug has been made to depend on 548542 however it could be done independently. It will still be flawed until that bug is resolved.
Comment 5 Fat-Zer 2015-05-22 08:00:33 UTC
Created attachment 403742 [details, diff]
alternative patch to fix the bug without bug #548542

> 2. This bug has been made to depend on 548542 however it could be done
> independently. It will still be flawed until that bug is resolved.
Yes it is, but IMO it's a bit hacky way...
Here is a simple patch to fix it without bug #548542.
Comment 6 Ian Delaney (RETIRED) gentoo-dev 2015-05-25 08:56:39 UTC
Frankly this appears adequate. Although a hack it would work. I shall simply await input form another dev here, however if it goes unreplied over ~ 2 weeks I am tempted to simply patch and close. Better to have a working ebuild than awaiting a seemingly unresponsive upstream.
Comment 7 Ian Delaney (RETIRED) gentoo-dev 2015-05-29 05:58:40 UTC
  29 May 2015; Ian Delaney <idella4@gentoo.org> clementine-1.2.3.ebuild,
  clementine-9999.ebuild:
  fix to absent dep crypto++, patch thanks to Fat-Zer from bug #548544
Comment 8 Fat-Zer 2015-06-02 10:44:19 UTC
works for me ;) 
but I suppose you are awaiting for a feedback from somebody else...
Comment 9 Ian Delaney (RETIRED) gentoo-dev 2015-06-02 15:19:13 UTC
You'll do fine