Having two Solvers() that do different things is ugly. Thanks for cleaning that up.

This probably conflicts with #4692 as will anything that touches script.

Thanks, I hadn't seen that PR and it definitely interests me since I'm almost exclusively working on refactor script lately. I don't mind to rebase all my work on top of that since the hardest part was funding out the changes I wanted to make.

Dependent on #4692

Rebased on top of #4754

Rebased on top of #4755

Closing until #4754 is merged.

I haven't reviewed fully, but this seems like it could be simplified on top #4890.

I have already a version rebased on top of that, but I don't see the simplification...

You don't need to pass txTo and nIn around - just pass the SIgnatureHasher around instead.

Oh no, sorry - you still need it as the function updates the transaction itself, not just produce the signature. Nevermind.

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4694_87784ec8d0720affac6d133bf550a0a6471b530b/ for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

Rebased.

Closing until 0.10