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

Relative paths in 'require' statement #17

Open
wied03 opened this issue Apr 11, 2016 · 3 comments
Open

Relative paths in 'require' statement #17

wied03 opened this issue Apr 11, 2016 · 3 comments
Assignees
Milestone

Comments

@wied03
Copy link
Collaborator

wied03 commented Apr 11, 2016

Once opal/opal#1286 is addressed, need to figure out how to deal with this. Not require_relative but require.

Ideally, we use only the current directory to resolve paths with a leading dot, which is how MRI seems to behave, then we strip that out when the Opal.module is declared with Opal.modules['something'] and instead of Opal.modules['./something']

@wied03 wied03 added this to the 0.10.0 milestone Apr 11, 2016
@wied03
Copy link
Collaborator Author

wied03 commented Apr 12, 2016

@cj - what was your conclusion on this?

@wied03 wied03 changed the title Remove Node Style Require Support? Remove Node Style Require Support (for requiring opal assets)? Apr 12, 2016
@wied03 wied03 modified the milestones: 1.0.1, 1.0.0 Apr 12, 2016
@wied03 wied03 changed the title Remove Node Style Require Support (for requiring opal assets)? Relative paths in 'require' statement Apr 13, 2016
@wied03 wied03 added bug and removed question labels Apr 13, 2016
@wied03 wied03 modified the milestones: 1.1.0, 1.0.1 Apr 13, 2016
@elia
Copy link

elia commented Apr 18, 2016

relevant: opal/opal#1286 (comment)

@wied03
Copy link
Collaborator Author

wied03 commented Apr 21, 2016

opal/opal#1454 might address this once it's cleaned up and bootstrap issues dealt with

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants