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

[Xen-devel] Re: [PATCH] xen bootloader

To: Jeremy Katz <katzj@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH] xen bootloader
From: Mike Wray <mike.wray@xxxxxx>
Date: Fri, 29 Apr 2005 08:59:34 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 29 Apr 2005 08:07:24 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1114744237.17750.39.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: <9cde8bff0504270221eded71f@xxxxxxxxxxxxxx> <1114744237.17750.39.camel@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla Thunderbird 1.0.2 (X11/20050317)
Jeremy Katz wrote:
On Wed, 2005-04-27 at 18:21 +0900, aq wrote:

Here is a patch to fix few problems in Xen bootloader you sent to the
list yesterday.

Thanks.  Sorry for the delay in responding, I've been underwater with
stuff for FC4 test3.

list of changes:
- make a dummy tools/pygrub/src/__init__.py

I still don't know why this didn't work, but thanks.

- extend filesystem abstraction by adding file_exist() method. this
method is used to check for existent of a file given its name. now
ext2fs implements this method.
Great, thanks. My only comment about the ext2fs code for this is that
you might as well inline the contents of ext2_file_exist in
ext2fs_file_exist... you don't gain anything by having that for reuse
that I can see.

- pygrub opens and parses /boot/grub/menu.lst or /boot/grub/grub.conf,
in that order.
- add /usr/lib/python to system path (see pygrub). without this
change, pygrub cannot find grub python package.

Per the other changes I submitted a week or so ago, you want to append
this to the path, not prepend it.

- remove few blank lines


Fair enough.

$ diffstat xen-grub.patch
__init__.py            |    1
fsys/__init__.py       |    5 +++-
fsys/ext2/ext2module.c |   51 ++++++++++++++++++++++++++++++++++++++++---------
pygrub                 |   12 +++++++++--
4 files changed, 57 insertions(+), 12 deletions(-)

Signed-off-by: Nguyen Anh Quynh <aquynh@xxxxxxxxx>


I'll apply this to my local tree.  Mike -- based on your later mail,
should I gather that you've applied things to your tree and are in the
process of pushing to -unstable?  Or should I go about trying to do
something more sane to maintain this on the side for a while?

Yes, I've applied the patch and it should be making its way out
to unstable, possibly today.

BTW, I had to install e2fsprogs-devel before it would compile, so
I added a note that you need that in a README.

I wonder why the patch you sent to list is in CR/LF format? arent you
programming on Windows, are you ;-). could you please remove them next
time ?


Hrmm, I have no clue either.  The previous patch appears to not be in
DOS format but the second one is.  And they should have been exported
identically from my local bk repository.  I'll double-check next time
before sending.

Mike

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