Re: [PULL] fixes (inc. softdep) & try to to remove unused modules first

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


Jon Masters wrote:
Hi Alan,

Thanks for your patches. As I mentioned below, I'm pulling everything
but the rename removal (for reasons explained). There is a pre2 version
of 3.12 in git and I made tarballs last week, but I am waiting until
pre3 this week before I make an announcement. Then the final 3.12 should
just be exactly the same if nothing horrible comes up in testing.

Great.

On Wed, 2010-03-17 at 10:27 +0000, Alan Jenkins wrote:

Alan Jenkins (8):
(fixes)
      doc: fix typo
      doc: fix sgml errors
      modprobe: fix dump_config
      modprobe: fix softdep flags

Thanks, these are great. I guess a markup validator would be handy. Are
you building the SGML with different flags, tools, or did I miss it?

I've been using the debian package "docbook-to-man" (via the Makefile, no custom flags). It's always been a bit noisy, and the errors didn't stop it from generating the manpages.

On the softdep front, I think it's also worth fixing the ignore flags to
actually offer a choice of ignoring install *or* remove but not both.
The current implementation doesn't do what might be apparent.

Not sure what you have in mind here. Do we want something that's not covered by

   modprobe -i ...

and

   modprobe -i -r ...

? Or should we check for and reject "--remove --ignore-install"? Or is the code just ugly in some respect?

Thanks
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Home]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Hacking TiVo]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Big List of Linux Books]     [16.7MP]

Add to Google Powered by Linux