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

Carthage #5

Merged
merged 5 commits into from
Apr 20, 2019
Merged

Carthage #5

merged 5 commits into from
Apr 20, 2019

Conversation

madhavajay
Copy link
Contributor

This adds an Xcode Project and Preprocessing target which builds geos.framework for iOS, macOS and tvOS.

It would be a good idea to add test targets which run the test files and add this to Travis CI or something.

- Added macOS and tvOS targets
- Updated README.md
- Cleaned up scheme names and removed xcuserdata
- Added Preprocessor target as a Target Dependency to all 3 Platforms
- Updated podspec to support tvOS
- Requires a version bump and pod trunk push
@macdrevx
Copy link
Member

Looks great. I'm going to bump the version number and make a new release that includes this.

@macdrevx
Copy link
Member

I actually think we need to break with our tradition of following the version numbers from geos itself. Instead, the version numbers we use here will just be for our build config, not for the underlying lib.

@macdrevx macdrevx merged commit 6886cf8 into GEOSwift:master Apr 20, 2019
@madhavajay
Copy link
Contributor Author

Awesome! Also don't forget you can add tvOS support now to the podspec for the next podspec release and it should also update the README.md badge from Cocoapods.

@macdrevx
Copy link
Member

Pushing it now. Trunk pushes were disabled for maintenance on the specs repo when I tried doing it over the weekend.

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.

2 participants