[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] Re: [PATCH] xen bootloader
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? > 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. Jeremy _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |