#30 EXTRAS for elilo, device slashes

v1.0_(example)
open
nobody
5
2003-09-17
2003-09-17
No

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

  • Erik Jacobson

    Erik Jacobson - 2003-09-17

    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.

     
  • Nobody/Anonymous

    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.

     
  • Erik Jacobson

    Erik Jacobson - 2003-09-18

    Logged In: YES
    user_id=730453

    No problem. Will do.

     
  • Erik Jacobson

    Erik Jacobson - 2003-09-18

    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!

     
  • Erik Jacobson

    Erik Jacobson - 2003-09-18

    Logged In: YES
    user_id=730453

    really attaching. Didn't check the box last time.

     
  • Erik Jacobson

    Erik Jacobson - 2003-09-18

    diff for EFI.pm as described above

     
  • dann frazier

    dann frazier - 2003-09-23

    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!