Skip to content
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

Add swift version to podspec #519

Merged
merged 1 commit into from
Jan 25, 2019
Merged

Add swift version to podspec #519

merged 1 commit into from
Jan 25, 2019

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