Bug 479756 – sys-fs/lvm2-2.02.99 - add check for kernel config option CONFIG_UEVENT_HELPER_PATH

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Alexander Tsoy 2013-08-04 21:30:50 UTC]

Please add the check for kernel config option CONFIG_UEVENT_HELPER_PATH. It should be set to an empty value. This was originally requested in bug 451452, which was wrongly closed as dublicate.

https://bugs.gentoo.org/show_bug.cgi?id=451452#c9

[Comment 1 Alexander Tsoy 2013-08-04 21:33:57 UTC]

Created attachment 355124 [details, diff]
lvm2-2.02.99.ebuild.patch

I'm not sure, that this check is completely correct. May be a better solution exists.

[Comment 2 Samuli Suominen (RETIRED) 2013-08-05 18:20:42 UTC]

robbat2, what do you think?

[Comment 3 Alexander Tsoy 2013-08-05 18:58:46 UTC]

(In reply to Alexander Tsoy from comment #1)

> I'm not sure, that this check is completely correct.

Seems this check should be placed after check_extra_config(), because the latter also call linux-info_get_any_version() which is mandatory. Comment in linux-info.eclass says:

# Before using any of the config-handling functions in this eclass, you must
# ensure that one of the following functions has been called (in order of
# preference), otherwise you will get bugs like #364041):
# linux-info_pkg_setup
# linux-info_get_any_version
# get_version
# get_running_version

[Comment 4 Alexander Tsoy 2013-08-05 19:02:55 UTC]

Created attachment 355188 [details, diff]
lvm2-2.02.99-r1.ebuild.patch

[Comment 5 Robin Johnson 2013-08-05 20:21:00 UTC]

ssuominen: +1 to commit

[Comment 6 Samuli Suominen (RETIRED) 2013-08-07 11:32:05 UTC]

(In reply to Robin Johnson from comment #5)
> ssuominen: +1 to commit

+  07 Aug 2013; Samuli Suominen <[e-mail]> lvm2-2.02.99-r1.ebuild:
+  Include check for value of UEVENT_HELPER_PATH wrt #479756 by Alexander Tsoy.
+  Remove useless warning about USE="-static" and sys-kernel/genkernel which now
+  works with dynamic libraries since version 3.4.30.