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

fix: app name contains spaces #194

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

heyikang
Copy link

@heyikang heyikang commented May 9, 2024

fix #193

@heyikang heyikang requested a review from a team as a code owner May 9, 2024 07:11
Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

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

It's not clear how this fixes your issue, can you add an explanation in the PR for why the absolute path with a space in it works but the relative path with a space does not?

@kangfenmao
Copy link

kangfenmao commented Aug 27, 2024

App Name: 900 TG

• Notarizing app: /path/to/dist/mac-arm64/900 TG.app
⨯ Failed to display codesign info on your application with code: 1

900 TG.app: No such process
Failed to codesign your application with code: 1
900 TG.app: No such process
900 TG.app: No such process

This PR fixed this issue

@erickzhao erickzhao self-assigned this Sep 4, 2024
@shide1989
Copy link

shide1989 commented Sep 9, 2024

@MarshallOfSound
This PR actually solves the fact that the spawn of the codesign MacOS utility has the correct fullpath to the packaged app.

The previous version would simply try to codesign the packaged app in the current working directory (which is wrong of course) because of wrong arguments passing.
Which causes official Electron boilerplates like (https://www.electronforge.io/templates/vite-+-typescript) to fail to pass the signing step when using yarn make.

@heyikang
Copy link
Author

When notarizing, using a relative path for the codesign parameter and an application name with spaces causes the notarization to fail. This issue does not occur with absolute paths.

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.

Notarize fail
5 participants