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

[iOS] Add swift version to podspec #690

Merged
merged 1 commit into from
Apr 16, 2020
Merged

[iOS] Add swift version to podspec #690

merged 1 commit into from
Apr 16, 2020

Conversation

bang9
Copy link
Contributor

@bang9 bang9 commented Aug 6, 2019

added Swift version, default version is Swift 4.0(with Auto linking)
It occur Swift compile error when building after auto link, so we need specify Swift version in pod spec (it can be fixed by manually choose Swift version above 4.2, but it is not efficiently)

image

I'm using react-native-track-player@2.0.0-rc13

@ovr
Copy link
Contributor

ovr commented Oct 29, 2019

ping @dcvz @Guichaguri

Can you merge this onliner fix? This will fix:

- `react-native-track-player` does not specify a Swift version and none of the targets (`XXXXXXX`) integrating it have the `SWIFT_VERSION` attribute set. Please contact the author or set the `SWIFT_VERSION` attribute in at least one of the targets that integrate this pod.

And cherry pick to 1.x branch

edwardlai3582 added a commit to edwardlai3582/react-native-track-player that referenced this pull request Dec 24, 2019
@bang9 bang9 requested a review from Guichaguri February 4, 2020 13:26
@curiousdustin curiousdustin requested a review from dcvz April 14, 2020 15:29
@curiousdustin curiousdustin changed the title Update react-native-track-player.podspec [iOS] Add swift version to podspec Apr 16, 2020
@curiousdustin curiousdustin merged commit fa9dc66 into doublesymmetry:dev Apr 16, 2020
curiousdustin added a commit to curiousdustin/react-native-track-player that referenced this pull request Apr 23, 2020
…player into bugfix/ios-events

* 'dev' of github.com:react-native-kit/react-native-track-player:
  [iOS] Simplify Podfile
  [iOS] Replace git submodule with SwiftAudio podspec dependency (doublesymmetry#904)
  Added Discord invite link to the issue chooser
  Removed the old issue template
  Updated issue templates to the new model
  [iOS] Fix local images not loading on lock screen. doublesymmetry#562 (doublesymmetry#897)
  [iOS] Update podspec swift version to match SwiftAudio submodule. Cleanup quotes and other formatting.
  [iOS] Add swift version to podspec (doublesymmetry#690)
  [iOS] Resolve promise for updateMetadataForTrack (doublesymmetry#843)
  [iOS] fix: Exclude SwiftAudio unnecessary files (doublesymmetry#873)

# Conflicts:
#	ios/RNTrackPlayer/RNTrackPlayer.swift
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants