commit:     2e1c05e68ecf30083b5fe1e2f83aa0c5548ce9a0
Author:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
AuthorDate: Fri May 17 21:45:44 2024 +0000
Commit:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
CommitDate: Fri May 17 23:02:54 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2e1c05e6

linux-mod-r1.eclass: update depmod.d comment

Likely a non-issue with dracut now (untested) but will leave it
alone for now and just updating comment. Maybe if users start
having more problems that'd require it or if we want to more
liberally rename modules install paths while ensuring it does not
use the old one on a same kernel (point of this was just to make
things more robust and is not hard-required).

Also adjust the commented out function to match dracut.conf.d (guess
glob makes it more straight forward, not that it's necessary).

Signed-off-by: Ionen Wolkens <ionen <AT> gentoo.org>

 eclass/linux-mod-r1.eclass | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/eclass/linux-mod-r1.eclass b/eclass/linux-mod-r1.eclass
index 6060074d8734..6d35a7068bfb 100644
--- a/eclass/linux-mod-r1.eclass
+++ b/eclass/linux-mod-r1.eclass
@@ -583,10 +583,11 @@ modules_post_process() {
        (( ${#mods[@]} )) ||
                die "${FUNCNAME[0]} was called with no installed modules under 
${path}"
 
-       # TODO?: find way for sane use with dracut (its 90kernel-modules-extra
-       # parses depmod.d files directly and assumes should include its modules
-       # which can lead to unnecessarily increased size or stale modules)
-#      _modules_process_depmod.d "${mods[@]#"${path}/"}"
+       # TODO?: look into re-introducing after verifying it works as expected,
+       # formerly omitted because dracut's 90kernel-modules-extra parses 
depmod.d
+       # files directly and assumes should include its modules but we now 
create
+       # dracut omit files that *hopefully* prevent this
+#      _modules_process_depmod.d "${mods[@]##*/}"
 
        _modules_process_dracut.conf.d "${mods[@]##*/}"
        _modules_process_strip "${mods[@]}"

Reply via email to