Fix #253 Add licenseMergesUrl also in report goals

There is still something missing in this fix #253. It should be included in next release because licenseMergeUrl in not reliable if there is more then one raw in content of file linked by licenseMergeUrl
mojohaus · Feb 25, 2019 · b7439a9 · b7439a9
1 parent 9e343a3
commit b7439a9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/it/aggregate-third-party-report-merge-licences/license.merges b/src/it/aggregate-third-party-report-merge-licences/license.merges
@@ -1 +1,2 @@
-APACHE|The Apache Software License, Version 2.0|Apache License, Version 2.0|Apache Public License 2.0
+MIT - MIT License|The MIT License|MIT License
+APACHE|The Apache Software License, Version 2.0|Apache License, Version 2.0|Apache Public License 2.0
diff --git a/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java b/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java
@@ -392,7 +392,7 @@ protected void init()
             getLog().warn( "" );
             if ( UrlRequester.isStringUrl( licenseMergesUrl ) )
             {
-                licenseMerges = Arrays.asList( UrlRequester.getFromUrl( licenseMergesUrl ) );
+                licenseMerges = Arrays.asList( UrlRequester.getFromUrl( licenseMergesUrl ).split( "\n" ) );
             }
         }
     }