[Bug 219399] New: yast2-sound exports illegal keys for the XML file
https://bugzilla.novell.com/show_bug.cgi?id=219399 Summary: yast2-sound exports illegal keys for the XML file Product: openSUSE 10.2 Version: Beta 2 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: AutoYaST AssignedTo: lslezak@novell.com ReportedBy: ug@novell.com QAContact: qa@suse.de the yast2-sounds exports the following data: <volume_settings config:type="list"> <volume_entry> <Front config:type="integer">72</Front> <Headphone config:type="integer">72</Headphone> <Master config:type="integer">72</Master> <Master Mono config:type="integer">72</Master Mono> </volume_entry> </volume_settings> the <Master Mono config:type="integer">72</Master Mono> is not legal. There is a SPACE in the name of the key "Master Mono" which leads to illegal XML code like you see. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 lslezak@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mvidner@novell.com Status|NEW |NEEDINFO Info Provider| |ug@novell.com ------- Comment #1 from lslezak@novell.com 2006-11-09 23:09 MST ------- Hmm, "Master Mono" is a channel name. What is the solution? Convert spaces to during export and convert them back to spaces during import? I think it should be fixed in the XML agent, not in Yast modules. What do you think? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 ug@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Info Provider|ug@novell.com | ------- Comment #2 from ug@novell.com 2006-11-13 05:38 MST ------- I talked to our XML guru and he said, it's not allowed to name an element with a whitespace or in it in XML1.0 So, how about this? change the XML structure in the way: <volume_settings config:type="list"> <volume_entry> <key>Master Mono</key> <value config:type="integer">72</value> <volume_entry> <volume_entry> ... </volume_entry> </volume_settings> that would make the schema check a little bit better too, because we don't need the "Anything" in the RNC file then. The bad thing is, we are not compatible with old profiles and we need to do some code changes in yast2-sound. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 lslezak@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Info Provider| |ug@novell.com ------- Comment #3 from lslezak@novell.com 2006-12-05 06:35 MST ------- Fixed in yast2-sound-2.15.0, added backward compatible conversion function for importing the old format. Uwe, could you please send me a new RNC file which matches the new export format? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 ug@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Info Provider|ug@novell.com | ------- Comment #4 from ug@novell.com 2006-12-06 07:38 MST ------- the volume settings look good now but what about: <modules_conf config:type="list"> <module_conf> <alias>snd-card-0</alias> <model>AW35/Pro</model> <module>snd-cs4236</module> <options> <cport>1</cport> <dma1>1</dma1> <dma2>5</dma2> <enable>1</enable> <fm_port>222</fm_port> <index>0</index> <irq>12</irq> <isapnp>0</isapnp> <mpu_irq>7</mpu_irq> </options> are the options the same for each soundcard? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 lslezak@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Info Provider| |ug@novell.com ------- Comment #5 from lslezak@novell.com 2006-12-06 07:47 MST ------- No, the options are driver specific. There should be allowed any string. (Name of a module option doesn't contain spaces so there should be no problem.) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 ug@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED ------- Comment #6 from ug@novell.com 2006-12-06 08:27 MST ------- Created an attachment (id=108525) --> (https://bugzilla.novell.com/attachment.cgi?id=108525&action=view) sound.rnc -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 lslezak@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED ------- Comment #7 from lslezak@novell.com 2006-12-06 08:41 MST ------- Thanks! Submitted into SVN, will be included in yast2-sound-2.15.1 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 ug@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | ------- Comment #8 from ug@novell.com 2006-12-06 08:52 MST ------- please wait, the RNC file has an error. Sorry, I'll send you a new version -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 ug@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #108525|0 |1 is obsolete| | ------- Comment #9 from ug@novell.com 2006-12-06 09:00 MST ------- Created an attachment (id=108532) --> (https://bugzilla.novell.com/attachment.cgi?id=108532&action=view) fixed version -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=219399 lslezak@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |FIXED ------- Comment #10 from lslezak@novell.com 2006-12-06 23:29 MST ------- Applied, thanks! -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
participants (1)
-
bugzilla_noreply@novell.com