Skip to content

Commit 37453ee

Browse files
authored
Add more instructions about jar hell (#26837)
Including the fact that IntelliJ 2017.2 seems to have a spurious jar on its classpath.
1 parent 8fe9a20 commit 37453ee

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

CONTRIBUTING.md

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,16 @@ then `File->New Project From Existing Sources`. Point to the root of
106106
the source directory, select
107107
`Import project from external model->Gradle`, enable
108108
`Use auto-import`. Additionally, in order to run tests directly from
109-
IDEA 2017.2 and above it is required to disable IDEA run launcher,
110-
which can be achieved by adding `-Didea.no.launcher=true`
111-
[JVM option](https://intellij-support.jetbrains.com/hc/en-us/articles/206544869-Configuring-JVM-options-and-platform-properties)
109+
IDEA 2017.2 and above it is required to disable IDEA run launcher to avoid
110+
finding yourself in "jar hell", which can be achieved by adding the
111+
`-Didea.no.launcher=true` [JVM
112+
option](https://intellij-support.jetbrains.com/hc/en-us/articles/206544869-Configuring-JVM-options-and-platform-properties)
113+
or by adding `idea.no.launcher=true` to the
114+
`idea.properties`[https://www.jetbrains.com/help/idea/file-idea-properties.html]
115+
file which can be accessed under Help > Edit Custom Properties within IDEA. You
116+
may also need to [remove `ant-javafx.jar` from your
117+
classpath][https://github.com/elastic/elasticsearch/issues/14348] if that is
118+
reported as a source of jar hell.
112119

113120
The Elasticsearch codebase makes heavy use of Java `assert`s and the
114121
test runner requires that assertions be enabled within the JVM. This

0 commit comments

Comments
 (0)