On 4/29/05, Mike Wray <mike.wray@xxxxxx> wrote:
> 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.
Mike, will you push those code to -unstable soon? I am looking forward
to it. ( I am having few improvements, but not sure what to patch
against)
thank you,
aq
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home