You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by GitBox <gi...@apache.org> on 2022/09/28 12:57:55 UTC
[GitHub] [jena-site] sszuev opened a new pull request, #123: update jena-repack.md: add a bit more information (gradle), cleanup
sszuev opened a new pull request, #123:
URL: https://github.com/apache/jena-site/pull/123
minor changes in jena-repack.md
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jena.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jena-site] afs merged pull request #123: update jena-repack.md: add a bit more information (gradle), cleanup
Posted by GitBox <gi...@apache.org>.
afs merged PR #123:
URL: https://github.com/apache/jena-site/pull/123
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jena.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jena-site] sszuev commented on a diff in pull request #123: update jena-repack.md: add a bit more information (gradle), cleanup
Posted by GitBox <gi...@apache.org>.
sszuev commented on code in PR #123:
URL: https://github.com/apache/jena-site/pull/123#discussion_r983144693
##########
source/documentation/notes/jena-repack.md:
##########
@@ -27,32 +28,42 @@ for Fuseki. It uses the maven shade plugin with a `transformer`.
This is an extract from the POM:
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-shade-plugin</artifactId>
- <configuration>
- ...
- <transformers>
- <transformer
- implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
- ... other transformers ...
- </transformers>
-
+```xml
+<plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-shade-plugin</artifactId>
+ <configuration>
+ ...
+ <transformers>
+ <transformer
+ implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
+ ... other transformers ...
+ </transformers>
+ </configuration>
+</plugin>
+```
See
[jena-fuseki2/jena-fuseki-server/pom.xml](https://github.com/apache/jena/blob/main/jena-fuseki2/jena-fuseki-server/pom.xml)
for the complete shade plugin setup used by Fuseki.
-If doing manually, create a single file in your application jar the
-all lines of all the services resource files. The order does not matter
-- Jena calls modules in the right order.
-
-For Gradle, the [shadowJar plugin](https://imperceptiblethoughts.com/shadow/)
+#### Gradle
+For Gradle, the [shadowJar plugin](https://plugins.gradle.org/plugin/com.github.johnrengelman.shadow)
has the
[mergeServiceFiles](https://imperceptiblethoughts.com/shadow/configuration/merging/#merging-service-descriptor-files)
operation.
- // Merging Service Files
- shadowJar {
- mergeServiceFiles()
- . . .
- }
+```groovy
+plugins {
+ ...
+ id "com.github.johnrengelman.shadow" version "7.1.2"
+}
+shadowJar {
+ mergeServiceFiles()
+}
+...
+```
+
+#### Manual assembling
+If doing manually, create a single file (`META-INF/services/org.apache.jena.sys.JenaSubsystemLifecycle`) in your application jar containing the
+all lines of all the services resource files. The order does not matter.
Review Comment:
fixed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jena.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [jena-site] kinow commented on a diff in pull request #123: update jena-repack.md: add a bit more information (gradle), cleanup
Posted by GitBox <gi...@apache.org>.
kinow commented on code in PR #123:
URL: https://github.com/apache/jena-site/pull/123#discussion_r982854792
##########
source/documentation/notes/jena-repack.md:
##########
@@ -27,32 +28,42 @@ for Fuseki. It uses the maven shade plugin with a `transformer`.
This is an extract from the POM:
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-shade-plugin</artifactId>
- <configuration>
- ...
- <transformers>
- <transformer
- implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
- ... other transformers ...
- </transformers>
-
+```xml
+<plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-shade-plugin</artifactId>
+ <configuration>
+ ...
+ <transformers>
+ <transformer
+ implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer"/>
+ ... other transformers ...
+ </transformers>
+ </configuration>
+</plugin>
+```
See
[jena-fuseki2/jena-fuseki-server/pom.xml](https://github.com/apache/jena/blob/main/jena-fuseki2/jena-fuseki-server/pom.xml)
for the complete shade plugin setup used by Fuseki.
-If doing manually, create a single file in your application jar the
-all lines of all the services resource files. The order does not matter
-- Jena calls modules in the right order.
-
-For Gradle, the [shadowJar plugin](https://imperceptiblethoughts.com/shadow/)
+#### Gradle
+For Gradle, the [shadowJar plugin](https://plugins.gradle.org/plugin/com.github.johnrengelman.shadow)
has the
[mergeServiceFiles](https://imperceptiblethoughts.com/shadow/configuration/merging/#merging-service-descriptor-files)
operation.
- // Merging Service Files
- shadowJar {
- mergeServiceFiles()
- . . .
- }
+```groovy
+plugins {
+ ...
+ id "com.github.johnrengelman.shadow" version "7.1.2"
+}
+shadowJar {
+ mergeServiceFiles()
+}
+...
+```
+
+#### Manual assembling
+If doing manually, create a single file (`META-INF/services/org.apache.jena.sys.JenaSubsystemLifecycle`) in your application jar containing the
+all lines of all the services resource files. The order does not matter.
Review Comment:
s/containing the all lines/containing the lines ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@jena.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org