You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2009/04/22 16:35:09 UTC

svn commit: r767541 - /tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java

Author: hlship
Date: Wed Apr 22 14:35:08 2009
New Revision: 767541

URL: http://svn.apache.org/viewvc?rev=767541&view=rev
Log:
TAP5-130: Tapestry component report fails if the project's path name contains spaces

Modified:
    tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java

Modified: tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java?rev=767541&r1=767540&r2=767541&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java (original)
+++ tapestry/tapestry5/trunk/tapestry-component-report/src/main/java/org/apache/tapestry/mojo/ComponentReport.java Wed Apr 22 14:35:08 2009
@@ -57,6 +57,8 @@
 
     private static final Pattern TAPESTRY5_PATTERN = Pattern.compile("(org\\.apache\\.tapestry5[#_\\w\\.]*)");
 
+    private static final char QUOTE = '"';
+
     /**
      * Identifies the application root package.
      *
@@ -807,7 +809,7 @@
 
     private String toArgumentPath(List<String> paths)
     {
-        StringBuilder builder = new StringBuilder();
+        StringBuilder builder = new StringBuilder(5000).append(QUOTE);
 
         String sep = "";
 
@@ -819,7 +821,7 @@
             sep = SystemUtils.PATH_SEPARATOR;
         }
 
-        return builder.toString();
+        return builder.append(QUOTE).toString();
     }
 
     public Map<String, ClassDescription> readXML(String path) throws MavenReportException