Re: [Devil-linux-develop] Re: [Devil-linux-commit] build/config/etc/init.d network,1.31,1.32

[Bruce S. <bw...@ar...>]

> > > > I upgraded a firewall of mine and all the static route commands bombed.
> > > > After a little looking it appeared that the "if" statement below was
> > > > backwards, so I swapped the logic.  I didn't spend a lot of time looking
> > > > at the logic of the code, only the results of my routes (which now work)
> > > > I hope I didn't screw it up too bad, Friedrich!  :-)
> > > >       
> >   
> > > > > 				# add the route
> > > > > -				if [ -n "${netmask##*.*}" ]; then
> > > > > +				if [ -z "${netmask##*.*}" ]; then
> > > > > 					echo "   adding route to $CMD ${target}${netmask:+/$netmask}${gateway:+ via gateway $gateway} on $DEVICE"
> > > > > 					route add -$CMD ${target} ${netmask:+netmask $netmask} ${gateway:+gw $gateway} dev $DEVICE
> > > > > 				else
> > > > >         

Yeah, I committed the change last Friday.   I upgraded a couple servers
today to the latest, and it works for me now.

What revision number are you showing for the script?  Sometimes
sourceforge's anon CVS servers a little behind.  It should be 1.32.

Otherwise, it's a one character change you can do manually on line 266,
see above.

 - BS