Author: jdsn Date: Mon Sep 1 20:21:51 2008 New Revision: 50538 URL: http://svn.opensuse.org/viewcvs/yast?rev=50538&view=rev Log: fixed type conversion in autoYaST (bnc#418648) Modified: trunk/x11/VERSION trunk/x11/package/yast2-x11.changes trunk/x11/src/autoyast/modules/AutoinstX11.ycp Modified: trunk/x11/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/VERSION?rev=50538&r1=50537&r2=50538&view=diff ============================================================================== --- trunk/x11/VERSION (original) +++ trunk/x11/VERSION Mon Sep 1 20:21:51 2008 @@ -1 +1 @@ -2.17.1 +2.17.2 Modified: trunk/x11/package/yast2-x11.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/package/yast2-x11.changes?rev=50538&r1=50537&r2=50538&view=diff ============================================================================== --- trunk/x11/package/yast2-x11.changes (original) +++ trunk/x11/package/yast2-x11.changes Mon Sep 1 20:21:51 2008 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Mon Sep 1 20:20:12 CEST 2008 - jdsn@suse.de + +- fixed type conversion in autoYaST setup (bnc#418648) +- 2.17.2 + +------------------------------------------------------------------- Fri Aug 29 23:02:24 CEST 2008 - jdsn@suse.de - tablet configuration support during installation (fate#302888) Modified: trunk/x11/src/autoyast/modules/AutoinstX11.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/x11/src/autoyast/modules/AutoinstX11.ycp?rev=50538&r1=50537&r2=50538&view=diff ============================================================================== --- trunk/x11/src/autoyast/modules/AutoinstX11.ycp (original) +++ trunk/x11/src/autoyast/modules/AutoinstX11.ycp Mon Sep 1 20:21:51 2008 @@ -141,7 +141,9 @@ } current_vendor = (string) XLib::getMonitorVendor(); current_model = (string) XLib::getMonitorModel(); - x11["color_depth"] = (string) XLib::getActiveColorDepth(); + // store this value with type integer in the map so it gets exported + // correctly to the autoyast profile (bnc#418648). + x11["color_depth"] = tointeger(XLib::getActiveColorDepth()); x11["resolution"] = (string) XLib::getActiveResolutionString(); display["min_hsync"] = tointeger(XLib::getHsyncMin()); display["max_hsync"] = tointeger(XLib::getHsyncMax()); @@ -184,7 +186,8 @@ XLib::setResolution ( resolution ); // convert value to integer (bnc#307967) - integer colors = tointeger(x11["color_depth"]:16); + // made obsolete by fix for (bnc#418648), cast the value to be sure it is integer + integer colors = (integer) x11["color_depth"]:16; y2milestone("X11-Auto: setup color depth <%1>",colors); XLib::setDefaultColorDepth ( tostring(colors) ); -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org