[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#521595: [checks/files] should warn about non .conf files in etc/modprobe.d



On Fri, 2009-05-08 at 18:18 -0500, Raphael Geissert wrote:
> Adam D. Barratt wrote:
> > 
> > +Info: Files in <tt>/etc/modprobe.d</tt> that do not have a <tt>.conf</tt>
> > + suffix are no longer processed by certain (including future) versions of
> > + module-init-tools/modprobe.
> > 
> > Is that actually true right now?  My recent experience suggests not and
> > http://lists.debian.org/debian-devel/2009/03/msg00276.html implies it
> > will keep accepting them (albeit noisily) for quite a while yet.
> > 
> 
> After what we discussed on IRC I've changed the description.

Thanks for the patch.  Unfortunately, it had a somewhat fatal bug, in
that it actually matched /all/ files in /etc/modprobe.d.  The problem is
that "/etc/modprobe.d/foo.conf" matches "^etc/modprobe\.d/.+(?!\.conf)$"
as the .+ consumes the entire of "foo.conf", which is then not followed
by ".conf".

I've applied a version with a fixed filename check and a rejigged -
hopefully that translates well enough :-) - description.

Regards,

Adam



Reply to: