chore: optimize build process (#6)

Author: johnnzhou

Makefile

@@ -1,3 +1,5 @@
+DOCKER_IMAGE=swift:6.1.2
+
 .PHONY: clean
 clean:
 	rm -rf .build .swiftpm Package.resolved || true
@@ -8,4 +10,11 @@ build-test:
 
 .PHONY: build-release
 build-release:
-	swift build --static-swift-stdlib -c release
+	swift build --static-swift-stdlib -c release
+
+dev:
+	docker run --rm -it \
+		-v $(shell pwd):/app \
+		-w /app \
+		$(DOCKER_IMAGE) \
+		/bin/bash

docker/build_debian.dockerfile

@@ -2,6 +2,12 @@
 
 set -e
 
+if [ -z "$1" ]; then
+    echo "Error: No argument provided."
+    echo "Usage: $0 {ubuntu|debian|macos}"
+    exit 1
+fi
+
 if [[ ! -z "$(git status --porcelain=v1 2>/dev/null)" ]]; then
     echo "There are uncommitted changes in the local tree, please commit or discard them"
     exit 1
@@ -38,18 +44,34 @@ binary_name_prefix="lcl-cli-$release_tag-$arch"
 function build_for_linux() {
     local platform=$1
     local image_name="$image_name_prefix-$platform"
+    case "$platform" in
+        ubuntu)
+            tag="noble"
+            ;;
+        debian)
+            tag="bookworm"
+            ;;
+        *)
+            echo "Error: Unsupported platform: $platform"
+            exit 1
+            ;;
+    esac
+    local docker_image="swift:6.1.2-$tag"
 
     echo "arch: $arch"
     echo "platform: $platform"
     echo "image_name: $image_name"
     echo "binary_name: $binary_name_prefix-$platform"
+    echo "directory: $(pwd)"
 
-    docker build -t "$image_name" -f "docker/build_$platform.dockerfile" .
-    local container_id=$(docker create "$image_name")
-    docker cp "$container_id:/lcl" "release/$binary_name_prefix-$platform"
-    docker rm -v "$container_id"
-    docker image rm -f "$image_name"
+    docker run --rm \
+        -v "$(pwd)":/app \
+        -v "$(pwd)/release":/app/release \
+        -w /app \
+        "$docker_image" \
+        bash -c "swift build --static-swift-stdlib -c release && strip .build/release/lcl && mv .build/release/lcl release/$binary_name_prefix-$platform"
 
+    docker rmi -f "$docker_image" || true
     echo "Binary for $platform has been successfully built!"
 }
 
@@ -59,12 +81,27 @@ function build_for_macos() {
     echo "binary_name: $binary_name_prefix-macos"
 
     make build-release
-    mv .build/release/lcl release/lcl
+    mv .build/release/lcl release/$binary_name_prefix-macos
     strip release/lcl
     ./release/lcl --help
-    mv release/lcl "release/$binary_name_prefix-macos"
 }
 
-build_for_linux ubuntu
-build_for_linux debian
-build_for_macos
+# Use a case statement to call the correct function
+case "$1" in
+    ubuntu)
+        build_for_linux ubuntu
+        ;;
+    debian)
+        build_for_linux debian
+        ;;
+    macos)
+        build_for_macos
+        ;;
+    *)
+        echo "Error: Invalid argument '$1'."
+        echo "Usage: $0 {ubuntu|debian|macos}"
+        exit 1
+        ;;
+esac
+
+exit 0