Skip to content

Allow custom module path normalisation #184

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

Merged
merged 3 commits into from
Dec 6, 2015

Conversation

stesie
Copy link
Member

@stesie stesie commented Dec 6, 2015

Currently the module path normalisation is done by V8Js itself in C++ code.

However, for example to load modules from a node.js module tree, it is required to influence the module path normalisation, as the files are scattered over sub directories and this needs to be taken into consideration.

Hence this allows to provide a custom normalisation routine, which overrides the V8Js default one, if set. Otherwise V8Js' default routine is used.

Otherwise long module identifiers might get cut off.
This might be perfectly valid, if you're using a third-party module,
which requires a module yet doesn't use it in the code paths hit
and hence you just want to stub it out.
stesie added a commit that referenced this pull request Dec 6, 2015
Allow custom module path normalisation
@stesie stesie merged commit 28f0611 into phpv8:master Dec 6, 2015
@stesie stesie deleted the cust-module-normalisation branch December 31, 2015 19:59
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.

1 participant