search

vert-x / vertx-maven

Vert.x 2.x is deprecated - use instead
https://github.com/vert-x3/vertx-maven-starter
Other
22 stars 31 forks source link

vertx:runModXXX should not use hardcoded paths for target/classes and source/main/resources #15

Closed bckfnn closed 10 years ago

bckfnn commented 11 years ago

It is bad style since it is possible to configure maven to use different source and target folders. OTOH I have very little sympathy for maven project with a non-standard layout so this issue is minor.

purplefox commented 11 years ago

I agree the classpaths should be configurable.

Fancy submitting a PR? :)

purplefox commented 11 years ago

Come to think of it... this is already possible if you use mvn vertx:runModOnCP and provide the classpath property in the pom.xml:

https://github.com/vert-x/vertx-maven/blob/master/archetype/src/main/resources/archetype-resources/pom.xml#L121

It's not currently configurable though if you use runModIDEA/runModEclipse

purplefox commented 11 years ago

I've made some changes so that runModIDEA and runModEclipse are now also configurable using the classpath property. Can you take a look and check if this works for you?

purplefox commented 10 years ago

runmodXXX now no longer exists and a vertx_classpath.txt is used to configure classpath