Commits

Min'an Tan committed f546405 Merge

Merged in BUILDENG-3328 (pull request #11)

BUILDENG-3328: not failing if the file doesn't have --

Comments (0)

Files changed (3)

Add a comment to this file

src/it/license-files/missing-licenses-files/licenses/filewithoutseparator.txt

Empty file added.

src/it/license-files/missing-licenses-files/postbuild.groovy

 assert content.contains(" - jquery-hashchange--1.4.txt")
 assert content.contains("Consider updating jquery-hashchange--1.3.txt")
 
+assert content.contains("Some license files are missing")
+
 return true;
 

src/main/java/org/codehaus/mojo/license/LicenseFilesCheckMojo.java

 
     private String retrievePrefixFromFileName(final String fileName)
     {
-        return fileName.substring(0, fileName.lastIndexOf(SEPARATOR));
+        final int endIndex = fileName.lastIndexOf(SEPARATOR);
+        return endIndex > 0? fileName.substring(0, endIndex) : fileName;
     }
 
     //The rules for the file names can be found here: https://extranet.atlassian.com/x/S52Keg
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.