WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] xen-booloader: pygrub improvement & bug fix

To: Jeremy Katz <katzj@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen-booloader: pygrub improvement & bug fix
From: aq <aquynh@xxxxxxxxx>
Date: Sat, 30 Apr 2005 00:52:00 +0900
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 29 Apr 2005 15:51:40 +0000
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=WRevZf2B8HcmjvaSHGxQPNJEEEwIs7eEgCIrwmaiLBthgu67w6v52aaobuz6Tu3eSstXcU3M9nYExesFpVYzAPqx9aNEoUVEC8PZ+iGxsNhussH3MzJD0AhPAp9Dil/vCF9qTTvE5sjXFNteZGyTwZS1KagOka/Lszac+5/dbO0=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1114787869.3574.9.camel@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <9cde8bff050428205660478f38@xxxxxxxxxxxxxx> <1114787869.3574.9.camel@xxxxxxxxxxxxxx>
Reply-to: aq <aquynh@xxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On 4/30/05, Jeremy Katz <katzj@xxxxxxxxxx> wrote:
> On Fri, 2005-04-29 at 12:56 +0900, aq wrote:
> > here is a patch to improve and fix few bugs in pygrub of xen-booloader.
> 
> Cool, thanks.  Comments below
> 
> > List of changes:
> > - temporarily remove usage of use_default_colors(), since python 2.3
> > doesnt support this method.
> 
> It would be better to actually check for the method and call it if
> available.. something like
>   if hasattr(curses, 'use_default_colors') curses.use_default_colors()

fine. but anyway i am implementing color feature, so perhaps we dont
need those default color.

> 
> > - allow user to press 'q' to quit pygrub
> 
> I had this at one point and then removed it -- the question is if you
> quit, then what are you wanting to boot?  I guess it would be aborting
> the domain boot (which is the result you get), but thinking further down
> the road, what does that mean on a reboot?  So I just decided the best
> thing to do was not to allow this.

for me, it is nice to allow pepole to cancel the tool. without this
option, how can you allow people to quit once they dont want to run
anymore? so i think it is fine to keep it that way.
 
> 
> > - deinitialize curses before quitting.
> 
> curses should get de-initialized when you leave curses.wrapper -- are
> you not seeing this?

right, thanks for pointing out this.

by the way, pygrub is an ugly name. the fact that this tool is written
in python is what users care least. how about renamming it to a more
make-sense name? like xengrub, or (even better) xenloader?

regards,
aq

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

<Prev in Thread] Current Thread [Next in Thread>