FS#41601 - [lvm2] missing dependency

Attached to Project: Arch Linux
Opened by Daniel Wendler (BMRMorph) - Monday, 18 August 2014, 08:34 GMT
Last edited by Eric Belanger (Snowman) - Wednesday, 20 August 2014, 01:33 GMT
Task Type Bug Report
Category Packages: Core
Status Closed
Assigned To Eric Belanger (Snowman)
Thomas Bächler (brain0)
Architecture All
Severity High
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

In last version the --with-cache=internal feature was enabled.
This feature works like expected but you have to install as runtime dependency the thin-provisioning-tools.
If you use the cache feature the system could not activate the lv on boot because it search for cache_check binary.
This could lead to an unbootable system.
This binary belongs to the "thin-provisioning-tools" package.
So i suggest to add thin-provisioning-tools to the dependency list.
I believe it is also needed for features which comes with the --with-thin=internal configure option.
(In gentoo ebuild both configure options depend on the thin use flag)

Additional info:
* package version(s)
2.02.109-1


Steps to reproduce:
Enable caching on an lv and reboot without installed thin-provisioning-tools
This task depends upon

Closed by  Eric Belanger (Snowman)
Wednesday, 20 August 2014, 01:33 GMT
Reason for closing:  Fixed
Additional comments about closing:  lvm2-2.02.109-2 in testing depends on thin-provisioning-tools
Comment by Doug Newgard (Scimmia) - Monday, 18 August 2014, 15:16 GMT
It's already an optdep:
thin-provisioning-tools (optional) - For thin provisioning support

Maybe the description just needs updated?
Comment by Eric Belanger (Snowman) - Tuesday, 19 August 2014, 01:13 GMT
Thomas: Should we just update the optdepends message or move thin-provisioning-tools to [core] and make it a depends? I can do the change. Just want your opinion.
Comment by Thomas Bächler (brain0) - Tuesday, 19 August 2014, 15:56 GMT
I never knew the --with-cache=internal feature. Since thin-provisioning-tools is tiny and has virtually no dependencies, we should just make it a dep.

Loading...