Check for the existence of Rubinius::LIB_PATH and actually tell the user in case of error #1671

Merged
merged 3 commits into from Apr 29, 2012

3 participants

@unleashed

Current code in kernel/loader.rb ignores the check for existence of Rubinius::LIB_PATH, and the logic also disables the message about main library not existing, since it depends on Rubinius::LIB_PATH being nil, which is never the case.

Here's a fix for that, in which we first test the path specified in environment variable RBX_LIB (if any) and then the configured Rubinius::LIB_PATH, checking for existence and sending a message to the user in case the path does not exist.

Feel free to ignore the other two commits, they're small cleanups with typo fixes and a microoptimization.

@travisbot

This pull request passes (merged ade22cb into ae1fad4).

@abyx abyx merged commit 3867bfb into rubinius:master Apr 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment