Change release profile in cargo

Use strip from cargo instead of calling the strip command Enable LTO, CGU to 1 and change panic to abort
Radiicall · Apr 14, 2024 · 807668c · 807668c
1 parent 33f38b5
commit 807668c
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 12 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -37,8 +37,8 @@ jobs:
           command: build
           args: --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/release/jellyfin-rpc && mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-linux
+      - name: Rename binary
+        run: mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-linux
 
       - name: Run cargo test
         uses: actions-rs/cargo@v1
@@ -125,8 +125,8 @@ jobs:
           command: build
           args: --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/release/jellyfin-rpc && mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-darwin
+      - name: Rename binary
+        run: mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-darwin
 
       - name: Run cargo test
         uses: actions-rs/cargo@v1
@@ -171,8 +171,8 @@ jobs:
           command: build
           args: --target aarch64-apple-darwin --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/aarch64-apple-darwin/release/jellyfin-rpc && mv target/aarch64-apple-darwin/release/jellyfin-rpc target/aarch64-apple-darwin/release/jellyfin-rpc-arm64-darwin
+      - name: Rename binary
+        run: mv target/aarch64-apple-darwin/release/jellyfin-rpc target/aarch64-apple-darwin/release/jellyfin-rpc-arm64-darwin
 
       - name: Upload artifact
         uses: actions/upload-artifact@v3

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -35,8 +35,8 @@ jobs:
           command: build
           args: --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/release/jellyfin-rpc && mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-linux
+      - name: Rename binary
+        run: mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-linux
 
       - name: Run cargo test
         uses: actions-rs/cargo@v1
@@ -128,8 +128,8 @@ jobs:
           command: build
           args: --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/release/jellyfin-rpc && mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-darwin
+      - name: Rename binary
+        run: mv target/release/jellyfin-rpc target/release/jellyfin-rpc-x86_64-darwin
 
       - name: Run cargo test
         uses: actions-rs/cargo@v1
@@ -177,8 +177,8 @@ jobs:
           command: build
           args: --target aarch64-apple-darwin --workspace --locked --release
 
-      - name: Strip and rename binary
-        run: strip target/aarch64-apple-darwin/release/jellyfin-rpc && mv target/aarch64-apple-darwin/release/jellyfin-rpc target/aarch64-apple-darwin/release/jellyfin-rpc-arm64-darwin
+      - name: Rename binary
+        run: mv target/aarch64-apple-darwin/release/jellyfin-rpc target/aarch64-apple-darwin/release/jellyfin-rpc-arm64-darwin
 
       - name: Release
         uses: softprops/action-gh-release@v1

diff --git a/Cargo.toml b/Cargo.toml
@@ -9,3 +9,9 @@ default-members = [
     "jellyfin-rpc",
     "jellyfin-rpc-cli",
 ]
+
+[profile.release]
+strip = true
+lto = true
+codegen-units = 1
+panic = "abort"
diff --git a/jellyfin-rpc-cli/Cargo.toml b/jellyfin-rpc-cli/Cargo.toml
@@ -43,3 +43,9 @@ default-features = false
 features = ["rustls-tls", "json"]
 version  = "0.11"
 optional = true
+
+[profile.release]
+strip = true
+lto = true
+codegen-units = 1
+panic = "abort"