Provide build flag to Disable compilation of libyang dependent interf…

…aces

vkarri@6c5cda5d64fa:/sonic/src/sonic-swss-common$ debuild -b -us -uc -Pnopython2,noyangmod
vkarri@6c5cda5d64fa:/sonic/src$ sudo dpkg -i libswsscommon_1.0.0_amd64.deb
vkarri@6c5cda5d64fa:/sonic/src$ sudo dpkg -i python3-swsscommon_1.0.0_amd64.deb
vkarri@6c5cda5d64fa:/sonic/src$ python3 -c "from swsscommon.swsscommon import DecoratorSubscriberStateTable, DecoratorTable, YangDefaultDecorator"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'DecoratorSubscriberStateTable' from 'swsscommon.swsscommon' (/usr/lib/python3/dist-packages/swsscommon/swsscommon.py)
vkarri@6c5cda5d64fa:/sonic/src$ python3 -c "from swsscommon.swsscommon import DecoratorTable, YangDefaultDecorator"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'DecoratorTable' from 'swsscommon.swsscommon' (/usr/lib/python3/dist-packages/swsscommon/swsscommon.py)
vkarri@6c5cda5d64fa:/sonic/src$ python3 -c "from swsscommon.swsscommon import YangDefaultDecorator"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'YangDefaultDecorator' from 'swsscommon.swsscommon' (/usr/lib/python3/dist-packages/swsscommon/swsscommon.py)
vkarri@6c5cda5d64fa:/sonic/src$ python3 -c "from swsscommon.swsscommon import SubscriberStateTable, ProducerStateTable, Table, DBConnector"

<Default Behavior>
vkarri@6c5cda5d64fa:/sonic$ make -f slave.mk target/debs/bookworm/python3-swsscommon_1.0.0_amd64.deb-install
vkarri@6c5cda5d64fa:/sonic$ python3 -c "from swsscommon.swsscommon import DecoratorSubscriberStateTable, DecoratorTable, YangDefaultDecorator"
vkarri@6c5cda5d64fa:/sonic$

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>

common/Makefile.am

@@ -26,10 +26,8 @@ common_libswsscommon_la_SOURCES = \
  common/configdb.cpp \
  common/dbconnector.cpp \
  common/dbinterface.cpp \
- common/defaultvalueprovider.cpp \
  common/sonicv2connector.cpp \
  common/table.cpp \
- common/decoratortable.cpp \
  common/json.cpp \
  common/producertable.cpp \
  common/producerstatetable.cpp \
@@ -60,24 +58,32 @@ common_libswsscommon_la_SOURCES = \
  common/exec.cpp \
  common/saiaclschema.cpp \
  common/subscriberstatetable.cpp \
- common/decoratorsubscriberstatetable.cpp \
  common/timestamp.cpp \
  common/warm_restart.cpp \
  common/luatable.cpp \
  common/countertable.cpp \
  common/redisutility.cpp \
  common/restart_waiter.cpp \
- common/profileprovider.cpp \
  common/zmqclient.cpp \
  common/zmqserver.cpp \
- common/asyncdbupdater.cpp \
+ common/asyncdbupdater.cpp  \
  common/redis_table_waiter.cpp
 
 common_libswsscommon_la_CXXFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(LIBNL_CFLAGS) $(CODE_COVERAGE_CXXFLAGS)
 common_libswsscommon_la_CPPFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(LIBNL_CPPFLAGS) $(CODE_COVERAGE_CPPFLAGS)
-common_libswsscommon_la_LIBADD = -lpthread $(LIBNL_LIBS) $(CODE_COVERAGE_LIBS) -lzmq -lboost_serialization -luuid -lyang
+common_libswsscommon_la_LIBADD = -lpthread $(LIBNL_LIBS) $(CODE_COVERAGE_LIBS) -lzmq -lboost_serialization -luuid
 common_libswsscommon_la_LDFLAGS = -Wl,-z,now $(LDFLAGS)
 
+if YANGMODS
+common_libswsscommon_la_SOURCES += \
+ common/defaultvalueprovider.cpp \
+ common/profileprovider.cpp \
+ common/decoratortable.cpp \
+ common/decoratorsubscriberstatetable.cpp
+
+common_libswsscommon_la_LIBADD += -lyang
+endif
+
 common_swssloglevel_SOURCES = \
  common/loglevel.cpp \
  common/loglevel_util.cpp

common/configdb.h

@@ -255,6 +255,12 @@ class ConfigDBConnector_Native : public SonicV2Connector_Native
  ret.setdefault(table_name, {})[self.deserialize_key(row)] = entry
  return ret
 
+%}
+#endif
+
+#if defined(SWIG) && defined(SWIGPYTHON) && defined(ENABLE_YANG_MODULES)
+%pythoncode %{
+
  class YangDefaultDecorator(object):
  def __init__(self, config_db_connector):
  self.connector = config_db_connector

configure.ac

@@ -37,8 +37,16 @@ AC_ARG_ENABLE(python2,
  no) python2=false ;;
  *) AC_MSG_ERROR(bad value ${enableval} for --enable-python2) ;;
 esac],[python2=true])
+AC_ARG_ENABLE(yangmods,
+[ --enable-yangmods Generate classes that depend on libyang],
+[case "${enableval}" in
+ yes) yangmods=true ;;
+ no) yangmods=false ;;
+ *) AC_MSG_ERROR(bad value ${enableval} for --enable-yangmods) ;;
+esac],[yangmods=true])
 AM_CONDITIONAL(DEBUG, test x$debug = xtrue)
 AM_CONDITIONAL(PYTHON2, test x$python2 = xtrue)
+AM_CONDITIONAL(YANGMODS, test x$yangmods = xtrue)
 if test x$CONFIGURED_ARCH = xarmhf && test x$CROSS_BUILD_ENVIRON = xy; then
  AM_CONDITIONAL(ARCH64, false)
 else

debian/rules

@@ -28,6 +28,11 @@ else
 CONFIGURE_ARGS += --disable-python2
 endif
 
+# Skip compiling classes that depend on libyang
+ifneq (,$(filter noyangmod,$(DEB_BUILD_PROFILES)))
+CONFIGURE_ARGS += --disable-yangmods
+endif
+
 # main packaging script
 %:
  dh $@

pyext/py2/Makefile.am

@@ -6,6 +6,10 @@ pyext_py2__swsscommon_la_CPPFLAGS = -std=c++11 -Icommon -I/usr/include/python$(P
 pyext_py2__swsscommon_la_LDFLAGS = -module -Wl,-z,now
 pyext_py2__swsscommon_la_LIBADD = common/libswsscommon.la -lpython$(PYTHON_VERSION)
 
+if YANGMODS
+SWIG_FLAG += -DENABLE_YANG_MODULES
+endif
+
 pyext/py2/swsscommon_wrap.cpp: $(SWIG_SOURCES)
  $(SWIG) $(SWIG_FLAG) -Icommon -o $@ $<
 

pyext/py3/Makefile.am

@@ -6,6 +6,10 @@ pyext_py3__swsscommon_la_CPPFLAGS = -std=c++11 -Icommon -I/usr/include/python$(P
 pyext_py3__swsscommon_la_LDFLAGS = -module -Wl,-z,now
 pyext_py3__swsscommon_la_LIBADD = common/libswsscommon.la $(PYTHON3_BLDLIBRARY)
 
+if YANGMODS
+SWIG_FLAG += -DENABLE_YANG_MODULES
+endif
+
 pyext/py3/swsscommon_wrap.cpp: $(SWIG_SOURCES)
  $(SWIG) $(SWIG_FLAG) -Icommon -o $@ $<
 

pyext/swsscommon.i

@@ -251,10 +251,12 @@ T castSelectableObj(swss::Selectable *temp)
 
 %include "schema.h"
 %include "dbconnector.h"
+#ifdef ENABLE_YANG_MODULES
 %include "defaultvalueprovider.h"
+%include "profileprovider.h"
+#endif
 %include "sonicv2connector.h"
 %include "pubsub.h"
-%include "profileprovider.h"
 %include "selectable.h"
 %include "select.h"
 %include "rediscommand.h"
@@ -278,7 +280,9 @@ T castSelectableObj(swss::Selectable *temp)
 %apply std::vector<std::pair<std::string, std::string>>& OUTPUT {std::vector<std::pair<std::string, std::string>> &ovalues};
 %apply std::string& OUTPUT {std::string &value};
 %include "table.h"
+#ifdef ENABLE_YANG_MODULES
 %include "decoratortable.h"
+#endif 
 %clear std::vector<std::string> &keys;
 %clear std::vector<std::string> &ops;
 %clear std::vector<std::vector<std::pair<std::string, std::string>>> &fvss;
@@ -318,7 +322,9 @@ T castSelectableObj(swss::Selectable *temp)
 %include "consumertable.h"
 %include "consumerstatetable.h"
 %include "subscriberstatetable.h"
+#ifdef ENABLE_YANG_MODULES
 %include "decoratorsubscriberstatetable.h"
+#endif
 
 %apply std::string& OUTPUT {std::string &op};
 %apply std::string& OUTPUT {std::string &data};

tests/Makefile.am

@@ -34,17 +34,21 @@ tests_tests_SOURCES = tests/redis_ut.cpp \
  tests/countertable_ut.cpp \
  tests/timer_ut.cpp \
  tests/cli_ut.cpp \
- tests/defaultvalueprovider_ut.cpp \
  tests/events_common_ut.cpp \
  tests/events_service_ut.cpp \
  tests/events_ut.cpp \
  tests/restart_waiter_ut.cpp \
  tests/redis_table_waiter_ut.cpp \
  tests/binary_serializer_ut.cpp \
  tests/zmq_state_ut.cpp \
- tests/profileprovider_ut.cpp \
  tests/main.cpp
 
 tests_tests_CFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(CFLAGS_GTEST) $(LIBNL_CFLAGS)
 tests_tests_CPPFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(CFLAGS_GTEST) $(LIBNL_CFLAGS) -fno-access-control
 tests_tests_LDADD = $(LDADD_GTEST) -lpthread common/libswsscommon.la $(LIBNL_LIBS) $(CODE_COVERAGE_LIBS) sonic-db-cli/libsonicdbcli.la -lzmq -luuid -lboost_serialization
+
+if YANGMODS
+tests_tests_SOURCES += \
+ tests/defaultvalueprovider_ut.cpp \
+ tests/profileprovider_ut.cpp
+endif