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

Pickup basho-patches in nodetool [JIRA: TOOLS-137] #189

Open
sargun opened this issue Aug 7, 2015 · 4 comments
Open

Pickup basho-patches in nodetool [JIRA: TOOLS-137] #189

sargun opened this issue Aug 7, 2015 · 4 comments

Comments

@sargun
Copy link
Contributor

sargun commented Aug 7, 2015

Right now, when you run nodetool from node_package, it doesn't add basho-patches to the code path when running escript. This means that if you've patches any of core OTP, you must overwrite those files as opposed to simply dropping a replacement in basho-patches.

@Basho-JIRA Basho-JIRA changed the title Pickup basho-patches in nodetool Pickup basho-patches in nodetool [JIRA: TOOLS-137] Aug 7, 2015
@Basho-JIRA
Copy link

Can this be prioritized in the next set of DevOps work?

_[posted via JIRA by Derek Somogyi]_

@Basho-JIRA
Copy link

pending feedback from Greg

_[posted via JIRA by David Andersen]_

@Basho-JIRA
Copy link

reviewed with Greg. need to sync up with Sargun again for additional info.

_[posted via JIRA by David Andersen]_

@Basho-JIRA
Copy link

This could probably done by adding something like this to the main/1 function in https://github.com/basho/node_package/blob/develop/priv/base/nodetool

true = code:add_pathz(filename:dirname(escript:script_name()) ++ "/../lib/basho-patches"),

The tricky part of this would be getting knowledge of where the basho-patches dir is. The above works for "relocatable releases" where the erts location is at the same directory level as the lib directory. Would have to do some thinking and investigating on where nodetool gets called from to get it passed in otherwise.

_[posted via JIRA by Drew Kerrigan]_

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