We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The com.github.spullara.mustache.java:compiler dependency is not being shaded in the Elasticsearch fat jar:
com.github.spullara.mustache.java:compiler
$ curl -s http://repo1.maven.org/maven2/org/elasticsearch/elasticsearch/1.1.1/elasticsearch-1.1.1.jar | jar tv | grep -i mustache 0 Wed Apr 16 14:28:38 EDT 2014 org/elasticsearch/script/mustache/ 5705 Wed Apr 16 14:28:38 EDT 2014 org/elasticsearch/script/mustache/MustacheScriptEngineService.class 1598 Wed Apr 16 14:28:38 EDT 2014 org/elasticsearch/script/mustache/JsonEscapingMustacheFactory.class 3246 Wed Apr 16 14:28:38 EDT 2014 org/elasticsearch/script/mustache/MustacheScriptEngineService$MustacheExecutableScript.class 0 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/ 175 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/Binding.class 639 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/Code.class 0 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/ 6514 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/DefaultCode.class 1951 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/DefaultMustache.class 758 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/DepthLimitedWriter.class 4121 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/ExtendCode.class 1025 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/ExtendNameCode.class 1737 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/IterableCode$1.class 5438 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/IterableCode.class 2183 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/NotIterableCode.class 3939 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/PartialCode.class 1772 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/ValueCode$1.class 5351 Wed Apr 16 14:28:42 EDT 2014 com/github/mustachejava/codes/ValueCode.class [...]
Strange since it is being included in the list of shaded dependencies: https://github.com/elasticsearch/elasticsearch/blob/master/pom.xml#L585
Maybe it's because it is declared as an optional dependency? https://github.com/elasticsearch/elasticsearch/blob/master/pom.xml#L176
The text was updated successfully, but these errors were encountered:
good catch I will provide a PR in a bit...
Sorry, something went wrong.
Shade mustache into org.elasticsearch.common package
13f37b3
Previously we shared the jar but never rewrote the packages such that the shading had no effect. Closes elastic#6192
7dcb9d4
Previously we shared the jar but never rewrote the packages such that the shading had no effect. Closes #6192
45314bc
1aea0ff
s1monw
Successfully merging a pull request may close this issue.
The
com.github.spullara.mustache.java:compiler
dependency is not being shaded in the Elasticsearch fat jar:Strange since it is being included in the list of shaded dependencies: https://github.com/elasticsearch/elasticsearch/blob/master/pom.xml#L585
Maybe it's because it is declared as an optional dependency? https://github.com/elasticsearch/elasticsearch/blob/master/pom.xml#L176
The text was updated successfully, but these errors were encountered: