Please push this patch asap. My other comments should be addressed asap. Better commit message: "dtm: fix bashism in monitor script [232]"
Thanks, Hans > -----Original Message----- > From: A V Mahesh [mailto:[email protected]] > Sent: den 21 november 2013 06:25 > To: Hans Feldt > Subject: Re: [devel] [PATCH 1 of 1] mds: correct osaf-transport-monitor > script to avoid looping when sh is not bash [#232] > > Hi Hans, > > Have you got a chance to look in this patch. > > -AVM > > On 11/15/2013 9:49 AM, [email protected] wrote: > > osaf/services/infrastructure/dtms/scripts/osaf-transport-monitor.in | 8 > > ++++---- > > 1 files changed, 4 insertions(+), 4 deletions(-) > > > > > > diff --git > > a/osaf/services/infrastructure/dtms/scripts/osaf-transport-monitor.in > > b/osaf/services/infrastructure/dtms/scripts/osaf- > transport-monitor.in > > --- a/osaf/services/infrastructure/dtms/scripts/osaf-transport-monitor.in > > +++ b/osaf/services/infrastructure/dtms/scripts/osaf-transport-monitor.in > > @@ -43,8 +43,8 @@ start() { > > fi > > > > if [ ! "$MDS_TRANSPORT" = "TIPC" ] ; then > > - COUNT=0 > > - while ((COUNT < 15)) > > + count=0 > > + while [ $count -le 15 ] > > do > > pidval=`cat $pkgpiddir/$prog.pid` > > if ! test -d /proc/$pidval ; then > > @@ -52,9 +52,9 @@ start() { > > $pkglibdir/opensaf_reboot 0 > > exit 0; > > fi > > - #logger "osafdtmd PID : $pidval monitor COUNT > > = $COUNT" > > + #logger "osafdtmd PID : $pidval monitor count > > = $count" > > sleep 1 > > - ((COUNT=COUNT+1)) > > + count=`expr $count + 1` > > done > > else > > sleep 15 > > > > ------------------------------------------------------------------------------ > > DreamFactory - Open Source REST & JSON Services for HTML5 & Native Apps > > OAuth, Users, Roles, SQL, NoSQL, BLOB Storage and External API Access > > Free app hosting. Or install the open source package on any LAMP server. > > Sign up and see examples for AngularJS, jQuery, Sencha Touch and Native! > > http://pubads.g.doubleclick.net/gampad/clk?id=63469471&iu=/4140/ostg.clktrk > > _______________________________________________ > > Opensaf-devel mailing list > > [email protected] > > https://lists.sourceforge.net/lists/listinfo/opensaf-devel ------------------------------------------------------------------------------ Shape the Mobile Experience: Free Subscription Software experts and developers: Be at the forefront of tech innovation. Intel(R) Software Adrenaline delivers strategic insight and game-changing conversations that shape the rapidly evolving mobile landscape. Sign up now. http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
