463218 – dev-libs/leveldb should optionally depend on app-arch/snappy

Bug 463218 - dev-libs/leveldb should optionally depend on app-arch/snappy

Summary: dev-libs/leveldb should optionally depend on app-arch/snappy

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] Library (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Patrick Lauer

URL:
Whiteboard:
Keywords:

Depends on:	463308
Blocks:
	Show dependency tree

Reported:	2013-03-25 12:25 UTC by Denis Kaganovich
Modified:	2013-03-26 09:12 UTC (History)
CC List:	0 users

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Denis Kaganovich 2013-03-25 12:25:43 UTC

dev-libs/leveldb requires [optional] app-arch/snappy dependence. It linking with snappy if snappy installed. Found looking into sys-cluster/ceph 0.59 - it use system leveldb and just check snappy for it, but if snappy is not installed before - leveldb will be installed too, but without snappy linkage.

According to maintainer's look, it may be useflag or hard dependence, but IMHO need to be fixed.

It easy to check by "emerge leveldb|grep -i snappy" with and without app-arch/snappy.

Reproducible: Always

Comment 1 Patrick Lauer gentoo-dev

2013-03-26 05:54:29 UTC

We'll have to wait on #463308 for this, but I think it's easiest as an unconditional dep

Thanks for finding this little bonus feature :)

Comment 2 Patrick Lauer gentoo-dev

2013-03-26 09:12:34 UTC

+  26 Mar 2013; Patrick Lauer <patrick@gentoo.org> +leveldb-1.9.0-r3.ebuild,
+  -leveldb-1.9.0-r1.ebuild, -leveldb-1.9.0-r2.ebuild:
+  Adding snappy dep #463218

Oh, that was fast :)