Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

[WIP] Integrate Arbiter #1

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

[WIP] Integrate Arbiter #1

wants to merge 25 commits into from

Conversation

jspahrsummers
Copy link
Member

@jspahrsummers jspahrsummers commented Oct 2, 2016

Resolves ArbiterLib/Arbiter#62.

To do:

  • Documentation
  • Fix Arbiter's build order so that ReactiveTask is no longer building before ReactiveCocoa
  • Refactor Arbiter's support for commit pinning, since it doesn't work for Carthage
  • Restore Git reference support
  • Restore support for lastResolved and dependenciesToUpdate
  • Improve error reporting
  • Pin to Arbiter master

@jspahrsummers
Copy link
Member Author

jspahrsummers commented Oct 2, 2016

Made some good progress here, but there are some memory errors, and not all Carthage features are supported yet.

(And, of course, Arbiter only works serially right now, so I expect performance/UX to be worse.)

@jspahrsummers
Copy link
Member Author

jspahrsummers commented Oct 2, 2016

Was just now able to run this on a fresh clone of Carthage. 🎉

It mostly worked. The commit pin for Nimble was ignored (as expected, because I didn't implement it yet), and the build order was slightly wrong, but version selections were otherwise correct.

@jspahrsummers
Copy link
Member Author

Build order is fixed. 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants