Tidy up logging from unpack
---------------------------

                 Key: MDEP-191
                 URL: http://jira.codehaus.org/browse/MDEP-191
             Project: Maven 2.x Dependency Plugin
          Issue Type: Improvement
          Components: unpack
    Affects Versions: 2.0
            Reporter: Tom Palmer
            Assignee: Brian Fox
            Priority: Trivial
         Attachments: tidy-unpack-logging.patch

The attached patch changes the logging output from unpacking artifacts by 
logging on a single line to make it more consistent with other logging messages 
(e.g. installing an artifact). It also logs the includes/excludes only when 
they are not null. For example the output (taken from the test run of the 
plugin) changes from:

{noformat}
[INFO] Unpacking 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/localTestRepo/test-1.0.jar
 to
  
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/outputDirectory
   with includes **/test2.txt and excludes:null
[INFO] Unpacking 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/localTestRepo/test-1.0.jar
 to
  
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/outputDirectory
   with includes null and excludes:**/*
{noformat}

to:

{noformat}
[INFO] Unpacking 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/localTestRepo/test-1.0.jar
 to 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/outputDirectory
 with includes **/test2.txt
[INFO] Unpacking 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/localTestRepo/test-1.0.jar
 to 
/Users/tom/Development/Projects/maven/plugins/maven-dependency-plugin/target/unit-tests/unpack/outputDirectory
 with excludes **/*
{noformat}


-- 
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

        

Reply via email to