Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(bundler): support custom sign command on Windows #9902

Merged
merged 2 commits into from
Jun 3, 2024
Merged

feat(bundler): support custom sign command on Windows #9902

merged 2 commits into from
Jun 3, 2024

Conversation

muwoo
Copy link

@muwoo muwoo commented May 29, 2024

feat: Migrate v2 functionality to v1.x: support custom sign command on Windows (Discussed in #9900)

  • cargo test passes
  • cargo clippy passes

@muwoo muwoo requested a review from a team as a code owner May 29, 2024 07:51
Copy link
Member

@amrbashir amrbashir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you need to run cargo fmt and cargo clippy -- -Dwarnings

@muwoo
Copy link
Author

muwoo commented Jun 3, 2024

Looks like you need to run cargo fmt and cargo clippy -- -Dwarnings

done

@amrbashir amrbashir merged commit 5909662 into tauri-apps:1.x Jun 3, 2024
32 checks passed
Copy link

@MystiPanda MystiPanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it needs to be modified here.

@amrbashir Could you please take a look at it?

@@ -68,6 +67,7 @@ pub fn bundle_project(settings: &Settings, updater: bool) -> crate::Result<Vec<P
let nsis_toolset_path = tauri_tools_path.join("NSIS");

if !nsis_toolset_path.exists() {
create_dir_all(&nsis_toolset_path)?;
Copy link

@MystiPanda MystiPanda Jun 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creating an NSIS folder in advance will cause subsequent renaming failures after extract

rename(_tauri_tools_path.join("nsis-3.08"), nsis_toolset_path)?;

@tr3ysmith
Copy link
Contributor

@amrbashir any thoughts on when v1.7 would be released that contains this fix?

@amrbashir
Copy link
Member

I don't want to make promises but hopefully this week

@tr3ysmith
Copy link
Contributor

@amrbashir I see 1.7 came out, but I don't see anything about this feature in the release notes, is it indeed in there?

@amrbashir
Copy link
Member

@ttrushin
Copy link

ttrushin commented Jul 2, 2024

Maybe a silly question, but I have tauri updated to 1.7.1 (and tauri-build updated to 1.5.3) so everything appears to be updated to latest, but I cannot build with "signCommand".

Error `tauri.conf.json` error on `tauri > bundle > windows`: Additional properties are not allowed ('signCommand' was unexpected)

Anyone know why?

@amrbashir
Copy link
Member

make sure to also update the CLI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants