Bug 427380 - Review Request: efibootmgr - EFI Boot Manager
Summary: Review Request: efibootmgr - EFI Boot Manager
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chris Lumens
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-03 17:04 UTC by Matt Domsch
Modified: 2008-02-16 03:02 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-16 03:02:14 UTC
Type: ---
Embargoed:
clumens: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Matt Domsch 2008-01-03 17:04:28 UTC
Spec URL: http://domsch.com/linux/fedora/efibootmgr/efibootmgr.spec
SRPM URL: http://domsch.com/linux/fedora/efibootmgr/efibootmgr-0.5.4-1.fc9.src.rpm
Description: 
efibootmgr displays and allows the user to edit the Intel Extensible
Firmware Interface (EFI) Boot Manager variables.  Additional
information about EFI can be found at
http://developer.intel.com/technology/efi/efi.htm and http://uefi.org/.


efibootmgr has previously been included in the elilo SRPM, but is being split out into its own package because elilo itself will be going away once grub has that functionality.  efibootmgr is ExclusiveArch i386, x86_64, and ia64 because the EFI environment only exists on those architectures.

Comment 1 Chris Lumens 2008-01-09 14:47:25 UTC
rpmlint runs cleanly.

> URL: http://linux.dell.com/%{name}/

I don't see efibootmgr-0.5.4 (the version out of %{version}) available so I
can't check that the sources in the package match the upstream source.

> Conflicts: elilo < 3.6-5
> Obsoletes: elilo < 3.6-5

I believe the Obsoletes should be enough to make sure that elilo gets replaced.

> touch -r src/man/man8/%{name}.8 src/man/man8/%{name}.8.gz

Not sure what the point of that in the %install section is, unless you need to
fake out install somehow.

Comment 2 Matt Domsch 2008-01-09 16:25:01 UTC
Chris, thanks for the quick review.

I'll post 0.5.4 as soon as this passes review, I just wanted to wait to roll 
any critical found changes from this review into the release.

I was told on IRC that the Conflicts is necessary too.  notting, advice?

the touch statement is just to get the timestamp to match the file from the 
tarball, so that it doesn't get set to a newer timestamp unnecessarily just 
because it got compressed.

Comment 3 Bill Nottingham 2008-01-09 16:34:35 UTC
Thinking... the conflicts might help ensure the right thing happens if someone
just does 'yum install efibootmgr' on a system that already has elilo. But I
don't think it's absolutely necessary.

Comment 4 Matt Domsch 2008-01-09 17:26:23 UTC
Thanks Bill.  Seeing as how elilo will hopefully be disapperaing in favor of 
grub w/ EFI capability, the comps file will list efibootmgr but not elilo.  So 
upgrades from a distro version which used the combo elilo package, to a distro 
version that doesn't, needs that line in the efibootmgr package (as efibootmgr 
will be listed in comps).

Chris, with this addressed, any remaining concerns?

Thanks,
Matt

Comment 5 Chris Lumens 2008-01-09 17:47:45 UTC
Nope, looks fine to me.  Thanks.

Comment 6 Matt Domsch 2008-01-09 19:39:14 UTC
New Package CVS Request
=======================
Package Name: efibootmgr
Short Description: EFI Boot Manager
Owners: mdomsch,notting,clumens
Branches: 
InitialCC:  
Cvsextras Commits: yes

Comment 7 Kevin Fenzi 2008-01-09 21:42:27 UTC
cvs done.

Comment 8 Matt Domsch 2008-02-01 01:10:03 UTC
build will be coordinated with build of elilo package removing efibootmgr.


Note You need to log in before you can comment on or make changes to this bug.