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

After dist, play do not pick up jars from lib directory #123

Closed
hadesara opened this issue Jan 8, 2014 · 3 comments
Closed

After dist, play do not pick up jars from lib directory #123

hadesara opened this issue Jan 8, 2014 · 3 comments

Comments

@hadesara
Copy link

hadesara commented Jan 8, 2014

After Play dist
In the bash/bat script under bin. Currently it has a hardcoded list of jar files that must be loaded in runtime. However if you add any additional jar files in the 'lib' directory it will not be pickup by runtime. Instead, better approach would to loop through the lib directory and all the jar files.

@jsuereth
Copy link
Member

jsuereth commented Jan 8, 2014

We purposely don't do this for a very subtle reason: Jar ordering.

It is entirely possible that the order in which you pass jars is important for correct execution of the program, thanks to shaded ibraries, multi-versioned libraries and all sorts of other madness. If you need additional jars at runtime, I believe you can sneak them into the classpath that the packager looks at by hand.

Also, in the M1 you can provide your own BASH template which does just as you mention. If you'd like to contribute an option to use such a variant BASH template instead, I'd be all for that. However, I think the default behavior should be to have the order of the JARs at production match your dev environment. Configuring another option is then a conscious choice, but one that's probably ok for 80-90% of applications.

@hadesara
Copy link
Author

hadesara commented Jan 8, 2014

If so, at least this: #124 should be doable right?
Where do I find this BASH template?

@jsuereth
Copy link
Member

jsuereth commented Jan 8, 2014

@jsuereth jsuereth closed this as completed Jan 8, 2014
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