Skip to content

Commit

Permalink
do not remove maven installation which does not have a home, like mav…
Browse files Browse the repository at this point in the history
…en installation with automatic installer (bug associated with JENKINS-14510 fix ... sorry..)
  • Loading branch information
mpailloncy committed Oct 1, 2012
1 parent 14098be commit e6685a7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/main/java/hudson/tasks/Maven.java
Expand Up @@ -353,7 +353,7 @@ public void setInstallations(MavenInstallation... installations) {
if(installations != null) {
Collections.addAll(tmpList, installations);
for(MavenInstallation installation : installations) {
if(Util.fixEmptyAndTrim(installation.getName()) == null || Util.fixEmptyAndTrim(installation.getHome()) == null) {
if(Util.fixEmptyAndTrim(installation.getName()) == null) {
tmpList.remove(installation);
}
}
Expand Down

0 comments on commit e6685a7

Please sign in to comment.