Skip to content

Commit

Permalink
Issue #3241 no Main Class attribute in MANIFEST for jetty-runner
Browse files Browse the repository at this point in the history
Signed-off-by: olivier lamy <oliver.lamy@gmail.com>
  • Loading branch information
olamy committed Jan 9, 2019
1 parent da3d692 commit 19c4eb1
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 17 deletions.
39 changes: 22 additions & 17 deletions jetty-runner/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -35,23 +35,6 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId>
<configuration>
<!-- jetty-runner is not an OSGi component -->
<skip>true</skip>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
<archive>
<manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>
</archive>
</configuration>
</plugin>
</plugins>
<pluginManagement>
<plugins>
Expand All @@ -63,6 +46,28 @@
<skip>true</skip>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId>
<configuration>
<!-- jetty-runner is not an OSGi component -->
<skip>true</skip>
<!-- there is no way to skip MANIFEST creation by the plugin -->
<manifestLocation>${java.io.tmpdir}/NON_USED_MANIFEST</manifestLocation>

This comment has been minimized.

Copy link
@joakime

joakime Jan 10, 2019

Contributor

Perhaps this should be somewhere in target/ directory instead?

This comment has been minimized.

Copy link
@olamy

olamy Jan 10, 2019

Author Member

yup good point

</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<configuration>
<archive>
<manifestFile>src/main/resources/MANIFEST.MF</manifestFile>
<manifest>
<mainClass>org.eclipse.jetty.runner.Runner</mainClass>
</manifest>
</archive>
</configuration>
</plugin>
</plugins>
</pluginManagement>
</build>
Expand Down
1 change: 1 addition & 0 deletions jetty-runner/src/main/resources/MANIFEST.MF
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Comment: Jetty Runner

0 comments on commit 19c4eb1

Please sign in to comment.