rfscholte commented on a change in pull request #24:
URL: https://github.com/apache/maven-ear-plugin/pull/24#discussion_r523746626
##########
File path: src/site/apt/modules.apt.vm
##########
@@ -116,6 +116,30 @@ EAR Modules
* <<moduleId>> - sets the id of the module in the generated application.xml.
+ * <<libDirectory>> - sets archive directory which contains Java libraries
(JARs)
+ packaged into archive. Default is <<<null>>> meaning that archive doesn't
+ contain packaged libraries.
+ If libraries are located at the root of archive then use single slash ('/')
+ to configure that.
+ If <<<libDirectory>>> is <<<null>>> then <<<Class-Path>>> setting of module
+ <<<MANIFEST.MF>>> is modified to reference libraries which are packaged in
EAR.
+ Modification here means:
+
+ * If <<<MANIFEST.MF>>> has no <<<Class-Path>>> setting then it is added.
+
+ * Existing elements of <<<Class-Path>>> setting which match libraries
packaged
+ in EAR are replaced with references to the matching libraries in EAR.
+
+ * Other libraries packaged in EAR are appended to <<<Class-Path>>> setting
in order
+ of EAR dependency on library if
+ {{{./ear-mojo.html#skipClassPathModification}skipClassPathModification}}
parameter
+ is false.
+
+ If <<<libDirectory>>> is not <<<null>>> and if
Review comment:
This makes we wonder if the explicit skinnyModules parameter is
required. Setting the libDirectory seems enough to understand that artifacts
from the modules are moved here.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
