ci: debug symbols by default in bazel.release binary. (#949)

envoyproxy · May 12, 2017 · 640ea54 · 640ea54
1 parent fc747b3
commit 640ea54
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/ci/build_alpine_container/Makefile b/ci/build_alpine_container/Makefile
@@ -14,9 +14,8 @@ binary:
 	mkdir -p ./bin/configs
 	docker run -t -i --name build-envoy-alpine-ct -v ${ROOT_DIR}/../../:/source -v $(ENVOY_BUILD_DIR):/build lyft/envoy-build:latest /bin/bash -c \
 		"set -x && cd /source && \
-		cp /build/envoy/source/exe/envoy /tmp/envoy && \
-		cd /tmp/ && strip envoy && \
-		mv /tmp/envoy /source/ci/build_alpine_container/bin/stripped/ && \
+		cp /build/envoy/source/exe/envoy /source/ci/build_alpine_container/bin/stripped/ && \
+		strip /source/ci/build_alpine_container/bin/stripped/envoy && \
 		cp /build/envoy/source/exe/envoy /source/ci/build_alpine_container/bin/debug/ && \
 		cp /source/configs/google_com_proxy.json /source/ci/build_alpine_container/bin/configs/"
 	docker rm -f build-envoy-alpine-ct

diff --git a/ci/do_ci.sh b/ci/do_ci.sh
@@ -10,11 +10,11 @@ echo "building using ${NUM_CPUS} CPUs"
 function bazel_release_binary_build() {
   echo "Building..."
   cd "${ENVOY_CI_DIR}"
-  bazel --batch build ${BAZEL_BUILD_OPTIONS} -c opt //source/exe:envoy-static.stripped.stamped
+  bazel --batch build ${BAZEL_BUILD_OPTIONS} -c opt //source/exe:envoy-static.stamped
   # Copy the envoy-static binary somewhere that we can access outside of the
   # container.
   cp -f \
-    "${ENVOY_CI_DIR}"/bazel-genfiles/source/exe/envoy-static.stripped.stamped \
+    "${ENVOY_CI_DIR}"/bazel-genfiles/source/exe/envoy-static.stamped \
     "${ENVOY_DELIVERY_DIR}"/envoy
 }