FitRunnerMojo.ensureDirectoryExists() method should use File.mkdirs() instead 
of File.mkdir()
---------------------------------------------------------------------------------------------

                 Key: MFIT-7
                 URL: http://jira.codehaus.org/browse/MFIT-7
             Project: Maven 2.x FIT Plugin
          Issue Type: Bug
    Affects Versions: 2.0-beta-2
         Environment: WinXP Pro SP2, Maven 2.0.4, JDK 1.5.0_06
            Reporter: John Mufarrige
            Priority: Trivial
             Fix For: 2.0-beta-2
         Attachments: FitRunnerMojo_patch.java

If the specified outputDirectory does not exist, the ensureDirectoryExists() 
method will attempt to create it using the mkdir() method of java.io.File.  
However this only works if the outputDirectory references a single directory 
level below what already exists.  Say for example you are using outputDirectory 
= c:/temp/target/test-classes/fit and currently only c:/temp exists.  The 
ensureDirectoryExists() method will fail with a wrapped IOException because 
mkdir() can only create a single directory level.  In other words, if 
outputDirectory = c:/temp/target/test-classes/fit and currently the 
c:/temp/target/test-classes directory exists then mkdir() will create the fit 
directory for you.  However if this were changed to mkdirs() instead of 
mkdir(), it would create the entire directory structure instead of just one 
level, so if ensureDirectoryExists() were modified to call mkdirs(), and the 
plugin were called with outputDirectory = c:/temp/target/test-classes/fit and 
currently only c:/temp exists, then ensureDirectoryExists() would correctly 
create the c:/temp/target/test-classes/fit directory structure.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira



---------------------------------------------------------------------
To unsubscribe from this list please visit:

    http://xircles.codehaus.org/manage_email

Reply via email to