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
support producer-fast mode for plugin #63
Conversation
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.
Why do want to add support for producer-fast in a CocoaPods plugin?
This feature is veeery experimental and there is no active development nor plans to make it GA.
@@ -47,6 +47,8 @@ class XcodeProjBuildSettingsIntegrateAppender: BuildSettingsIntegrateAppender { | |||
result["CC"] = wrappers.cc.path | |||
result["LD"] = wrappers.ld.path | |||
result["LIBTOOL"] = wrappers.libtool.path | |||
} else { | |||
result.removeValue(forKey: "CC") |
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 class and function is an appender. I think it shouldn't remove anything. If you want to support build settings deintegration, that should be handled in a separate type.
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.
Hm. Did not mention class name. Removed it for now
@@ -169,7 +171,7 @@ def self.enable_xcremotecache(target, repo_distance, xc_location, xc_cc_path, mo | |||
postbuild_script.dependency_file = "$(TARGET_TEMP_DIR)/postbuild.d" | |||
|
|||
# Mark a sha as ready for a given platform and configuration when building the final_target | |||
if mode == 'producer' && target.name == final_target | |||
if (mode == 'producer' || mode == 'producer') && target.name == final_target |
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.
probably a typo for producer-fast
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.
for sure it is! :)
These small changes add support for "producer-fast" mode in plugin.
There's also an addition (117-118 lines) where it removes "CC" override in case project was switched from consumer to producer