On Thu, Apr 26, 2007 at 10:00:10AM +0200, Andrew Beekhof wrote:
> On 4/26/07, Simon Horman <[EMAIL PROTECTED]> wrote:
> >----- Forwarded message from Simon Horman <[EMAIL PROTECTED]> -----
> >
> >Date: Mon, 23 Apr 2007 11:25:36 +0900
> >From: Simon Horman <[EMAIL PROTECTED]>
> >To: Erich Schubert <[EMAIL PROTECTED]>, [EMAIL PROTECTED]
> >Subject: Re: Bug#420206: heartbeat-2: Bashism in IPAddr2
> >Message-ID: <[EMAIL PROTECTED]>
> >References: <[EMAIL PROTECTED]>
> >MIME-Version: 1.0
> >Content-Type: text/plain; charset=us-ascii
> >Content-Disposition: inline
> >In-Reply-To: <[EMAIL PROTECTED]>
> >User-Agent: mutt-ng/devel-r804 (Debian)
> >Status: RO
> >Content-Length: 567
> >Lines: 20
> >
> >On Fri, Apr 20, 2007 at 08:38:59PM +0200, Erich Schubert wrote:
> >> Package: heartbeat-2
> >> Version: 2.0.7-2
> >> Severity: normal
> >>
> >> The IPAddr2 script contains bashisms.
> >> /usr/lib/ocf/resource.d/heartbeat/IPaddr2:
> >>
> >> 
> >IF_MAC=${IF_MAC:0:2}:${IF_MAC:2:2}:${IF_MAC:4:2}:${IF_MAC:6:2}:${IF_MAC:8:2}:${IF_MAC:10:2}
> >>
> >> Hotfix: replace /bin/sh in the first line by /bin/bash
> >> Other scripts might be affected as well.
> >
> >Thanks, I'll get this fixed. Please let me know if you find any more.
> 
> i'll push up a fix momentarily

Since IPaddr2 is Linux specific, I guess it's OK to have it run by
bash.

> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

-- 
Dejan
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to