[Buildroot] [PATCH] netsnmp: add option to enable debugging code

[Peter Korsgaard]

>>>>> "Arnout" == Arnout Vandecappelle <arnout at mind.be> writes:

 Arnout> On Thursday 15 March 2012 22:04:27 Luca Ceresoli wrote:
 >> +ifneq ($(BR2_PACKAGE_NETSNMP_ENABLE_DEBUGGING),y)
 >> +       NETSNMP_CONF_OPT += --disable-debugging
 >> +endif

 Arnout>  Peter, is it correct that you prefer to have both the positive
 Arnout> and the negative option in such a situation?  I.e.

 Arnout> ifeq ($(BR2_PACKAGE_NETSNMP_ENABLE_DEBUGGING),y)
 Arnout>        NETSNMP_CONF_OPT += --enable-debugging
 Arnout> else
 Arnout>        NETSNMP_CONF_OPT += --disable-debugging
 Arnout> endif

I don't feel strongly about it, but the 2nd approach means you don't
need to look at configure.ac to check if it defaults to --enable (and it
won't break if the default is ever changed).

-- 
Bye, Peter Korsgaard