don't rename bin if its already named bootstrap. fixes #7

Author: softprops

build.sh

@@ -20,15 +20,18 @@ function package() {
     rm "$file.zip" > 2&>/dev/null || true
     # note: would use printf "@ $(basename $file)\n@=bootstrap" | zipnote -w "$file.zip"
     # if not for https://bugs.launchpad.net/ubuntu/+source/zip/+bug/519611
-    mv "$file" bootstrap
+    if [ "$file" != ./bootstrap ] && [ "$file" != bootstrap ]; then
+        mv "${file}" bootstrap
+    fi
     zip "$file.zip" bootstrap
     rm bootstrap
 }
 
 cd "$CARGO_TARGET_DIR"/release
 (
+    export -f package
     if [ -z "$BIN" ]; then
-        find -maxdepth 1 -executable -type f -exec package {} \;
+        find -maxdepth 1 -executable -type f -exec bash -c 'package "$0"' {} \;
     else
         package "$BIN"
     fi

tests/test-func/Cargo.toml

@@ -4,6 +4,12 @@ version = "0.1.0"
 authors = ["softprops <d.tangren@gmail.com>"]
 edition = "2018"
 
+# users don't have to do this
+# but when they do, the builder packaging still work
+[[bin]]
+name = "bootstrap"
+path = "src/main.rs"
+
 [dependencies]
 lambda_runtime = "0.1"
 serde_json = "1.0"

tests/test.sh

@@ -46,17 +46,19 @@ DIST=$(cd $HERE/..; echo $PWD)
 
 cd ${HERE}/test-func
 
+# test packaing with a single binary
 function package_bin() {
     rm target/lambda/release/ > /dev/null 2>&1
     docker run --rm \
-    -e BIN=test-func \
+    -e BIN="$1" \
     -v ${PWD}:/code \
     -v ${HOME}/.cargo/registry:/root/.cargo/registry \
     -v ${HOME}/.cargo/git:/root/.cargo/git \
     softprops/lambda-rust && \
     ls target/lambda/release/test-func.zip > /dev/null 2>&1
 }
 
+# test packaging all binaries
 function package_all() {
     rm target/lambda/release/ > /dev/null 2>&1
     docker run --rm \
@@ -68,7 +70,7 @@ function package_all() {
 }
 
 # package tests
-assert_success "it packages single bin" package_bin
+assert_success "it packages single bin" package_bin bootstrap
 
 assert_success "it packages all bins" package_all