firmware update problem

Matt Domsch Matt_Domsch at dell.com
Sat Apr 12 23:56:00 CDT 2008


On Sat, Apr 12, 2008 at 04:37:36PM +0200, Eric Doutreleau wrote:
> Keith Hopkins a ?crit :
> > On 04/11/2008 09:38 PM, Adam Williams wrote:
> >   
> >> Brian A. Seklecki wrote:
> >
> > Actually, the bug is in function distro_version()
> > where it tries to
> >          elif (echo "${DISTRO_REL_RPM}" | grep enterprise-release > /dev/null 2>&1) ; then
> >              # this is for Oracle Enterprise Linux 5+
> >              fi
> > Which is a "no no" only because the if-then-fi expects you to actually do something between then-fi, and a comment does not qualify.  So, it's a bug in the script (or a design problem in bash, take your pick ;) )
> > They probably meant to have something like "REDHAT_RELEASE=1" in between then-fi.
> >
> > Adding "| bash -xv" will show this clearly.
> >   
> Indeed the script is a bit buggy
> thank for your help

Looks like Michael Brown fixed this on Thursday, but it took a bit to
get synced out.  Give it a try now.

Thanks,
Matt

-- 
Matt Domsch
Linux Technology Strategist, Dell Office of the CTO
linux.dell.com & www.dell.com/linux



More information about the Linux-PowerEdge mailing list