Bugs item #808136, was opened at 2003-09-17 14:49 Message generated for change (Comment added) made by dannf You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=380273&aid=808136&group_id=24006
Category: Boot Loaders Group: v1.0 (example) Status: Open Resolution: None Priority: 5 Submitted By: Erik Jacobson (erikj1) Assigned to: Nobody/Anonymous (nobody) Summary: EXTRAS for elilo, device slashes Initial Comment: 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. ---------------------------------------------------------------------- Comment By: dann frazier (dannf) Date: 2003-09-22 23:40 Message: 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! ---------------------------------------------------------------------- Comment By: Erik Jacobson (erikj1) Date: 2003-09-18 07:31 Message: Logged In: YES user_id=730453 really attaching. Didn't check the box last time. ---------------------------------------------------------------------- Comment By: Erik Jacobson (erikj1) Date: 2003-09-18 07:30 Message: 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! ---------------------------------------------------------------------- Comment By: Erik Jacobson (erikj1) Date: 2003-09-18 07:02 Message: Logged In: YES user_id=730453 No problem. Will do. ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2003-09-18 00:23 Message: 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. ---------------------------------------------------------------------- Comment By: Erik Jacobson (erikj1) Date: 2003-09-17 14:54 Message: 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. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=380273&aid=808136&group_id=24006 ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Sisuite-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/sisuite-devel
