maven plugin - support for maven commands
Bug #1502031 reported by
maarten ectors
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Snapcraft |
Fix Released
|
Medium
|
Sergio Schvezov | ||
Bug Description
This is a feature request:
The MQTT Paho client is doing lots of things and extensive testing when you use the default maven plugin. The documentation recommends to use:
mvn package -DskipTests
But in the maven plugin there is no option to specify package and -DskipTests
So something like
javamqtt:
plugin: maven-project
mvnoptions: package -D skipTests
source: git://git.
Related branches
lp:~sergiusens/snapcraft/maven
- John Lenton (community): Approve
-
Diff: 205 lines (+109/-5)9 files modifiedexamples/tomcat-maven-webapp/snapcraft.yaml (+1/-1)
integration-tests/data/simple-maven/my-app/pom.xml (+18/-0)
integration-tests/data/simple-maven/my-app/src/main/java/com/canonical/testapp/App.java (+13/-0)
integration-tests/data/simple-maven/my-app/src/main/resources/META-INF/application.properties (+1/-0)
integration-tests/data/simple-maven/my-app/src/test/java/com/canonical/testapp/AppTest.java (+38/-0)
integration-tests/data/simple-maven/snapcraft.yaml (+15/-0)
integration-tests/units/jobs.pxu (+7/-0)
snapcraft/plugins/jdk.py (+0/-3)
snapcraft/plugins/maven.py (+16/-1)
| Changed in snapcraft: | |
| milestone: | none → 0.4 |
| Changed in snapcraft: | |
| assignee: | nobody → Sergio Schvezov (sergiusens) |
| Changed in snapcraft: | |
| importance: | Undecided → Medium |
| Changed in snapcraft: | |
| status: | New → Fix Committed |
Revision history for this message
| Leo Arias (elopio) wrote | #1 |
Revision history for this message
| Sergio Schvezov (sergiusens) wrote | #2 |
| Changed in snapcraft: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The fix comes with a nice integration test that confirms the fix. Nice work!