-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
[iOS] Use configuration type when adding ndebug flag to pods in release #48193
[iOS] Use configuration type when adding ndebug flag to pods in release #48193
Conversation
@@ -664,7 +664,7 @@ def self.add_ndebug_flag_to_pods_in_release(installer) | |||
|
|||
installer.aggregate_targets.each do |aggregate_target| | |||
aggregate_target.xcconfigs.each do |config_name, config_file| | |||
is_release = config_name.downcase.include?("release") || config_name.downcase.include?("production") | |||
is_release = aggregate_target.user_build_configurations[config_name] == :release |
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.
@@ -678,7 +678,7 @@ def self.add_ndebug_flag_to_pods_in_release(installer) | |||
|
|||
installer.target_installation_results.pod_target_installation_results.each do |pod_name, target_installation_result| | |||
target_installation_result.native_target.build_configurations.each do |config| | |||
is_release = config.name.downcase.include?("release") || config.name.downcase.include?("production") | |||
is_release = config.type == :release |
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.
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.
This is great, amazing work. Thank you so much!
@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@cipolleschi merged this pull request in 462fae4. |
This pull request was successfully merged by @benhandanyan in 462fae4 When will my fix make it into a release? | How to file a pick request? |
Summary:
While I was working on fixing the iOS debugger logic based on configuration name regex match, I wanted to know if other logic was also based on configuration names. I think I found and fixed the only other configuration name-based logic in the repo in this PR.
Changelog:
[IOS] [CHANGED] - Use configuration type when adding ndebug flag to pods in release
Test Plan:
In a fresh react-native project, I added to the Podfile:
to confirm my logic. It output the following:
I also updated the applicable tests I could find for this logic.