Skip to content

Commit

Permalink
apacheGH-2342: Fix assembly warnings building the download
Browse files Browse the repository at this point in the history
  • Loading branch information
afs committed Mar 18, 2024
1 parent 1fbbb52 commit 696ad22
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 44 deletions.
File renamed without changes.
45 changes: 1 addition & 44 deletions apache-jena/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -152,9 +152,6 @@
<plugins>
<plugin>
<artifactId>maven-assembly-plugin</artifactId>
<configuration>
<tarLongFileMode>gnu</tarLongFileMode>
</configuration>
<executions>
<execution>
<id>create-jena-download</id>
Expand All @@ -164,54 +161,14 @@
</goals>
<configuration>
<appendAssemblyId>false</appendAssemblyId>
<tarLongFileMode>posix</tarLongFileMode>
<descriptors>
<descriptor>assembly-jena-zip.xml</descriptor>
<descriptor>assembly-jena-download.xml</descriptor>
</descriptors>
<tarLongFileFormat>gnu</tarLongFileFormat>
</configuration>
</execution>
</executions>
</plugin>

<!--
Use the dependencies plugin to copy the dependencies into the lib/ directory which makes the scripts work in dev enivronments
- -> JENA-1173
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
<executions>
<execution>
<id>copy-libs-for-scripts</id>
<phase>package</phase>
<goals>
<goal>copy-dependencies</goal>
</goals>
<configuration>
<overWriteSnapshots>true</overWriteSnapshots>
<includeScope>runtime</includeScope>
<includeTypes>jar</includeTypes>
<outputDirectory>lib/</outputDirectory>
</configuration>
</execution>
</executions>
</plugin>
-->

<!--
As we copy dependencies into lib/ to make scripts work in dev environments we also need to clean up that directory
- -> JENA-1173
<plugin>
<artifactId>maven-clean-plugin</artifactId>
<configuration>
<filesets>
<fileset>
<directory>lib/</directory>
</fileset>
</filesets>
</configuration>
</plugin>
-->
</plugins>
</build>

Expand Down

0 comments on commit 696ad22

Please sign in to comment.