#30 EXTRAS for elilo, device slashes
Hi there.
The attached patch will address the following issues.
Note that one part
of the patch may have been applied to CVS already by
Dan.
The following patch to EFI.pm addresses these issues.
- Adding additional locations for finding elilo stuff
[already applied to CVS by Dan]
- Allowing one to use the 'EXTRAS' line for ELILO.
This is good for
Altix since SGI needs the "relocatable" flag added
to elilo.conf
- Changing the efi friendly support to reverse how it
handles slashes.
My version of ELILO (and I think all) want slashes
in paths for
kernels and initrd's to be unix-style back slashes,
not forward slashes.
So EFI.pm makes it so kernel images with slashes
never work instead of
always work. I didn't spot this before since RedHat
doesn't put slashes
in kernel names. SuSE does because it stores them
in the root of EFI and
not the directory with elilo.efi.
Discussion
Logged In: YES
user_id=730453
I forgot to add this.
This patch perhaps touches two lines that a previous patch
in CVS already
updated.
Also, one more thing is done by the patch I forgot to add
above:
- In the kernel section, Append was handled correctly. In
the Boot secction, append didn't use double quotes which
caused trouble when supplying certain kernel options.
Logged In: NO
would you mind submitting a patch against cvs head?
instructions for checking out cvs are under the cvs link on
this sourceforge site.
Logged In: YES
user_id=730453
No problem. Will do.
Logged In: YES
user_id=730453
OK. So here is an EFI.pm diffed against what I got from
CVS. I confirmed that
only what I have changed is in the diff.
I'm not sure what you do about the VERSION line though - so
check that out
before applying. Thanks!
Logged In: YES
user_id=730453
really attaching. Didn't check the box last time.
diff for EFI.pm as described above
Logged In: YES
user_id=146718
looks good - committed.
fyi, file version stuff is automatically done by cvs - you
don't need to change it.
thanks again!