Merge pull request #1115 from napi-rs/fix-cargo-name-restrict

fix(cli): should not throw if cargoName is provided but no package.name

.github/workflows/cli-binary.yml

@@ -51,6 +51,13 @@ jobs:
         env:
           RUST_BACKTRACE: 1
 
+      - name: Pass -p and --cargo-name to build
+        run: |
+          node ./cli/scripts/index.js build -p napi-examples-binary --cargo-name napi-examples-binary 
+          ./napi-examples-binary
+        env:
+          RUST_BACKTRACE: 1
+
       - name: Clear the cargo caches
         run: |
           cargo install cargo-cache --no-default-features --features ci-autoclean

cli/src/build.ts

@@ -204,6 +204,8 @@ export class BuildCommand extends Command {
     let cargoPackageName: string
     if (tomlContent.package?.name) {
       cargoPackageName = tomlContent.package.name
+    } else if (this.cargoName) {
+      cargoPackageName = this.cargoName
     } else {
       throw new TypeError('No package.name field in Cargo.toml')
     }