Fixes issue #194 - 'Waiting for new REPL process ack' stays in the st…

…atus bar
ccw-ide · Dec 10, 2010 · b189c50 · b189c50
1 parent 46c4c40
commit b189c50
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/ccw.core/src/ccw/launching/ClojureLaunchDelegate.java b/ccw.core/src/ccw/launching/ClojureLaunchDelegate.java
@@ -35,6 +35,8 @@
 import org.eclipse.debug.core.ILaunchConfiguration;
 import org.eclipse.jdt.launching.IJavaLaunchConfigurationConstants;
 import org.eclipse.jdt.launching.JavaLaunchDelegate;
+import org.eclipse.jface.dialogs.ErrorDialog;
+import org.eclipse.ui.PlatformUI;
 import org.eclipse.ui.WorkbenchException;
 import org.eclipse.ui.console.ConsolePlugin;
 import org.eclipse.ui.console.IConsole;
@@ -112,12 +114,13 @@ public void done() {
 	                    				}
 	                    			});
                     			} catch (CoreException e) {
-                    				CCWPlugin.logError("unexpected exception during project refresh for auto-load on startup", e);
+                    				final String MSG = "unexpected exception during project refresh for auto-load on startup";
+                    				ErrorDialog.openError(PlatformUI.getWorkbench().getActiveWorkbenchWindow().getShell(),
+                    						"REPL Connexion failure", MSG, e.getStatus());
 	                    		}
 	                    	} else {
 	                    		connectRepl();
 	                    	}
-	                    	monitor.done();
 		                }
 		                private IProject getProject() {
 		            		try {
@@ -135,6 +138,7 @@ private void connectRepl() {
 		                    }
 		                }
 		            });
+		            monitor.done();
 		            return Status.OK_STATUS;
 				}
             };