On 5/23/11 2:06 PM, Ed W wrote:

> Please check carefully the change to g_perllib ... I sense it's
> previously wrong in one of the versions?

The installer rewrites the incorrect line so it really doesn't make any
difference. But you are correct that the shorewall6 value was wrong.

> 
> Could you please look through the remaining diff: there is a bunch of
> calls with "runit" added in shorewall6 - I haven't looked through the
> chain of code, but perhaps these are easy for you to reconcile?
> 
> The end result is near identical left/right bar the change of names
> shorewall -> shorewall6 and iptables -> iptables6
> 
> How do you feel about referring to just "$IPTABLES" in both scripts? Any
> issue with abstracting something like $SHOREWALL=shorewall6 to unify all
> commandline calls?

IPTABLES and IP6TABLES come from /etc/shorewall/shorewall.conf and
/etc/shorewall6/shorewall6.conf respectively. So neither can be renamed.
What could be done is to define g_tool and set it appropriately.

I've applied your patch to my tree.

Thanks!
-Tom
-- 
Tom Eastep        \ When I die, I want to go like my Grandfather who
Shoreline,         \ died peacefully in his sleep. Not screaming like
Washington, USA     \ all of the passengers in his car
http://shorewall.net \________________________________________________

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
What Every C/C++ and Fortran developer Should Know!
Read this article and learn how Intel has extended the reach of its 
next-generation tools to help Windows* and Linux* C/C++ and Fortran 
developers boost performance applications - including clusters. 
http://p.sf.net/sfu/intel-dev2devmay
_______________________________________________
Shorewall-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-users

Reply via email to