chruby
chruby copied to clipboard
postmodern
make tests pass on debian jessie
Perhaps we should have test/setup exit with -1 if it could not download the ruby?
That part (exit non-zero after an error) seems to work properly already. The trouble is just that (without this change) the next time you run make test, the test/opt/rubies dir already exists, and according to Makefile that means it is not necessary to run test/setup any more. Therefore, tests just fail until you manually clean up and start over.
btw I noticed a bit too late that the "just major version" change here duplicates #331. The discussion there says "8.0 is there now" but today 8.3 isn't there so it's broken again. I looked at the RVM source code (from which the code here was originally taken) and the current version uses only the major version. I figured things are most likely to keep working if chruby uses the same url RVM itself uses.
I'm going to experiment with trying to move the ruby version/path variables into the Makefile, that way we can define an explicit dependency on path/to/test/ruby-X.Y.Z.
@postmodern if the idea is to avoid the extra "done" file, moving the "skip if done" responsibility from Makefile to setup (see amended commit) is probably easier...