[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13529831#comment-13529831
 ] 

Hudson commented on ZOOKEEPER-721:
----------------------------------

Integrated in ZooKeeper-trunk #1770 (See 
[https://builds.apache.org/job/ZooKeeper-trunk/1770/])
    ZOOKEEPER-721. Minor cleanup related to the log4j version change from 
1.2.15 -> 1.2.16 (Sean Busbey via phunt) (Revision 1420551)

     Result = SUCCESS
phunt : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1420551
Files : 
* /zookeeper/trunk/CHANGES.txt
* /zookeeper/trunk/build.xml
* /zookeeper/trunk/src/java/lib/log4j-1.2.15.LICENSE.txt
* /zookeeper/trunk/src/java/lib/log4j-1.2.16.LICENSE.txt

                
> Minor cleanup related to the log4j version change from 1.2.15 -> 1.2.16
> -----------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-721
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-721
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 3.3.0
>            Reporter: Paolo Castagna
>            Assignee: Sean Busbey
>            Priority: Minor
>              Labels: ivy, maven
>             Fix For: 3.5.0
>
>         Attachments: ZOOKEEPER-721.patch
>
>
> The generated Maven pom.xml file contains:
> {code}
> <dependency>
>   <groupId>log4j</groupId>
>   <artifactId>log4j</artifactId>
>   <version>1.2.15</version>
>   <scope>compile</scope>
> </dependency>
> {code}
> It would be better to have:
> {code}
> <dependency>
>   <groupId>log4j</groupId>
>   <artifactId>log4j</artifactId>
>   <version>1.2.15</version>
>   <scope>compile</scope>
>   <exclusions>
>     <exclusion>
>       <groupId>javax.jms</groupId>
>       <artifactId>jms</artifactId>
>     </exclusion>
>     <exclusion>
>       <groupId>com.sun.jdmk</groupId>
>       <artifactId>jmxtools</artifactId>
>     </exclusion>
>     <exclusion>
>       <groupId>com.sun.jmx</groupId>
>       <artifactId>jmxri</artifactId>
>     </exclusion>
>     <exclusion>
>       <groupId>javax.mail</groupId>
>       <artifactId>mail</artifactId>
>     </exclusion>
>   </exclusions> 
> </dependency>
> {code}
> This would avoid to people depending on ZooKeeper artifacts the pain to 
> exclude such "broken" dependencies by themselves.
> A similar issue is discussed in HADOOP-6629, although I am not clear on how 
> to solve fix it using Ivy.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to