Revert "Make changes to building and packaging sairedis (#1116)" (#1141)

This may be causing or exposing some issues during tests.

This reverts commit 7b39fc4.

Author: saiarcot895

.azure-pipelines/build-template.yml

@@ -150,7 +150,7 @@ jobs:
       set -ex
       rm ../*.deb || true
       ./autogen.sh
-      DEB_BUILD_OPTIONS=nocheck DEB_CONFIGURE_EXTRA_FLAGS='--enable-code-coverage' dpkg-buildpackage -us -uc -b -Psyncd,vs -j$(nproc)
+      DEB_BUILD_OPTIONS=nocheck fakeroot debian/rules DEB_CONFIGURE_EXTRA_FLAGS='--enable-code-coverage' CFLAGS="" CXXFLAGS="" binary-syncd-vs
       mv ../*.deb .
     displayName: "Compile sonic sairedis with coverage enabled"
   - script: |
@@ -209,6 +209,6 @@ jobs:
         contents: 'syslog-all.tgz'
         targetFolder: $(Build.ArtifactStagingDirectory)
   - publish: $(Build.ArtifactStagingDirectory)/
-    artifact: ${{ parameters.syslog_artifact_name }}@$(System.JobAttempt)
+    artifact: ${{ parameters.syslog_artifact_name }}
     displayName: "Publish syslog artifacts"
     condition: always()

configure.ac

@@ -10,8 +10,8 @@ AC_PROG_CC
 AC_PROG_CXX
 AC_PROG_LIBTOOL
 AC_HEADER_STDC
+AM_PATH_PYTHON
 AM_PATH_PYTHON3
-AX_PYTHON3_DEVEL
 AX_CODE_COVERAGE
 AX_ADD_AM_MACRO_STATIC([])
 
@@ -101,14 +101,6 @@ AC_ARG_ENABLE(asan,
 	*) AC_MSG_ERROR(bad value ${enableval} for --enable-asan) ;;
 esac],[asan_enabled=false])
 
-AC_ARG_ENABLE(python2,
-[  --enable-python2     Generate Python 2 bindings],
-[case "${enableval}" in
-	yes) python2=true ;;
-	no)  python2=false ;;
-	*) AC_MSG_ERROR(bad value ${enableval} for --enable-python2) ;;
-esac],[python2=true])
-
 if test "x$asan_enabled" = "xtrue"; then
     CFLAGS_ASAN+=" -fsanitize=address"
     CFLAGS_ASAN+=" -DASAN_ENABLED"
@@ -119,26 +111,13 @@ if test "x$asan_enabled" = "xtrue"; then
     AC_SUBST(LDFLAGS_ASAN)
 fi
 
-if test "x$python2" = "xtrue"; then
-	m4_define_default([_AM_PYTHON_INTERPRETER_LIST],
-					  [python2 python2.7 python2.6 python2.5 python2.4 python2.3 python2.2 python2.1 python2.0])
-	AM_PATH_PYTHON
-	AX_PYTHON_DEVEL([< '3'])
-	PYTHON2_CPPFLAGS=$PYTHON_CPPFLAGS
-	PYTHON2_LIBS=$PYTHON_LIBS
-	AC_SUBST(PYTHON2_CPPFLAGS)
-	AC_SUBST(PYTHON2_LIBS)
-fi
-
 AM_CONDITIONAL(ASAN_ENABLED, test x$asan_enabled = xtrue)
-AM_CONDITIONAL(PYTHON2, test x$python2 = xtrue)
 
 AC_PATH_PROGS(SWIG, [swig3.0 swig])
 
 CXXFLAGS_COMMON=""
 CXXFLAGS_COMMON+=" -ansi"
 CXXFLAGS_COMMON+=" -fPIC"
-CXXFLAGS_COMMON+=" -pipe"
 CXXFLAGS_COMMON+=" -std=c++14"
 CXXFLAGS_COMMON+=" -Wall"
 CXXFLAGS_COMMON+=" -Wcast-align"

debian/compat

@@ -1 +1 @@
-12
+10

debian/control

@@ -2,33 +2,24 @@ Source: sonic
 Maintainer: Kamil Cudnik <kcudnik@microsoft.com>
 Section: net
 Priority: optional
-Build-Depends: debhelper (>= 12), autotools-dev, libzmq5-dev
+Build-Depends: debhelper (>=9), autotools-dev, libzmq5-dev
 Standards-Version: 1.0.0
 
 Package: syncd
 Architecture: any
-Build-Profiles: <syncd !rpc !vs>
-Depends: ${misc:Pre-Depends}
-Recommends: ${shlibs:Depends}
 Conflicts: syncd-rpc, syncd-vs
 Description: This package contains sync daemon for SONiC project.
   This sync daemon syncs the ASIC_DB in Redis database and the real ASIC via SAI.
 
 Package: syncd-rpc
 Architecture: any
-Build-Profiles: <syncd rpc !vs>
-Depends: ${misc:Pre-Depends}
-Recommends: ${shlibs:Depends}
 Conflicts: syncd, syncd-vs
 Description: This package contains sync daemon with rpc for SONiC project.
   This sync daemon syncs the ASIC_DB in Redis database and the real ASIC via SAI.
   This daemon contains saithrift rpc library for remote control of an ASIC.
 
 Package: syncd-vs
 Architecture: any
-Build-Profiles: <syncd !rpc vs>
-Depends: ${misc:Pre-Depends}
-Recommends: ${shlibs:Depends}
 Conflicts: syncd-rpc, syncd
 Description: This package contains sync daemon for SONiC project linked with virtual switch.
   This sync daemon syncs the ASIC_DB in Redis database and the real ASIC via SAI.
@@ -47,7 +38,6 @@ Description: This package contains development files for SAI-Redis.
 
 Package: libsaivs
 Architecture: any
-Depends: ${shlibs:Depends}, ${misc:Pre-Depends}
 Section: libs
 Description: This package contains SAI-VirtualSwitch implementation for SONiC project.
 
@@ -59,7 +49,6 @@ Description: This package contains development files for SAI-VirtualSwitch.
 
 Package: libsaimetadata
 Architecture: any
-Depends: ${shlibs:Depends}, ${misc:Pre-Depends}
 Section: libs
 Description: This package contains SAI-Metadata implementation for SONiC project.
 
@@ -69,6 +58,63 @@ Depends: libsaimetadata (= ${binary:Version})
 Section: libdevel
 Description: This package contains development files for SAI-Metadata.
 
+Package: syncd-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Conflicts: syncd-rpc-dbg, syncd-vs
+Depends:
+    syncd (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for syncd
+
+Package: syncd-rpc-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Conflicts: syncd-dbg, syncd-vs
+Depends:
+    syncd-rpc (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for syncd-rpc
+
+Package: syncd-vs-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Conflicts: syncd-dbg, syncd-rpc-dbg
+Depends:
+    syncd-vs (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for syncd-vs
+
+Package: libsairedis-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Depends:
+    libsairedis (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for libsairedis
+
+Package: libsaivs-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Depends:
+    libsaivs (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for libsaivs
+
+Package: libsaimetadata-dbg
+Architecture: any
+Section: debug
+Priority: extra
+Depends:
+    libsaimetadata (= ${binary:Version}),
+    ${misc:Depends}
+Description: debugging symbols for libsaimetadata
+
 Package: python-pysairedis
 Architecture: any
 Build-Profiles: <!nopython2>

debian/libsaimetadata-dev.install

@@ -1,5 +1,3 @@
 meta/sai*.h usr/include/sai
 meta/Sai*.h usr/include/sai
 SAI/meta/sai*.h usr/include/sai
-usr/lib/*/libsaimetadata.so
-usr/lib/*/libsaimeta.so

debian/libsairedis-dev.install

@@ -1,2 +1 @@
 lib/sairedis.h usr/include/sai
-usr/lib/*/libsairedis.so

debian/rules

@@ -3,6 +3,14 @@
 # output every command that modifies files on the build system.
 #export DH_VERBOSE = 1
 
+.ONESHELL:
+SHELL = /bin/bash
+.SHELLFLAGS += -ex
+
+# see EXAMPLES in dpkg-buildflags(1) and read /usr/share/dpkg/*
+DPKG_EXPORT_BUILDFLAGS = 1
+include /usr/share/dpkg/default.mk
+
 ifneq (${SWSS_COMMON_INC},)
 	SWSS_COMMON_CONFIG = "--with-swss-common-inc=${SWSS_COMMON_INC}"
 endif
@@ -13,7 +21,7 @@ ifneq (${SWSS_COMMON_LIB},)
 endif
 
 # see FEATURE AREAS in dpkg-buildflags(1)
-export DEB_BUILD_MAINT_OPTIONS = hardening=+all
+#export DEB_BUILD_MAINT_OPTIONS = hardening=+all
 
 # see ENVIRONMENT in dpkg-buildflags(1)
 # package maintainers to append CFLAGS
@@ -23,53 +31,78 @@ export DEB_BUILD_MAINT_OPTIONS = hardening=+all
 
 DOPACKAGES = $(shell dh_listpackages)
 
-configure_opts = --disable-static
-ifeq ($(ENABLE_ASAN), y)
-	configure_opts += --enable-asan
-endif
-
 # For Debian jessie, stretch, and buster, and Ubuntu bionic and focal, build
 # Python 2 bindings. This is controlled by the build profile being used.
-ifeq (,$(filter nopython2,$(DEB_BUILD_PROFILES)))
+ifneq (,$(filter python-pysairedis,$(DOPACKAGES)))
 configure_opts += --enable-python2
 else
 configure_opts += --disable-python2
 endif
 
-ifneq ($(filter syncd,$(DEB_BUILD_PROFILES)),)
-ifneq ($(filter rpc,$(DEB_BUILD_PROFILES)),)
-configure_opts += --enable-rpcserver
-endif
-ifneq ($(filter vs,$(DEB_BUILD_PROFILES)),)
-configure_opts += --with-sai=vs
-endif
-else
-configure_opts += --disable-syncd
-endif
+clean:
+	dh $@ --with autotools-dev
+
+build:
+	echo build stage is skipped. Please use binary to generate debian packages
 
-%:
-	dh $@
+binary: binary-syncd-rpc binary-syncd
+
+binary-sairedis:
+	echo '--enable-syncd=no' > /tmp/syncd-build
+	ENABLESYNCD=no dh clean  --with autotools-dev
+	dh build  -N syncd -N syncd-dbg -N syncd-rpc -N syncd-rpc-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+	dh binary -N syncd -N syncd-dbg -N syncd-rpc -N syncd-rpc-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+
+binary-syncd:
+	echo > /tmp/syncd-build
+	dh clean  --with autotools-dev
+	dh build  -N syncd-rpc -N syncd-rpc-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+	dh binary -N syncd-rpc -N syncd-rpc-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+
+binary-syncd-rpc: | binary-syncd
+	echo '--enable-rpcserver=yes' > /tmp/syncd-build
+	dh clean  --with autotools-dev
+	dh build  -N syncd -N syncd-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+	dh binary -N syncd -N syncd-dbg -N syncd-vs -N syncd-vs-dbg --with autotools-dev
+
+binary-syncd-vs:
+	echo '--with-sai=vs' > /tmp/syncd-build
+	dh clean  --with autotools-dev
+	dh build  -N syncd -N syncd-dbg -N syncd-rpc -N syncd-rpc-dbg --with autotools-dev
+	dh binary -N syncd -N syncd-dbg -N syncd-rpc -N syncd-rpc-dbg --with autotools-dev
 
 # dh_make generated override targets
 # This is example for Cmake (See https://bugs.debian.org/641051 )
 #override_dh_auto_configure:
 #	dh_auto_configure -- \
 #	-DCMAKE_LIBRARY_PATH=$(DEB_HOST_MULTIARCH)
 
+configure_opts =
+ifeq ($(ENABLE_ASAN), y)
+	configure_opts += --enable-asan
+endif
+
 override_dh_auto_configure:
-	dh_auto_configure -- ${SWSS_COMMON_CONFIG} $(configure_opts)
+	./autogen.sh
+	dh_auto_configure -- $(DEB_CONFIGURE_EXTRA_FLAGS) $(shell cat /tmp/syncd-build) ${SWSS_COMMON_CONFIG} $(configure_opts)
 
 override_dh_install:
 	dh_install
 	# Note: escape $ with an extra $ symbol
-ifneq ($(filter rpc,$(DEB_BUILD_PROFILES)),)
-	if [ -f debian/syncd-rpc/usr/bin/syncd_init_common.sh ] ; then
+	if egrep -q '(^| )--enable-rpcserver=yes( |$$)' /tmp/syncd-build && [ -f debian/syncd-rpc/usr/bin/syncd_init_common.sh ] ; then
 		sed -i 's|ENABLE_SAITHRIFT=0|ENABLE_SAITHRIFT=1 # Add a comment to fix https://github.com/Azure/sonic-buildimage/issues/2694 |' debian/syncd-rpc/usr/bin/syncd_init_common.sh
 	fi
-endif
 
 override_dh_installinit:
 	dh_installinit --init-script=syncd
 
 override_dh_shlibdeps:
-	$(LD_LIBRARY_PATH_CONFIG) dh_shlibdeps -- --ignore-missing-info -xlibsai
+	$(LD_LIBRARY_PATH_CONFIG) dh_shlibdeps --dpkg-shlibdeps-params=--ignore-missing-info
+
+override_dh_strip:
+	dh_strip -psyncd-rpc --dbg-package=syncd-rpc-dbg
+	dh_strip -psyncd --dbg-package=syncd-dbg
+	dh_strip -psyncd-vs --dbg-package=syncd-vs-dbg
+	dh_strip -plibsairedis --dbg-package=libsairedis-dbg
+	dh_strip -plibsaivs --dbg-package=libsaivs-dbg
+	dh_strip -plibsaimetadata --dbg-package=libsaimetadata-dbg