-
Notifications
You must be signed in to change notification settings - Fork 17
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(packager): add support to macOS .app bundles #17
Conversation
I have the DMG PR ready as soon as this gets merged. |
Nice, I will get this reviewed and merged asap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made some changes to make it more consistent with the codebase so I'd appreciate if you can test this again, just to make sure I didn't break anything.
const KEYCHAIN_ID: &str = "cargo-packager.keychain"; | ||
const KEYCHAIN_PWD: &str = "cargo-packager"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed these to be cargo-packager, that should be okay, no?
// we should use the bundle name (App name) as a MacOS standard. | ||
// version or platform shouldn't be included in the App name. | ||
let app_product_name = format!("{}.app", config.product_name); | ||
let app_bundle_path = config.out_dir().join(&app_product_name); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed this path so the final bundle is directly under config.out_dir()
but I am not sure if this change broke anything, so could you please test this? Ideally all targets operate like this:
config.out_dir/
|_ <target>/
|_ intermediate files needed while building
|_ <another-target>/
|_ intermediate files needed while building
|_ final-packge-file-here
|_ another-final-packge-file-here
All good @amr-crabnebula |
No description provided.