Commit 05554f17 authored by Gustavo Zacarias's avatar Gustavo Zacarias Committed by Peter Korsgaard
Browse files

quagga: fix build breakage without snmp 

Quagga no longer supports explicitly disabling snmp, and what's worse,
it interprets --disable-snmp as --enable-snmp.
Remove the disable configure option.
While at it add a comment about net-snmp requiring AgentX support for
quagga to build properly as suggested by Arnout.
Fixes:
http://autobuild.buildroot.net/results/a9bce6dd8065ddb1850a4e14f0e4eb257ee89286/



Signed-off-by: default avatarGustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: default avatarPeter Korsgaard <jacmet@sunsite.dk>
parent 978e7c74

package/quagga/Config.in

+3 −0
Original line number Diff line number Diff line
@@ -84,4 +84,7 @@ config BR2_PACKAGE_QUAGGA_SNMP 
	help

	  Make quagga information available via SNMP SMUX.



comment "AgentX support must be enabled in net-snmp or the build will break"

	depends on BR2_PACKAGE_QUAGGA_SNMP



endif

package/quagga/quagga.mk

+1 −3
Original line number Diff line number Diff line
@@ -26,10 +26,8 @@ QUAGGA_CONF_OPT += $(if $(BR2_PACKAGE_QUAGGA_OPAQUE_LSA),--enable-opaque-lsa,--d 


ifeq ($(BR2_PACKAGE_QUAGGA_SNMP),y)

QUAGGA_CONF_ENV += ac_cv_path_NETSNMP_CONFIG=$(STAGING_DIR)/usr/bin/net-snmp-config

QUAGGA_CONF_OPT += --enable-snmp

QUAGGA_CONF_OPT += --enable-snmp=agentx

QUAGGA_DEPENDENCIES += netsnmp

else

QUAGGA_CONF_OPT +=--disable-snmp

endif



$(eval $(autotools-package))