On 9/17/2013 5:44 AM, Martin Gignac wrote:
>> Might I suggest changing order of the commands in patch - we should
>> always default to using ip command and only fallback to legacy arp etc..
> 
> Considering that iproute/iproute2 is a requirement for Shorewall, I
> would think there shouldn't even be a need to ever fall back to legacy
> 'arp' and 'netstat'.
> 
> Attached is a small patch to replace the calls to the legacy binaries
> in lib.cli.
> 

Thanks, Martin

As I mentioned in an earlier post, I prefer 'arp -na' when available,
but the second hunk of your patch is fine.

-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

------------------------------------------------------------------------------
LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint
2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes
Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13. 
http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to