You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by jd...@apache.org on 2005/05/06 21:17:12 UTC

svn commit: r168639 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java

Author: jdcasey
Date: Fri May  6 12:17:11 2005
New Revision: 168639

URL: http://svn.apache.org/viewcvs?rev=168639&view=rev
Log:
Correcting module-path validation to error on '..' rather than '/'...'/' causes problems when modules are not direct subdirs of the current basedir.

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java?rev=168639&r1=168638&r2=168639&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/DefaultMaven.java Fri May  6 12:17:11 2005
@@ -188,7 +188,7 @@
                 for ( Iterator i = project.getModules().iterator(); i.hasNext(); )
                 {
                     String name = (String) i.next();
-                    if ( name.indexOf( '/' ) >= 0 )
+                    if ( name.indexOf( ".." ) >= 0 )
                     {
                         String message = "Illegal module name: " + name + " (cannot contain path characters)";
                         throw new ReactorException( message );



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org