Skip to content

Add thin jar launcher (downloads dependencies as required on startup) #7211

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

Closed
wants to merge 10 commits into from

Conversation

dsyer
Copy link
Member

@dsyer dsyer commented Oct 24, 2016

Grape and aether are sort of independent and it would be
useful to have the aether bits on their own to use as a library
in other places. The main entry point is the AetherEngine
with service locators for additional session configurations.

See gh-1813. This is not a complete solution for that yet, just
some refactoring to prepare the ground.

@dsyer dsyer changed the title Refactor aether parts of AetherGrapeEngine into new module Add thin jar launcher (downloads dependencies as required on startup) Oct 24, 2016
@dsyer
Copy link
Member Author

dsyer commented Oct 24, 2016

That last commit adds the actual ThinJarLauncher and a new packaging (-exec.jar) for the loader.

kdombeck and others added 4 commits October 24, 2016 11:14
Grape and aether are sort of independent and it would be
useful to have the aether bits on their own to use as a library
in other places. The main entry point is the AetherEngine
with service locators for additional session configurations.
@dsyer dsyer force-pushed the feature/thin-launcher branch from 52b1c8b to 2615f10 Compare October 24, 2016 10:15
@dsyer dsyer force-pushed the feature/thin-launcher branch from b6c26e6 to 954c8f2 Compare October 24, 2016 11:36
@philwebb philwebb added the for: team-attention An issue we'd like other members of the team to review label Oct 24, 2016
@dsyer
Copy link
Member Author

dsyer commented Oct 24, 2016

Moved to #7216 (this one has the wrong base branch and I don't know how to change it).

@dsyer dsyer closed this Oct 24, 2016
@philwebb philwebb removed the for: team-attention An issue we'd like other members of the team to review label Oct 24, 2016
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

3 participants