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.
Home