Fabian Vogt changed bug 1177577
What Removed Added
Status NEW RESOLVED
Resolution --- FIXED

Comment # 7 on bug 1177577 from
(In reply to Takashi Iwai from comment #6)
> (In reply to Fabian Vogt from comment #5)
> > (In reply to Takashi Iwai from comment #3)
> > > The current way to list up all modules explicitly, and I guess it's hard to
> > > process the list at the build time.  Maybe this needs two step processes:
> > > one to prepare the module list from modinfo of modules included in the
> > > target kernel, then repackage from the list.
> > 
> > I made a sr which does this:
> > 
> > %define crypto_modules \
> > %(rpm -ql %{kernel_package_name} | grep -E
> > 'kernel/crypto/|kernel/arch/.*/crypto/' | xargs basename -a | cut -d. -f1)
> > 
> > I don't really like that it's fragile (rpm ignores errors), but other than
> > that it's definitely better than mentioning everything explicitly IMO. What
> > do you think?
> 
> Looks like a good workaround!
> 
> BTW, did you check whether the regex really work as Michal mentioned in
> comment 4?

Yes, that's already used for some modules like ipt_* and nft_*, but it only
works on module names and not directories, so would still result in a long list
which needs manual updates.

I accepted and forwarded the sr now, so closing as fixed.


You are receiving this mail because: