Re: [patch 1/1] linux module postinst generalisation

[Henning Schild <henning.schild@siemens.com>]

Hey Roberto,

that patch should be inline so it can be reviewed and later applied. You
did send it as attachment. So now people have a harder time reviewing
it and replying to certain lines of it, and anyone who wanted to apply
it would maybe not be able to use "git am" as they are used to.

Am Fri, 16 Sep 2022 14:44:47 +0200
schrieb "Roberto A. Foglietta" <roberto.foglietta@gmail.com>:

> Hi all,
> 
>  linux module postinst generalisation is needed because sometimes
> (corner cases) ls /lib/modules includes not valid items that make
> depmod fails in total. In this way, it fails specifically only and no
> damage is done.

I think that patch kind of makes sense. But maybe you can go into more
detail how such a "broken" entry can be there in the first place.
Normally one would expect an isar-built image to have exactly one
kernel and one "working" module directory for that.
Maybe you should rather find the package that drops invalid directories
and repair that. Or maybe it is not "invalid" after all and our "ls"
matching is too simple.
And an interator over multiple directories would maybe also fail once
it finds the one "broken" directory.

regards,
Henning

> 
> Cheers, R-
>