commit bb800f81468b960e6d50d4709787596f2b2e3b5e
Author: Julien Ponge <julien@ponge.info>
Date:   Thu Dec 10 09:16:32 2009 +0100

    IZPACK-501: first attempt at resolving the issue.
    * The Unix users lookup is made lazily (previously in the constructor).
    * Navigation between panels is now made asynchronously in a separate thread which:
      - blocks the GUI on the EDT
      - performs the work on the created thread
      - releases the GUI on the EDT.

diff --git a/IzPack.ipr b/IzPack.ipr
index fba6519..70fcf09 100644
--- a/IzPack.ipr
+++ b/IzPack.ipr
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project relativePaths="true" version="4">
+<project version="4">
   <component name="AntConfiguration">
     <defaultAnt bundledAnt="true" />
     <buildFile url="file://$PROJECT_DIR$/src/build.xml">
@@ -18,13 +18,8 @@
     <option name="PROJECT_PROFILE" />
     <option name="USE_PROJECT_LEVEL_SETTINGS" value="false" />
   </component>
-  <component name="CodeStyleSettingsManager">
-    <option name="PER_PROJECT_SETTINGS" />
-    <option name="USE_PER_PROJECT_SETTINGS" value="false" />
-  </component>
   <component name="CompilerConfiguration">
     <option name="DEFAULT_COMPILER" value="Javac" />
-    <option name="DEPLOY_AFTER_MAKE" value="0" />
     <resourceExtensions>
       <entry name=".+\.(properties|xml|html|dtd|tld)" />
       <entry name=".+\.(gif|png|jpeg|jpg)" />
@@ -41,6 +36,7 @@
       <entry name="?*.tld" />
       <entry name="?*.xsl" />
     </wildcardResourcePatterns>
+    <annotationProcessing enabled="false" useClasspath="true" />
   </component>
   <component name="CopyrightManager" default="">
     <module2copyright />
@@ -75,18 +71,6 @@
     <option name="OUTPUT_DIRECTORY" />
   </component>
   <component name="IdProvider" IDEtalkID="F66F8E4B88443419F1E5C108B16FCE75" />
-  <component name="InspectionProjectProfileManager">
-    <option name="PROJECT_PROFILE" value="Project Default" />
-    <option name="USE_PROJECT_PROFILE" value="true" />
-    <version value="1.0" />
-    <profiles>
-      <profile version="1.0" is_locked="false">
-        <option name="myName" value="Project Default" />
-        <option name="myLocal" value="false" />
-      </profile>
-    </profiles>
-    <list size="0" />
-  </component>
   <component name="IssueNavigationConfiguration">
     <option name="links">
       <list>
@@ -97,13 +81,6 @@
       </list>
     </option>
   </component>
-  <component name="JavacSettings">
-    <option name="DEBUGGING_INFO" value="true" />
-    <option name="GENERATE_NO_WARNINGS" value="false" />
-    <option name="DEPRECATION" value="true" />
-    <option name="ADDITIONAL_OPTIONS_STRING" value="" />
-    <option name="MAXIMUM_HEAP_SIZE" value="128" />
-  </component>
   <component name="JavadocGenerationManager">
     <option name="OUTPUT_DIRECTORY" />
     <option name="OPTION_SCOPE" value="protected" />
@@ -270,7 +247,7 @@
       <module fileurl="file://$PROJECT_DIR$/IzPack.iml" filepath="$PROJECT_DIR$/IzPack.iml" />
     </modules>
   </component>
-  <component name="ProjectRootManager" version="2" languageLevel="JDK_1_5" assert-keyword="true" jdk-15="true" project-jdk-name="1.5" project-jdk-type="JavaSDK" />
+  <component name="ProjectRootManager" version="2" languageLevel="JDK_1_5" assert-keyword="true" jdk-15="true" project-jdk-name="1.6" project-jdk-type="JavaSDK" />
   <component name="ResourceManagerContainer">
     <option name="myResourceBundles">
       <value>
@@ -278,19 +255,26 @@
       </value>
     </option>
   </component>
-  <component name="RmicSettings">
-    <option name="IS_EANABLED" value="false" />
-    <option name="DEBUGGING_INFO" value="true" />
-    <option name="GENERATE_NO_WARNINGS" value="false" />
-    <option name="GENERATE_IIOP_STUBS" value="false" />
-    <option name="ADDITIONAL_OPTIONS_STRING" value="" />
-  </component>
   <component name="SvnBranchConfigurationManager">
     <option name="myConfigurationMap">
       <map>
         <entry key="$PROJECT_DIR$">
           <value>
             <SvnBranchConfiguration>
+              <option name="branchMap">
+                <map>
+                  <entry key="https://svn.codehaus.org/izpack/izpack-src/branches">
+                    <value>
+                      <list />
+                    </value>
+                  </entry>
+                  <entry key="https://svn.codehaus.org/izpack/izpack-src/tags">
+                    <value>
+                      <list />
+                    </value>
+                  </entry>
+                </map>
+              </option>
               <option name="branchUrls">
                 <list>
                   <option value="https://svn.codehaus.org/izpack/izpack-src/branches" />
diff --git a/src/lib/com/izforge/izpack/installer/InstallerFrame.java b/src/lib/com/izforge/izpack/installer/InstallerFrame.java
index dbea42d..79c40bc 100644
--- a/src/lib/com/izforge/izpack/installer/InstallerFrame.java
+++ b/src/lib/com/izforge/izpack/installer/InstallerFrame.java
@@ -135,7 +135,7 @@ public class InstallerFrame extends JFrame {
      * Registered GUICreationListener.
      */
     protected ArrayList<GUIListener> guiListener;
-       
+
     /**
      * Heading major text.
      */
@@ -211,13 +211,13 @@ public class InstallerFrame extends JFrame {
         // Builds the GUI
         loadIcons();
         loadCustomIcons();
-        loadPanels();               
+        loadPanels();
         buildGUI();
 
         // We show the frame
         showFrame();
         switchPanel(0);
-    }   
+    }
 
     public Debugger getDebugger() {
         return this.debugger;
@@ -283,7 +283,7 @@ public class InstallerFrame extends JFrame {
             }
 
             panel.setHelps(p.getHelpsMap());
-            
+
             List<String> preActivateActions = p.getPreActivationActions();
             if (preActivateActions != null)
             {
@@ -510,9 +510,9 @@ public class InstallerFrame extends JFrame {
         debugger = new Debugger(installdata, icons, rules);
         // this needed to fully initialize the debugger.
         JPanel debugpanel = debugger.getDebugPanel();
-        
+
         // create a debug panel if TRACE is enabled
-        if (Debug.isTRACE()) {                       
+        if (Debug.isTRACE()) {
             if (installdata.guiPrefs.modifier.containsKey("showDebugWindow")
                     && Boolean.valueOf(installdata.guiPrefs.modifier.get("showDebugWindow"))) {
                 JFrame debugframe = new JFrame("Debug information");
@@ -731,7 +731,7 @@ public class InstallerFrame extends JFrame {
             IzPanel panel = installdata.panels.get(installdata.curPanelNumber);
             IzPanel l_panel = installdata.panels.get(last);
             showHelpButton(panel.canShowHelp());
-            if (Debug.isTRACE()) {                
+            if (Debug.isTRACE()) {
                 debugger.switchPanel(panel.getMetadata(), l_panel.getMetadata());
             }
             Log.getInstance().addDebugMessage(
@@ -1433,7 +1433,7 @@ public class InstallerFrame extends JFrame {
             panel.executePreValidationActions();
             boolean isValid = doValidation ? panel.panelValidated() : true;
             panel.executePostValidationActions();
-            
+
             // check if we can display the next panel or if there was an error during actions that
             // disables the next button
             if (!nextButton.isEnabled()) { return; }
@@ -1548,12 +1548,33 @@ public class InstallerFrame extends JFrame {
      */
     class NavigationHandler implements ActionListener {
 
-        /**
-         * Actions handler.
-         *
-         * @param e The event.
-         */
-        public void actionPerformed(ActionEvent e) {
+        public void actionPerformed(final ActionEvent e) {
+            /*
+                Some panels activation may be slow, hence we
+                block the GUI, spin a thread to handle navigation then
+                release the GUI.
+             */
+            new Thread(new Runnable() {
+                public void run() {
+
+                    SwingUtilities.invokeLater(new Runnable() {
+                        public void run() {
+                            blockGUI();
+                        }
+                    });
+
+                    navigate(e);
+
+                    SwingUtilities.invokeLater(new Runnable() {
+                        public void run() {
+                            releaseGUI();
+                        }
+                    });
+                }
+            }).start();
+        }
+
+        private void navigate(ActionEvent e) {
             Object source = e.getSource();
             if (source == prevButton) {
                 navigatePrevious();
@@ -1562,7 +1583,6 @@ public class InstallerFrame extends JFrame {
             } else if (source == quitButton) {
                 exit();
             }
-
         }
     }
 
diff --git a/src/lib/com/izforge/izpack/util/os/Unix_Shortcut.java b/src/lib/com/izforge/izpack/util/os/Unix_Shortcut.java
index b510df9..1778c3c 100644
--- a/src/lib/com/izforge/izpack/util/os/Unix_Shortcut.java
+++ b/src/lib/com/izforge/izpack/util/os/Unix_Shortcut.java
@@ -51,7 +51,6 @@
 package com.izforge.izpack.util.os;
 
 import com.izforge.izpack.installer.AutomatedInstallData;
-import com.izforge.izpack.installer.InstallData;
 import com.izforge.izpack.installer.ResourceManager;
 import com.izforge.izpack.installer.ResourceNotFoundException;
 import com.izforge.izpack.util.Debug;
@@ -128,7 +127,7 @@ public class Unix_Shortcut extends Shortcut implements Unix_ShortcutConstants
 
     private static ShellScript uninstallScript = null;
 
-    private static ArrayList users = UnixUsers.getUsersWithValidShellsExistingHomesAndDesktops();
+    private List users;
 
     // private static ArrayList tempfiles = new ArrayList();
 
@@ -709,7 +708,7 @@ public class Unix_Shortcut extends Shortcut implements Unix_ShortcutConstants
      */
     private void installDesktopFileToAllUsersDesktop(File writtenDesktopFile)
     {
-        for (Object user1 : users)
+        for (Object user1 : getUsers())
         {
             UnixUser user = ((UnixUser) user1);
 
@@ -768,7 +767,7 @@ public class Unix_Shortcut extends Shortcut implements Unix_ShortcutConstants
         // su marc.eppelmann -c "/bin/cp /home/marc.eppelmann/backup.job.out.txt
         // /home/marc.eppelmann/backup.job.out2.txt"
 
-        for (Object user1 : users)
+        for (Object user1 : getUsers())
         {
             UnixUser user = ((UnixUser) user1);
 
@@ -983,7 +982,6 @@ public class Unix_Shortcut extends Shortcut implements Unix_ShortcutConstants
      * @param targetPath The Path in which the files should be written.
      * @param shortcutName The Name for the File
      * @param shortcutDef The Shortcut FileContent
-     * @param replaceSpaces Replaces Spaces in the Filename if true was given
      * @return The written File
      */
     private File writeAppShortcutWithSimpleSpacehandling(String targetPath, String shortcutName,
@@ -1428,4 +1426,11 @@ public class Unix_Shortcut extends Shortcut implements Unix_ShortcutConstants
         return ShortcutType;
         // return Shortcut.DESKTOP;
     }
+
+    private List getUsers() {
+        if (users == null) {
+            users = UnixUsers.getUsersWithValidShellsExistingHomesAndDesktops();    
+        }
+        return users;
+    }
 }