[Buildroot] [PATCH] netsnmp: add option to enable debugging code
Peter Korsgaard
jacmet at uclibc.org
Sun Mar 18 20:40:34 UTC 2012
>>>>> "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
More information about the buildroot
mailing list