Skip to content

Create a Mac target with tests#44

Merged
AliSoftware merged 2 commits intoAliSoftware:masterfrom
KyleLeneau:mac_target
Dec 15, 2013
Merged

Create a Mac target with tests#44
AliSoftware merged 2 commits intoAliSoftware:masterfrom
KyleLeneau:mac_target

Conversation

@KyleLeneau
Copy link
Contributor

After doing some wonderful testing with your library I wanted to contribute a Mac test target. I am developing a cross platform library and needed a mac framework for my testing.

@AliSoftware
Copy link
Owner

Nice one, will definitely merge this soon 👍

I don't have my Mac with me those days so it's hard to see this directly on GitHub, but do you confirm that you only made changes on the schemes/target generating the Example app and the Test target, but not the Static Library?

I'm asking because I never actually tested OHHTTPStubs on a Mac target myself, so I'm especially wondering if the CocoaPods' OHHTTPStubs.podspec file is configured correctly so that you were able to add pod "OHHTTPStubs" to your Podfile and execute a pod install/pod update to add it in your own Mac application without any difficulty?
Normally it should work without any problem, but as I never had the opportunity to check it in a real Mac project (only iOS projects), it seems to be the perfect occasion to confirm that using OHHTTPStubs with CocoaPods in a Mac app works properly.

Thanks for your feedback! 🍻

@KyleLeneau
Copy link
Contributor Author

I didn't really do anything with cocoa pods since I don't really use it. However I did change the static iOS library and I added a mac framework library to the Xcode project file targets. I also updated the iOS test target and added a mac text target to test the framework. All in all the Xcode project files has 4 total targets: Mac framework, Mac test target that test framework, iOS static library, and iOS test target that tests the static library.

I did not update or add an example Mac application or project and I did not test or update anything to do with the podfile.

AliSoftware added a commit that referenced this pull request Dec 15, 2013
@AliSoftware AliSoftware merged commit d6291c0 into AliSoftware:master Dec 15, 2013
AliSoftware added a commit that referenced this pull request Nov 3, 2015
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