[Linux-ia64] efibootmgr-0.3.3-test5

[Linux-ia64] efibootmgr-0.3.3-test5

[Matt_Domsch]

> The 00 shouldn't be there on the end.  My math must be 
> off.  I'll check.

Helps if I don't account for the NULL on the end of a string in two separate
places...

efibootmgr-0.3.3-test5 is now available at http://domsch.com/linux/ia64 for
your enjoyment.  If there are no other issues, I'll release this as 0.3.3
later this week.

Thanks,
Matt

--
Matt Domsch
Sr. Software Engineer
Dell Linux Solutions
www.dell.com/linux
#2 Linux Server provider with 17% in the US and 14% Worldwide (IDC)!
#3 Unix provider with 18% in the US (Dataquest)!

Re: [Linux-ia64] efibootmgr-0.3.3-test5

[Stephane Eranian]

Matt,

We have noticed (during RH7.1) install that efibootmgr was not checking
for options with an identical names and it was creating menu options
with the same name for instance if you reinstall the system a second time.


Would it be possible to add a warning/error/check that would avoid such a
thing ?

Thanks.

-- 
-Stephane

RE: [Linux-ia64] efibootmgr-0.3.3-test5

[Matt_Domsch]

> We have noticed (during RH7.1) install that efibootmgr was 
> not checking
> for options with an identical names and it was creating menu options
> with the same name for instance if you reinstall the system a 
> second time.
> 
> Would it be possible to add a warning/error/check that would 
> avoid such a thing ?

Given that you can have load options with the same name (for whatever good
it does), how might people want to see this fixed?
1) Warn the user, but do it anyway.  Probably doesn't help the installer app
that calls efibootmgr.
2) Warn the user, do it anyway, append text to it like [1].  Not too pretty.
3) Warn the user, don't do it, need yet another flag to force it.
4) Make installer app more intelligent, something like:
	for b in `efibootmgr | grep "Red Hat Linux" | awk '{print $1}'`; do 
		efibootmgr -b $b -B
	done


I prefer 4 myself, but could see adding 1) if desired.

--
Matt Domsch
Sr. Software Engineer
Dell Linux Solutions
www.dell.com/linux
#2 Linux Server provider with 17% in the US and 14% Worldwide (IDC)!
#3 Unix provider with 18% in the US (Dataquest)!

Re: [Linux-ia64] efibootmgr-0.3.3-test5

[Stephane Eranian]

Matt,
> 
> Given that you can have load options with the same name (for whatever good
> it does), how might people want to see this fixed?
> 1) Warn the user, but do it anyway.  Probably doesn't help the installer app
> that calls efibootmgr.
> 2) Warn the user, do it anyway, append text to it like [1].  Not too pretty.
> 3) Warn the user, don't do it, need yet another flag to force it.
> 4) Make installer app more intelligent, something like:
> 	for b in `efibootmgr | grep "Red Hat Linux" | awk '{print $1}'`; do 
> 		efibootmgr -b $b -B
> 	done
> 
> 
> I prefer 4 myself, but could see adding 1) if desired.
> 
Clearly the installer should do 4) but a warning would be good too.

-- 

-Stephane