[Dnsmasq-discuss] Enable HAVE_IPSET by default

[Kevin Darbyshire-Bryant]

On 21/03/2013 10:08, Simon Kelley wrote:
> <snip>
>
> Finally, if it's going to be on by default, and given the limited size
> delta/lack of library definitions, there's an argument for not making
> it compile-time selectable at all. Every compile-time switch
> contributes to the combinatorial explosion of possible binaries, and
> lots of bugs come from unanticipated interactions in untested
> compile-flag combinations.
>
>
> Opinions, anyone?
>
>
>
> Simon.

Hi Simon,

I'll express an opinion, based purely on my *very* limited experience of
integrating 2.66test16 into a recent version of Tomato to fix some IPv6
problems.  I keen an eye on latest git pushes and integrate those into
my own personal version for testing.  I'm very much waiting for 2.66
release to come out so that I can push that to the proper Tomato
maintainers.

Size is a very important consideration for Tomato as some versions are
expected to run on a 2.4 kernel and squeeze into 3.8MB of flash rom
space - bytes matter.  Having said that, the Tomato developers don't
have to upgrade to the latest dnsmasq (have been running 2.61 for some
time) and the continuing support of what must be regarded as legacy
hardware has to come to an end sometime.

Based very much on your 'compile time switches lead to untested
combinations of binaries' argument, I'd say remove it as an option and
make it a standard feature.

Well it's an opinion, but what do I know :-)

Kevin