Skip to content

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Oct 19, 2018

Flutter tooling creates a Generated.xcconfig file and sets a FLUTTER_BUILD_MODE=debug/release/profile as approrpiate. In an add to app scenario, we want this to be driven by the Xcode build mode. Current instructions tell users to create and update xcconfig files manually - this change will let Cocoapods take care of it for them.

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

Can you provide some context for the change? (in the commit comment)

@dnfield
Copy link
Contributor Author

dnfield commented Oct 19, 2018

@cbracken added a body to this - would you like some comments in the ruby file as well?

File.open(xcconfig_path, 'a+') do |file|
file.puts "#include \"#{File.join(framework_dir, 'Generated.xcconfig')}\""
if config.name == 'Release'
file.puts "FLUTTER_BUILD_MODE=release"
Copy link
Member

@cbracken cbracken Oct 19, 2018

Choose a reason for hiding this comment

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

If I'm reading this right, this only affects pods, and not the main project, correct? Specifically, when building for simulator, we only support debug builds. (#17735)

@dnfield
Copy link
Contributor Author

dnfield commented Oct 20, 2018

I'm going to close this and open a separate PR with the approach we discussed this afternoon.

@dnfield dnfield closed this Oct 20, 2018
@dnfield dnfield mentioned this pull request Oct 23, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants