415827 - jetty-start / update --help text for new command line options

+ Making help actually print out

jetty-start/src/main/java/org/eclipse/jetty/start/Main.java

@@ -419,7 +419,7 @@ public class Main
         // Show the usage information and return
         if (args.isHelp())
         {
-            usage();
+            usage(true);
         }
 
         // Show the version information and return
@@ -493,8 +493,6 @@ public class Main
             return;
         }
 
-        
-        
         // execute Jetty in another JVM
         if (args.isExec())
         {
@@ -609,7 +607,7 @@ public class Main
         }
     }
 
-    private void usage()
+    public void usage(boolean exit)
     {
         String usageResource = "org/eclipse/jetty/start/usage.txt";
         boolean usagePresented = false;
@@ -620,17 +618,29 @@ public class Main
                 try (InputStreamReader reader = new InputStreamReader(usageStream); BufferedReader buf = new BufferedReader(reader))
                 {
                     usagePresented = true;
+                    String line;
+                    while ((line = buf.readLine()) != null)
+                    {
+                        System.out.println(line);
                     }
                 }
             }
+            else
+            {
+                System.out.println("No usage.txt ??");
+            }
+        }
         catch (IOException e)
         {
-            StartLog.debug(e);
+            StartLog.warn(e);
         }
         if (!usagePresented)
         {
             System.err.println("ERROR: detailed usage resource unavailable");
         }
+        if (exit)
+        {
             System.exit(EXIT_USAGE);
         }
     }
+}

jetty-start/src/test/java/org/eclipse/jetty/start/MainTest.java

@@ -24,6 +24,7 @@ import java.util.List;
 
 import org.eclipse.jetty.toolchain.test.MavenTestingUtils;
 import org.junit.Assert;
+import org.junit.Ignore;
 import org.junit.Test;
 
 public class MainTest
@@ -81,6 +82,14 @@ public class MainTest
         main.listConfig(args);
     }
     
+    @Test
+    @Ignore("Just a bit noisy for general testing")
+    public void testHelp() throws Exception
+    {
+        Main main = new Main();
+        main.usage(false);
+    }
+
     @Test
     public void testWithCommandLine() throws Exception
     {