Skip to content

Conversation

@DmitryPR
Copy link

Define swift_version for the Podfile which is read when the library is
integrated

If this is not supplied then Macaw in Pods settings of the integrated
project gets the project's swift version which leads to a problem
if the integrator is using different Swift version in his project

For him the only workaround is to specify the post_install hook
where he sets the config.build_settings['SWIFT_VERSION'] = '4.2'

CocoaPods/CocoaPods#7327

For library developers who rely on this library this is also the problem
because the s.pod_target_xcconfig is ignored when using the cocoapods-packager

CocoaPods/cocoapods-packager#210

Define swift_version for the Podfile which is read when the library is
integrated

If this is not supplied then Macaw in Pods settings of the integrated
project gets the project's swift version which leads to a problem
if the integrator is using different Swift version in his project

For him the only workaround is to specify the post_install hook
where he sets the config.build_settings['SWIFT_VERSION'] = '4.2'

CocoaPods/CocoaPods#7327

For library developers who rely on this library this is also the problem
because the s.pod_target_xcconfig is ignored when using the cocoapods-packager

CocoaPods/cocoapods-packager#210
@ystrot
Copy link
Member

ystrot commented Jan 25, 2019

Hi Dmitry,

Thanks for PR, we'll try to fix it as you suggested.

@ystrot ystrot self-assigned this Jan 25, 2019
@ystrot ystrot merged commit 301d8eb into exyte:master Jan 25, 2019
@robertbarclay
Copy link

Can this be pod pushed as a version to the cocoapods repo as a new version please?

@ystrot
Copy link
Member

ystrot commented Feb 1, 2019

@robertbarclay sure, once we finish with the recent tasks we will publish a new version. I think it will be done in the nearest 2 weeks.

@ystrot ystrot added this to the 0.9.4 milestone Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants