Skip to content

Fix Signing#6069

Merged
Byron merged 3 commits into
gitbutlerapp:masterfrom
Byron:signing
Jan 24, 2025
Merged

Fix Signing#6069
Byron merged 3 commits into
gitbutlerapp:masterfrom
Byron:signing

Conversation

@Byron

@Byron Byron commented Jan 23, 2025

Copy link
Copy Markdown
Collaborator

Tasks

  • Debugging tool for environment variables
  • path-quoting for the command itself
  • switch to gitoxide on main
  • test it for real once!

@vercel

vercel Bot commented Jan 23, 2025

Copy link
Copy Markdown

@Byron is attempting to deploy a commit to the GitButler Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added rust Pull requests that update Rust code @gitbutler/desktop labels Jan 23, 2025
@vercel

vercel Bot commented Jan 23, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 24, 2025 9:03am

@Caleb-T-Owens

Copy link
Copy Markdown
Contributor

@Byron if the command has already been quoted, should we strip them out? For a short while the advice on discord and GH was to add them, but now they could be problematic?

@Byron

Byron commented Jan 24, 2025

Copy link
Copy Markdown
Collaborator Author

The issue was that the command is now run as shell-command on Windows, which also means commands would need to be quoted to handle paths with spaces like …/Program Files/… correctly.
It's quite sad I didn't see that coming as well.

Comment thread apps/desktop/src/routes/+layout.svelte
Comment thread crates/gitbutler-tauri/src/env.rs
@Byron
Byron deleted the signing branch January 24, 2025 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@gitbutler/desktop rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SSH: Unable to find program

3 participants