[Pkg-systemd-maintainers] Bug#627949: Bug#627949: /etc/modules parsing still broken

[brian m. carlson]

On Mon, Mar 10, 2014 at 05:41:51PM +0100, Michael Stapelberg wrote:
> Hi brian,
> 
> "brian m. carlson" <sandals at crustytoothpaste.net> writes:
> > This bug is still reproducible almost three years later.  My mute key
> > doesn't work properly because systemd doesn't parse the
> > volume_capabilities=1 part of the following line:
> >
> >   thinkpad_acpi volume_capabilities=1
> >
> > This file has worked the same way in Debian for the last decade, and is
> > even documented to do so:
> >
> >   # Parameters can be specified after the module name.
> >
> > Since systemd is going to be the default init system, it isn't
> > acceptable to break almost every installed system where this worked fine
> > under sysvinit.  It would be nice if this bug could get fixed sooner
> > rather than later so jessie isn't broken.
> Could you please help us by sending a patch for module-init-tools that
> does what Michael Biebl outlined in #27?

No, because I believe the right solution is to make systemd respect the
way the file works already, and not to break the semantics of the
current file in favor of another solution.  Making systemd split on
whitespace isn't very hard, and the fact is there's no good reason to
change the existing behavior.

In general, if there are things that switching to systemd breaks that
have worked for a long time (i.e. regressions), we need to fix systemd,
and not implement hacks that work around systemd's shortcomings.

-- 
brian m. carlson / brian with sandals: Houston, Texas, US
+1 832 623 2791 | http://www.crustytoothpaste.net/~bmc | My opinion only
OpenPGP: RSA v4 4096b: 88AC E9B2 9196 305B A994 7552 F1BA 225C 0223 B187
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://alioth-lists.debian.net/pipermail/pkg-systemd-maintainers/attachments/20140310/073a646c/attachment-0002.sig>