[Xen-changelog] [xen-unstable] pygrub: fix the parameter `default' and `timeout' in elilo.conf

[Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1223023331 -3600
# Node ID 78e98657021afff71e34758d01e2ac2d550bae00
# Parent  6ce634cc6e75c5166b21b09e5b5ab5abb96dae52
pygrub: fix the parameter `default' and `timeout' in elilo.conf

This patch fixes two issues related to the parameter `default' and
`timeout' in elilo.conf:

- LiloConf.py cannot interpret the parameter `default' and
  `timeout'. The first kernel always boot up even if the second kernel
  is specified by `default'. And `timeout' is ignored.

  This issue is introduced by cset 15953:70bb28b62ffb.

- If two kernels or more are installed, the last kernel cannot boot
  up even if it is specified by `default'.

Signed-off-by: KUWAMURA Shin'ya <kuwa@xxxxxxxxxxxxxx>
---
 tools/pygrub/src/LiloConf.py |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff -r 6ce634cc6e75 -r 78e98657021a tools/pygrub/src/LiloConf.py
--- a/tools/pygrub/src/LiloConf.py      Fri Oct 03 09:37:35 2008 +0100
+++ b/tools/pygrub/src/LiloConf.py      Fri Oct 03 09:42:11 2008 +0100
@@ -142,7 +142,7 @@ class LiloConfigFile(object):
         self.images.append(image)
 
     def _get_default(self):
-        for i in range(0, len(self.images) - 1):
+        for i in range(len(self.images)):
             if self.images[i].title == self._default:
                 return i
         return 0
@@ -150,8 +150,8 @@ class LiloConfigFile(object):
         self._default = val
     default = property(_get_default, _set_default)
 
-    commands = { "default": "self.default",
-                 "timeout": "self.timeout",
+    commands = { "default": "default",
+                 "timeout": "timeout",
                  "prompt": None,
                  "relocatable": None,
                  }

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog