jeffh / Fox

Property Based Testing Library for Objective-C and Swift. QuickCheck for Apple's Platforms.
http://fox-testing.rtfd.org
Other
624 stars 30 forks source link

Carthage support? #8

Closed modocache closed 9 years ago

modocache commented 9 years ago

Not sure if you care about supporting Carthage, but I thought I'd note some difficulties I had trying to integrate Fox with github/Archimedes (I thought it'd make an awesome example during my talk):

  1. Fox isn't versioned at all yet, so I can't install it unless I use my fork, which has temporary versions like v0.0.1. Obviously Fox'll be versioned at some point, so this isn't really a big deal. Also, I've opened an issue for Carthage to support unversioned projects (see https://github.com/Carthage/Carthage/issues/136).
  2. Carthage seems to build the test targets, which include a "Check Pods Manifest.lock" phase. But since Carthage doesn't run pod install, this step fails.

Bullet two on that list blocked me, so I ended up using Git submodules. You can see the end result at this private repo: https://github.com/modocache/Archimedes-Fox/commit/19ae6a8ea5628b6b9e6eeb47fc91bd7e72bf7bcd

jeffh commented 9 years ago

I think removing Cocoapods inside Fox and replace it with git submodules would fix this and cocoa pods isn't buying me much.

Yes, this will be versioned. Probably sometime after Thanksgiving.

jeffh commented 9 years ago

@modocache, I switched Fox to use frameworks and removed the Cocoapods dependency. This should be the last major change.

modocache commented 9 years ago

Thanks! :100:

That really should have done the trick, but now Carthage is complaining about some other silly garbage:

*** Building scheme "Cedar iOS SenTestingKit Tests" in Fox.xcodeproj
A shell task failed with exit code 65:
** BUILD FAILED **

Which is pretty ridiculous, considering the Cedar scheme is only exposed because it's being tested in that repository. I've had enough--I'll just include Fox in my project using Git submodules! :rage:

I will leave this open, though, just in case someone wants to look into it later.

modocache commented 9 years ago

https://twitter.com/jspahrsummers/status/539242618320740352

Whoops, actually we ran into this exact same problem with Quick--the solution is to create a workspace that contains Fox and all its dependencies. I think some consider this "more correct" whether you use Carthage or not.

jeffh commented 9 years ago

Ah, it makes sense to switch to a workspace then.

jeffh commented 9 years ago

I've moved Cedar into a workspace. I haven't ever got Carthage working, so I don't know if I'm missing something :(

I'm probably going to give up and let someone else try tackling this.