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

Dependency loading #6

Open
timmc opened this issue Oct 18, 2014 · 1 comment
Open

Dependency loading #6

timmc opened this issue Oct 18, 2014 · 1 comment

Comments

@timmc
Copy link

timmc commented Oct 18, 2014

I'm considering making some major changes to liverepl in response to pain points I've found while using it at work. The most important one is the lack of dependency loading at runtime. For example, we don't want to ship tools.trace with our servers, but if we need to inspect a misbehaving server, liverepl doesn't have a way to introduce that lib as needed. I'm thinking of loading both by Aether coordinates and by filesystem path.

Is this a feature you'd be likely to accept a PR for, or should I look at ways of making this feature more separable? (E.g. add support for loading just one additional dependency at liverepl launch, and that additional thing could be used by the user to bootstrap to a dependency loader.)

If you want to chat with me about it more synchronously, I'm always on freenode IRC as TimMc.

@djpowell
Copy link
Owner

Ooh thanks, I'll take a look when I get time. Sounds like a great idea!
I've been neglecting liverepl for ages, but I do find it pretty useful. I'll try and get back on top of it soon.

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

No branches or pull requests

2 participants