Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make tests pass on debian jessie #360

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

Conversation

tomclegg
Copy link

No description provided.

@postmodern
Copy link
Owner

Perhaps we should have test/setup exit with -1 if it could not download the ruby?

@tomclegg
Copy link
Author

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.

@tomclegg
Copy link
Author

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.

@postmodern
Copy link
Owner

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.

@tomclegg
Copy link
Author

@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...

@postmodern
Copy link
Owner

@tomclegg ah yes, let's move the directory test to test/setup.

@tomclegg
Copy link
Author

@postmodern is 0b14dbc more to your liking?

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