Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 580134 - media-sound/clementine : v1.3 stable bump
Summary: media-sound/clementine : v1.3 stable bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Fat-Zer
URL: https://github.com/clementine-player/...
Whiteboard:
Keywords:
Depends on:
Blocks: 588908
  Show dependency tree
 
Reported: 2016-04-15 20:15 UTC by Adrien D
Modified: 2016-08-08 20:45 UTC (History)
8 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adrien D 2016-04-15 20:15:33 UTC
Hi team Gentoo,

Is it possible to add in portage the last stable version of Clementine (1.3) which is available here : https://github.com/clementine-player/Clementine/releases

Thanks !

Reproducible: Always
Comment 1 Ulenrich 2016-04-17 08:16:39 UTC
clementine-1.3 landed

but following dependency does not exist yet in portage
dev-libs/crypto++-5.6.2-r4

I now just try to compile with existing
dev-libs/crypto++-5.6.2-r3
Comment 2 Ulenrich 2016-04-17 08:20:10 UTC
fails with dev-libs/crypto++-5.6.2-r3
-- Checking for module 'libspotify>=12.1.45'
--   
-- Checking for module 'taglib>=1.6'
--   Found taglib, version 1.10
-- Performing Test TAGLIB_HAS_OPUS
-- Performing Test TAGLIB_HAS_OPUS - Success
CMake Error at CMakeLists.txt:278 (message):
  Either crypto++ must be available or the non-GPL Spotify code must be compiled in
Comment 3 Ulenrich 2016-04-17 09:00:38 UTC
Tried with one of https://gpo.zugaina.org/dev-libs/libspotify
but same result using: dev-libs/libspotify-12.1.51
---
-- Checking for module 'libspotify>=12.1.45'
--
Clementine did not find it
Comment 4 Ulenrich 2016-04-17 09:54:24 UTC
depends on 
https://bugs.gentoo.org/show_bug.cgi?id=548542#c9
where 
@Bar-Lev revoked
dev-libs/crypto++-5.6.2-r4
Comment 5 Alon Bar-Lev (RETIRED) gentoo-dev 2016-04-17 14:56:07 UTC
When package assumes a dependency to behave in particular way, it does not mean that the dependency should just change.

libcrypto++ does not provide pkg-config file, this means that this (and other) dependencies should not make this assumption, nor "change" libcrypto++ just for the sake of an assumption.

In autoconf it was easy to specify LIBCRYPTOPP_CFLAGS="" and LIBCRYPTOPP_LIBS="-lcrypto++" to configure and resolve this mess.

I am not expert in cmake, but as long this upstream wrongly assumes pkg-config for libcrypto, there should be a way to override this as well, I've done some reading it should be by defining these variables (or other):

 XCRYPTOPP_FOUND=1
 XCRYPTOPP_LIBRARIES=crypto++
 XCRYPTOPP_LDFLAGS=-lcrypto++
 XCRYPTOPP_INCLUDE_DIR=
 XCRYPTOPP_CFLAGS=
 XCRYPTOPP=CRYPTOPP

Please help fix this ebuild if you know cmake better.
Please help to push upstream a variant that fallback to searching libcrypto++ as library if pkg-config fails, to meet upstream release management.

Thanks.
Comment 6 Patrice Clement gentoo-dev 2016-04-17 18:02:52 UTC
@Alon: please make sure to run repoman before widely removing a package's dep. Removing libcrypto++ broke the tree:
https://qa-reports.gentoo.org/output/gentoo-ci/a9608ae/output.html#media-sound/clementine

I have masked clementine-1.3 for now to address this warning:
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b6f6a35b0f0972e82c8addb7af748c075d801482

Please fix and remove the mask once everything's settled again. Thanks!
Comment 7 Fat-Zer 2016-04-19 22:26:04 UTC
There was a patch for media-sound/clementine-9999 in #548544 to force the libcrypto++ be found without altering the crypto++ ebuild:
https://bugs.gentoo.org/attachment.cgi?id=403742

Those lines was removed from 9999 a month ago and I'm not really understand why:
https://gitweb.gentoo.org/repo/gentoo.git/commit/media-sound/clementine/clementine-9999.ebuild?id=798265666093d42ba44871fbb932594cebd321d1
Comment 8 Ian Delaney (RETIRED) gentoo-dev 2016-04-26 01:07:54 UTC
Also Nikoli is long known as inactive. This package is destined for maintainer-needed unless another user offers to proxy maintain it.
Comment 9 Fat-Zer 2016-04-26 05:03:46 UTC
(In reply to Ian Delaney from comment #8)
> Also Nikoli is long known as inactive. This package is destined for
> maintainer-needed unless another user offers to proxy maintain it.

I may take up the maintainership...
Comment 10 Ian Delaney (RETIRED) gentoo-dev 2016-04-29 08:23:22 UTC
(In reply to Fat-Zer from comment #9)
> (In reply to Ian Delaney from comment #8)
> > Also Nikoli is long known as inactive. This package is destined for
> > maintainer-needed unless another user offers to proxy maintain it.
> 
> I may take up the maintainership...

go right ahead. You can review and re-prepare and ebuilds and patches presented here with 'your name' on them.
Comment 11 Fat-Zer 2016-08-08 20:45:59 UTC
Done some time ago.