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

Adjust path in require statement to work with Ruby 1.9.2 and later #2

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

Conversation

skissane
Copy link

In Ruby 1.9.2 and later, the current directory has been removed from the
LOAD_PATH, due to security reasons. This causes an error when running make. The
solution is to prefix the module name with ./ to force it to be loaded from the
current directory.

In Ruby 1.9.2 and later, the current directory has been removed from the
LOAD_PATH, due to security reasons. This causes an error when running make. The
solution is to prefix the module name with ./ to force it to be loaded from the
current directory.
Ignores all files generated by running make
@andychu
Copy link

andychu commented Dec 30, 2018

FWIW I just ran into this issue. I worked around it with

export RUBYLIB=.

(I don't know Ruby so this took some Googling to figure out, and then I encountered this issue.)

This is a nice piece of code, I linked it here :)

https://www.reddit.com/r/ProgrammingLanguages/comments/aazbwp/habermanvtparse_a_library_to_parse_terminal/

andychu pushed a commit to oilshell/vtparse that referenced this pull request Dec 30, 2018
@andi8086
Copy link

andi8086 commented Mar 5, 2024

:D Funily I also provided a fix for this, now 5 years later... a shame the author does not care about the pull requests...

@haberman
Copy link
Owner

haberman commented Mar 5, 2024

Sorry I haven't maintained this well. This is some code I wrote 17 years ago for a project I was doing at the time, but I no longer have the time or interest to maintain it. If you find it useful I'd encourage you to adopt it. If you do so I'd be happy to link to your version.

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.

None yet

4 participants