You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Tom Palmer (JIRA)" <ji...@codehaus.org> on 2008/12/01 17:45:19 UTC

[jira] Created: (MDEP-191) Tidy up logging from unpack

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

        

[jira] Closed: (MDEP-191) Tidy up logging from unpack

Posted by "Dan Tran (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MDEP-191?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dan Tran closed MDEP-191.
-------------------------

    Resolution: Fixed
      Assignee: Dan Tran  (was: Brian Fox)

fixed at

Revision: 894772
Author: dantran
Date: 7:01:10 PM, Wednesday, December 30, 2009
Message:
MDEP-191: Tidy up logging from unpack, submitted by Tom Palmer
----
Modified : /maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/AbstractDependencyMojo.java



> 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: Dan Tran
>            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