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

CocoaPods podspec for v1.1.0 #7

Merged
merged 1 commit into from Mar 12, 2015

Conversation

Projects
None yet
2 participants
@banjun
Copy link
Contributor

commented Mar 12, 2015

refs #2 #4

added a CocoaPods podspec file pointing to v1.1.0 and LICENSE file. (LICENSE file is required for pod validation.)

You can publish this pod after merge, according to the guide:
http://guides.cocoapods.org/making/getting-setup-with-trunk.html

  1. signing up: pod trunk register YOU@example.com 'YOUR NAME' --description='macbook air'
  2. sending to the Specs repo: pod trunk push Socket.IO-Client-Swift.podspec
@banjun

This comment has been minimized.

Copy link
Contributor Author

commented Mar 12, 2015

pod trunk push validation may fail with this podspec like below..., as v1.1.0 tag does not contain LICENSE file.

 -> Socket.IO-Client-Swift (1.1.0)
    - WARN  | [iOS] Unable to find a license file

in the case, please tag after the merge commit and modify podspec's s.source line.
for example, :tag => 'v1.1.0' -> :tag => 'v1.1.0-pod'

nuclearace added a commit that referenced this pull request Mar 12, 2015

Merge pull request #7 from banjun/cocoapods
CocoaPods podspec for v1.1.0

@nuclearace nuclearace merged commit ac2bc13 into socketio:master Mar 12, 2015

@nuclearace

This comment has been minimized.

Copy link
Member

commented Mar 12, 2015

@banjun I think I did it. But I have no idea.

@banjun

This comment has been minimized.

Copy link
Contributor Author

commented Mar 12, 2015

@nuclearace thank you for merging! 👍

I confirmed we can now pod search socket.io, pod install with pod 'Socket.IO-Client-Swift', and add s.dependency 'Socket.IO-Client-Swift', '~> 1.1' to another pod!

@nuclearace

This comment has been minimized.

Copy link
Member

commented Mar 12, 2015

@banjun if you want to update the README to include a CocoaPods install section, I'll merge it.

@banjun banjun deleted the banjun:cocoapods branch Mar 12, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.