Merge pull request softprops#44 from maxcountryman/feature/debuginfo

conditionally remove debuginfo from release builds

Author: softprops

README.md

@@ -24,6 +24,8 @@ isolate the lambda specific build artifacts from your host-local build artifacts
 
 > **⚠️ Note:** you can switch from the `release` profile to a custom profile like `dev` by providing a `PROFILE` environment variable set to the name of the desired profile. i.e. `-e PROFILE=dev` in your docker run
 
+> **⚠️ Note:** you can include debug symbols in optimized release build binaries by setting `DEBUGINFO`. By default, debug symbols will be stripped from the release binary and set aside in a separate .debug file.
+
 You will want to volume mount `/code` to the directory containing your cargo project.
 
 You can pass additional flags to `cargo`, the Rust build tool, by setting the `CARGO_FLAGS` docker env variable
@@ -77,4 +79,4 @@ $ unzip -o \
 # Ctrl-D to yield control back to your function
 ```
 
-Doug Tangren (softprops) 2018
+Doug Tangren (softprops) 2018

build.sh

@@ -5,6 +5,7 @@
 set -eo pipefail
 mkdir -p target/lambda
 export PROFILE=${PROFILE:-release}
+export DEBUGINFO=${DEBUGINFO}
 # cargo uses different names for target
 # of its build profiles
 if [[ "${PROFILE}" == "release" ]]; then
@@ -30,7 +31,7 @@ export CARGO_TARGET_DIR=$PWD/target/lambda
 
 function package() {
     file="$1"
-    if [[ "${PROFILE}" == "release" ]]; then
+    if [[ "${PROFILE}" == "release" ]] && [[ -z "${DEBUGINFO}" ]]; then
         objcopy --only-keep-debug "$file" "$file.debug"
         objcopy --strip-debug --strip-unneeded "$file"
         objcopy --add-gnu-debuglink="$file.debug" "$file"