Skip to content

[flutter_tools] Add --no-codesign support for macOS build #169034

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

knopp
Copy link
Member

@knopp knopp commented May 17, 2025

Unlike iOS, the macOS build currently does not support --no-codesign option to skip signing of application bundle. Having this option is useful for situation where the code-signing step is happens separately after the bundle is built (i.e. on CI).

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@knopp knopp requested a review from a team as a code owner May 17, 2025 16:33
@github-actions github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. a: desktop Running on desktop team-ios Owned by iOS platform team labels May 17, 2025
@knopp knopp requested a review from vashworth May 17, 2025 16:34
@knopp knopp force-pushed the macos_no_codesign branch from 3dc95d4 to 3fe7d52 Compare May 21, 2025 12:09
@@ -192,6 +193,10 @@ Future<void> buildMacOS({
}
}

if (!codesign) {
globals.printStatus('Codesigning disabled with --no-codesign.');
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I would move this into the command and make the message more like the iOS version:

if (environmentType == EnvironmentType.physical && !shouldCodesign) {
globals.printStatus(
'Warning: Building for device with codesigning disabled. You will '
'have to manually codesign before deploying to device.',
);
}

@knopp knopp force-pushed the macos_no_codesign branch 3 times, most recently from 11717f1 to 9d4c2de Compare June 9, 2025 12:13
@knopp knopp force-pushed the macos_no_codesign branch from 9d4c2de to 2c1c1d9 Compare June 11, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: desktop Running on desktop team-ios Owned by iOS platform team tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants