https://bugzilla.novell.com/show_bug.cgi?id=418648
User sschober@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=418648#c307967
Summary: yast2-x11: exported xml config fails to validate against
schema
Product: openSUSE 11.1
Version: Factory
Platform: Other
OS/Version: Other
Status: NEW
Severity: Normal
Priority: P5 - None
Component: AutoYaST
AssignedTo: jdsn@novell.com
ReportedBy: sschober@novell.com
QAContact: qa@suse.de
Found By: ---
The yast2-x11 module produces the following code on (autoyast) export:
<profile xmlns="http://www.suse.com/1.0/yast2ns"
xmlns:config="http://www.suse.com/1.0/configns">
<x11>
<!-- ... -->
24
<!-- ... -->
</x11>
</profile>
This fails to validate against the x11.rnc schema included in the module, which
states 'color_depth' should be of type integer:
color_depth =
element color_depth { INTEGER }
The easy solution, i think, would be to change this to '{ text }', but
'integer' appears to be the more fitting type. A quick look at the code in
<yast-svn>/x11/src/autoyast/modules/AutoinstX11.ycp revealed the following
statement:
144 x11["color_depth"] = (string) XLib::getActiveColorDepth();
Which might, according to my very basic understanding of the code, be changed
to:
144 x11["color_depth"] = tointeger(XLib::getActiveColorDepth());
This might also remove the need for the cast in line 187:
186 // convert value to integer (bnc#307967)
187 integer colors = tointeger(x11["color_depth"]:16);
--
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.