| Summary: | media-sound/clementine-9999: requires dev-libs/crypto++ | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Fat-Zer <fatzer2> |
| Component: | Current packages | Assignee: | Nikoli <nikoli> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | proxy-maint, sound |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| See Also: | https://bugs.gentoo.org/show_bug.cgi?id=654572 | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Bug Depends on: | 548542 | ||
| Bug Blocks: | |||
| Attachments: |
A propoused patch
alternative patch to fix the bug without bug #548542 |
||
|
Description
Fat-Zer
2015-05-03 23:31:11 UTC
Created attachment 402582 [details, diff]
A propoused patch
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 This bug in clementine requires the both fix in report https://bugs.gentoo.org/show_bug.cgi?id=548542 Compilation confirmed. 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. 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. 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. 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 works for me ;) but I suppose you are awaiting for a feedback from somebody else... You'll do fine |
