Bug 58465

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows 98; T312461)

Description of problem:
On neat, when I press the "Apply" button, or the "Yes" button on the
"Do you want to save your changes?" dialog box, it changes
/etc/modules.conf. In this process, it mangles comments and some
options in /etc/modules.conf. I attached below a diff of modules.conf
before and after neat changes it. Notice that:

1) I have some commented-out "alias 3c59x-something" entries from the
time that I was developing a patch for the 3Com 3c556 (now in the
3c59x driver). These entries were un-commented and changed into "alias
3c59x".

2) I like to have more information printed about the adapter, so I
have a "options 3c59x debug=2" line. This line disappeared.

3) I have two sets of entries for the sound configuration, for ALSA
and OSS; I leave the set for the drivers that I'm not using
commented-out. The sound-slot-0 line on the ALSA set was un-commented
(I'm not using ALSA, so its entries were commented-out; the line for
OSS was further below, so it is not in the diff). That's doubly weird:
I'm configuring the network, not sound...

[fred@polestar ~]# diff -u /etc/modules.conf.before /etc/modules.conf.after
--- /etc/modules.conf.before    Thu Jan 17 22:01:43 2002
+++ /etc/modules.conf.after     Thu Jan 17 22:13:41 2002
@@ -1,14 +1,13 @@
-#alias eth0 3c59x-556-Qk
+alias eth0 3c59x
 #options 3c59x-556-Qk debug=2
-#alias eth0 3c59x-55x-2217
+alias eth0 3c59x
 #options 3c59x-55x-2217 debug=2
-#alias eth0 andrew-3c59x
+alias eth0 3c59x
 #options andrew-3c59x debug=2
-#alias eth0 3c59x-2.2.18-pre16-1
+alias eth0 3c59x
 #options 3c59x-2.2.18-pre16-1 debug=2
 
 alias eth0 3c59x
-options 3c59x debug=2
 
 alias parport_lowlevel parport_pc
 options parport_pc io=0x378 irq=7 
@@ -21,7 +20,7 @@
 #alias snd-card-0 snd-card-intel8x0
 ## The line below makes rc.sysinit start alsa before
 ## /etc/rc.d/init.d/alsasound in Red Hat 7.0
-#alias sound-slot-0 i810_audio
+alias sound-slot-0 i810_audio
 ## OSS/Free portion - card #1
 #alias sound-service-0-0 snd-mixer-oss
 #alias sound-service-0-1 snd-seq-oss


Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. Add the following lines to /etc/modules.conf:

#alias eth0 3c59x-556-Qk
alias eth0 3c59x
options 3c59x debug=2
#alias sound-slot-0 i810_audio

2. Run neat, and press the "Apply" button.


Actual Results:  Entries are changed (see diff above)

Expected Results:  - Commented-out entries remain commented-out
- "options 3c59x debug=2" remains

Additional info: