Browse Source

Merge pull request #46333 from thaJeztah/remove_devicemapper_leftovers

hack, Dockerfile: remove devicemapper leftover,  build-tags, and libdevmapper-dev dependency
Sebastiaan van Stijn 1 year ago
parent
commit
634f6ec1d4
4 changed files with 1 additions and 18 deletions
  1. 0 2
      Dockerfile
  2. 0 1
      Dockerfile.simple
  3. 0 14
      hack/make.sh
  4. 1 1
      hack/test/unit

+ 0 - 2
Dockerfile

@@ -555,7 +555,6 @@ RUN --mount=type=cache,sharing=locked,id=moby-dev-aptlib,target=/var/lib/apt \
             pkg-config \
             pkg-config \
             dpkg-dev \
             dpkg-dev \
             libapparmor-dev \
             libapparmor-dev \
-            libdevmapper-dev \
             libseccomp-dev \
             libseccomp-dev \
             libsecret-1-dev \
             libsecret-1-dev \
             libsystemd-dev \
             libsystemd-dev \
@@ -583,7 +582,6 @@ RUN --mount=type=cache,sharing=locked,id=moby-build-aptlib,target=/var/lib/apt \
             gcc \
             gcc \
             libapparmor-dev \
             libapparmor-dev \
             libc6-dev \
             libc6-dev \
-            libdevmapper-dev \
             libseccomp-dev \
             libseccomp-dev \
             libsecret-1-dev \
             libsecret-1-dev \
             libsystemd-dev \
             libsystemd-dev \

+ 0 - 1
Dockerfile.simple

@@ -26,7 +26,6 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
 		cmake \
 		cmake \
 		git \
 		git \
 		libapparmor-dev \
 		libapparmor-dev \
-		libdevmapper-dev \
 		libseccomp-dev \
 		libseccomp-dev \
 		ca-certificates \
 		ca-certificates \
 		e2fsprogs \
 		e2fsprogs \

+ 0 - 14
hack/make.sh

@@ -84,24 +84,10 @@ if [ ! "$GOPATH" ]; then
 	exit 1
 	exit 1
 fi
 fi
 
 
-# Adds $1_$2 to DOCKER_BUILDTAGS unless it already
-# contains a word starting from $1_
-add_buildtag() {
-	[[ " $DOCKER_BUILDTAGS" == *" $1_"* ]] || DOCKER_BUILDTAGS+=" $1_$2"
-}
-
 if ${PKG_CONFIG} 'libsystemd' 2> /dev/null; then
 if ${PKG_CONFIG} 'libsystemd' 2> /dev/null; then
 	DOCKER_BUILDTAGS+=" journald"
 	DOCKER_BUILDTAGS+=" journald"
 fi
 fi
 
 
-# test whether "libdevmapper.h" is new enough to support deferred remove
-# functionality. We favour libdm_dlsym_deferred_remove over
-# libdm_no_deferred_remove in dynamic cases because the binary could be shipped
-# with a newer libdevmapper than the one it was built with.
-if command -v gcc &> /dev/null && ! (echo -e '#include <libdevmapper.h>\nint main() { dm_task_deferred_remove(NULL); }' | gcc -xc - -o /dev/null $(${PKG_CONFIG} --libs devmapper 2> /dev/null) &> /dev/null); then
-	add_buildtag libdm dlsym_deferred_remove
-fi
-
 # Use these flags when compiling the tests and final binary
 # Use these flags when compiling the tests and final binary
 
 
 if [ -z "$DOCKER_DEBUG" ]; then
 if [ -z "$DOCKER_DEBUG" ]; then

+ 1 - 1
hack/test/unit

@@ -12,7 +12,7 @@
 #
 #
 set -eux -o pipefail
 set -eux -o pipefail
 
 
-BUILDFLAGS=(-tags 'netgo libdm_no_deferred_remove journald')
+BUILDFLAGS=(-tags 'netgo journald')
 TESTFLAGS+=" -test.timeout=${TIMEOUT:-5m}"
 TESTFLAGS+=" -test.timeout=${TIMEOUT:-5m}"
 TESTDIRS="${TESTDIRS:-./...}"
 TESTDIRS="${TESTDIRS:-./...}"
 exclude_paths='/vendor/|/integration'
 exclude_paths='/vendor/|/integration'