search

molgenis / molgenis

MOLGENIS - for scientific data: management, exploration, integration and analysis.
https://molgenis.org
GNU Lesser General Public License v3.0
111 stars 99 forks source link

Fix configuration jetty in molgenis/pom.xml and application poms #2463

Closed jjettenn closed 8 years ago

jjettenn commented 9 years ago

Related to pull request "Move jetty plugin to pluginManagement block #1890"

When using the configuration of molgenis-app pom.xml in moilgenis-ase pom.xml the omx molgenis.home directory is used.

org.mortbay.jetty jetty-maven-plugin 8079 jetty-stop molgenis.home ${user.home}${file.separator}.molgenis${file.separator}omx log4j.configuration log4j-molgenis.properties

It seems that the configurations in the pluginManagement block in the app pom is not used. The configuration in the Molgenis pom.xml is used.

The second problem is that when using de following configuration in the ase app:

org.eclipse.jetty jetty-maven-plugin 9.2.6.v20141205 8079 jetty-stop manual molgenis.home ${user.home}${file.separator}.molgenis${file.separator}ase

The clean install is also starting the jetty server.

mswertz commented 9 years ago

how often/when do we encounter this bug?

dennishendriksen commented 8 years ago

@tommydeboer is this the same issue you encountered for bbmri-eric?

tommydeboer commented 8 years ago

Yes it's the same issue (see #4113). Also, when doing a clean install for the bbmri-eric app it doesn't start the jetty server so I can't reproduce that part of the problem, which makes me think that's fixed too (see #4114 and #4118).