Skip to content

Do not add CI build settings to macOS#71860

Merged
jmagman merged 1 commit into
flutter:masterfrom
jmagman:mac-build-settings
Dec 7, 2020
Merged

Do not add CI build settings to macOS#71860
jmagman merged 1 commit into
flutter:masterfrom
jmagman:mac-build-settings

Conversation

@jmagman

@jmagman jmagman commented Dec 7, 2020

Copy link
Copy Markdown
Member

Description

#43553 added the ability to set Xcode build settings from environment variables so the devicelab could set up provisioning profiles. Using the same settings for iOS and macOS is causing macOS builds to fail. We think it is related to newer versions of CocoaPods incorrectly signing frameworks? (only executables like apps or extensions should be signed).

Related Issues

Workaround #71613

@jmagman jmagman added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Dec 7, 2020
@jmagman jmagman requested a review from yusufm December 7, 2020 19:26
@jmagman jmagman self-assigned this Dec 7, 2020
@flutter-dashboard flutter-dashboard Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Dec 7, 2020
@flutter-dashboard

Copy link
Copy Markdown

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@google-cla google-cla Bot added the cla: yes label Dec 7, 2020
@yusufm yusufm requested a review from godofredoc December 7, 2020 19:29
@jmagman jmagman merged commit 9a31c39 into flutter:master Dec 7, 2020
@jmagman jmagman deleted the mac-build-settings branch December 7, 2020 20:18
christopherfujino pushed a commit to chris-forks/flutter that referenced this pull request Jan 20, 2021
christopherfujino added a commit that referenced this pull request Jan 22, 2021
* Flutter clean Flutter.podspec (#72306)

* Do not add CI build settings to macOS (#71860)

* update engine revision to 2f0af3715217a0c2ada72c717d4ed9178d68f6ed

Co-authored-by: Jenn Magder <magder@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants