[FIXED JENKINS-15043] 'status' field in perforce job descriptions is …

…now optional
jenkinsci · Sep 5, 2012 · 0e5b7c4 · 0e5b7c4
1 parent bc16080
commit 0e5b7c4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/com/tek42/perforce/parse/ChangelistBuilder.java b/src/main/java/com/tek42/perforce/parse/ChangelistBuilder.java
@@ -120,6 +120,8 @@ public Changelist build(StringBuilder sb) throws PerforceException {
 							// EXT-84 on 2007/09/25 *closed*
 							// or
 							// EXT-84 on 2007/09/25
+                                                        // or 
+                                                        // EXT-84 on 2007/09/25 by mwille
 							StringTokenizer details = new StringTokenizer(line);
 							job = new Changelist.JobEntry();
 							job.setJob(details.nextToken());
@@ -133,7 +135,8 @@ public Changelist build(StringBuilder sb) throws PerforceException {
 								if ("by".equals(possibleUser))
 								{
 									details.nextToken(); // user
-									status = details.nextToken(); // status
+                                                                        if (details.hasMoreTokens()) //status is optional
+                                                                            status = details.nextToken(); // status
 								}
 								else
 								{