You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@synapse.apache.org by ve...@apache.org on 2009/04/28 21:56:20 UTC
svn commit: r769518 - in /synapse/trunk/java: modules/transports/pom.xml
pom.xml
Author: veithen
Date: Tue Apr 28 19:56:19 2009
New Revision: 769518
URL: http://svn.apache.org/viewvc?rev=769518&view=rev
Log:
Downgraded log4j from 1.2.15 (latest) to 1.2.14.
Reason: 1.2.15 introduces additional transitive dependencies, some of which are not resolvable. For the build this can easily be addressed using an exclusion rule in the dependencyManagement entry for log4j. However, as Ruwan reported, this doesn't work with the IDEA Maven plugin, and I still see issues with the Eclipse plugin.
Note that log4j was upgraded to the latest version by Asankha in r748973, along with other dependencies. Since there is probably no particular reason to prefer 1.2.15 over previous versions, we should use 1.2.14 so that we can avoid cluttering the POM files with exclusions rules.
Modified:
synapse/trunk/java/modules/transports/pom.xml
synapse/trunk/java/pom.xml
Modified: synapse/trunk/java/modules/transports/pom.xml
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
==============================================================================
--- synapse/trunk/java/modules/transports/pom.xml (original)
+++ synapse/trunk/java/modules/transports/pom.xml Tue Apr 28 19:56:19 2009
@@ -185,12 +185,6 @@
<artifactId>axis2-transport-testkit</artifactId>
<version>${axis2.transport.version}</version>
<scope>test</scope>
- <exclusions>
- <exclusion>
- <groupId>com.sun.jmx</groupId>
- <artifactId>jmxri</artifactId>
- </exclusion>
- </exclusions>
</dependency>
<!-- HTTP transport dependencies -->
Modified: synapse/trunk/java/pom.xml
URL: http://svn.apache.org/viewvc/synapse/trunk/java/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
==============================================================================
--- synapse/trunk/java/pom.xml (original)
+++ synapse/trunk/java/pom.xml Tue Apr 28 19:56:19 2009
@@ -448,32 +448,6 @@
<groupId>log4j</groupId>
<artifactId>log4j</artifactId>
<version>${log4j.version}</version>
- <exclusions>
- <exclusion>
- <groupId>javax.mail</groupId>
- <artifactId>mail</artifactId>
- </exclusion>
- <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>oro</groupId>
- <artifactId>oro</artifactId>
- </exclusion>
- <exclusion>
- <groupId>junit</groupId>
- <artifactId>junit</artifactId>
- </exclusion>
- </exclusions>
</dependency>
<dependency>
@@ -700,8 +674,8 @@
<type>jar</type>
<exclusions>
<exclusion>
- <groupId>com.sun.jmx</groupId>
- <artifactId>jmxri</artifactId>
+ <groupId>log4j</groupId>
+ <artifactId>log4j</artifactId>
</exclusion>
</exclusions>
</dependency>
@@ -867,7 +841,9 @@
<JLex.version>0.0</JLex.version>
<!-- misc -->
<junit.version>3.8.2</junit.version>
- <log4j.version>1.2.15</log4j.version>
+ <!-- Warning: log4j versions above 1.2.14 introduce additional transitive dependencies,
+ some of which are not available from the Maven central repository. -->
+ <log4j.version>1.2.14</log4j.version>
<xmlunit.version>1.1</xmlunit.version>
<derby.version>10.4.2.0</derby.version>
<wrapper.version>3.2.3</wrapper.version>
Re: svn commit: r769518 - in /synapse/trunk/java:
modules/transports/pom.xml pom.xml
Posted by Ruwan Linton <ru...@gmail.com>.
Hi Asankha,
Yeah, I assume this is due to a change in the axis2. I will try to find the
exact reason for this. At the same time ValidateMediatorTest seems to be
failing because the commons-lang dependency is removed. I think we need to
keep the commons-lang dependency on the test scope.
Thanks,
Ruwan
On Fri, May 1, 2009 at 11:53 AM, Asankha C. Perera <as...@apache.org>wrote:
> Hi Ruwan
>
>> I am not clear why... but there with this version of log4j it seems like
>> the console log is completely out of format and also the shutdown hook log
>> is not captured. This could be due to some other issue but when I run the
>> build with the 1.2.15 dependency I cannot see this issue.
>>
>> Anyone else seeing this or is this just local to me?
>>
> Yes, I also saw a change in console logging.. also when running a smoke
> test with Sample #150, the stockquote client never seems to quit.. still not
> sure why, did you notice that?
>
> cheers
> asankha
>
> --
> Asankha C. Perera
> AdroitLogic, http://adroitlogic.org
>
> http://esbmagic.blogspot.com
>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
> For additional commands, e-mail: dev-help@synapse.apache.org
>
>
--
Ruwan Linton
Senior Software Engineer & Product Manager; WSO2 ESB; http://wso2.org/esb
WSO2 Inc.; http://wso2.org
email: ruwan@wso2.com; cell: +94 77 341 3097
blog: http://ruwansblog.blogspot.com
Re: svn commit: r769518 - in /synapse/trunk/java: modules/transports/pom.xml
pom.xml
Posted by "Asankha C. Perera" <as...@apache.org>.
Hi Ruwan
> I am not clear why... but there with this version of log4j it seems
> like the console log is completely out of format and also the shutdown
> hook log is not captured. This could be due to some other issue but
> when I run the build with the 1.2.15 dependency I cannot see this issue.
>
> Anyone else seeing this or is this just local to me?
Yes, I also saw a change in console logging.. also when running a smoke
test with Sample #150, the stockquote client never seems to quit.. still
not sure why, did you notice that?
cheers
asankha
--
Asankha C. Perera
AdroitLogic, http://adroitlogic.org
http://esbmagic.blogspot.com
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org
Re: svn commit: r769518 - in /synapse/trunk/java:
modules/transports/pom.xml pom.xml
Posted by Ruwan Linton <ru...@gmail.com>.
Yes this fixes the logging issue and your reasoning is indeed correct :-)
Thanks,
Ruwan
On Fri, May 1, 2009 at 5:07 PM, Andreas Veithen
<an...@gmail.com>wrote:
> What happened is that log4j disappeared from the distribution because
> it is no longer a transitive dependency. I added it as an explicit
> dependency of the distribution and this solves the problem for the log
> format. This makes sense because the individual JARs we include in the
> distribution should only depend on commons-logging, but not on any
> particular logging implementation. It is only when we assemble the
> distribution that we decide which logging framework we want to use.
>
> Andreas
>
> On Fri, May 1, 2009 at 07:08, Ruwan Linton <ru...@gmail.com> wrote:
> > Andreas,
> >
> > I am not clear why... but there with this version of log4j it seems like
> the
> > console log is completely out of format and also the shutdown hook log is
> > not captured. This could be due to some other issue but when I run the
> build
> > with the 1.2.15 dependency I cannot see this issue.
> >
> > Anyone else seeing this or is this just local to me?
> >
> > Thanks,
> > Ruwan
> >
> > On Wed, Apr 29, 2009 at 1:26 AM, <ve...@apache.org> wrote:
> >>
> >> Author: veithen
> >> Date: Tue Apr 28 19:56:19 2009
> >> New Revision: 769518
> >>
> >> URL: http://svn.apache.org/viewvc?rev=769518&view=rev
> >> Log:
> >> Downgraded log4j from 1.2.15 (latest) to 1.2.14.
> >>
> >> Reason: 1.2.15 introduces additional transitive dependencies, some of
> >> which are not resolvable. For the build this can easily be addressed
> using
> >> an exclusion rule in the dependencyManagement entry for log4j. However,
> as
> >> Ruwan reported, this doesn't work with the IDEA Maven plugin, and I
> still
> >> see issues with the Eclipse plugin.
> >>
> >> Note that log4j was upgraded to the latest version by Asankha in
> r748973,
> >> along with other dependencies. Since there is probably no particular
> reason
> >> to prefer 1.2.15 over previous versions, we should use 1.2.14 so that we
> can
> >> avoid cluttering the POM files with exclusions rules.
> >>
> >> Modified:
> >> synapse/trunk/java/modules/transports/pom.xml
> >> synapse/trunk/java/pom.xml
> >>
> >> Modified: synapse/trunk/java/modules/transports/pom.xml
> >> URL:
> >>
> http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
> >>
> >>
> ==============================================================================
> >> --- synapse/trunk/java/modules/transports/pom.xml (original)
> >> +++ synapse/trunk/java/modules/transports/pom.xml Tue Apr 28 19:56:19
> 2009
> >> @@ -185,12 +185,6 @@
> >> <artifactId>axis2-transport-testkit</artifactId>
> >> <version>${axis2.transport.version}</version>
> >> <scope>test</scope>
> >> - <exclusions>
> >> - <exclusion>
> >> - <groupId>com.sun.jmx</groupId>
> >> - <artifactId>jmxri</artifactId>
> >> - </exclusion>
> >> - </exclusions>
> >> </dependency>
> >>
> >> <!-- HTTP transport dependencies -->
> >>
> >> Modified: synapse/trunk/java/pom.xml
> >> URL:
> >>
> http://svn.apache.org/viewvc/synapse/trunk/java/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
> >>
> >>
> ==============================================================================
> >> --- synapse/trunk/java/pom.xml (original)
> >> +++ synapse/trunk/java/pom.xml Tue Apr 28 19:56:19 2009
> >> @@ -448,32 +448,6 @@
> >> <groupId>log4j</groupId>
> >> <artifactId>log4j</artifactId>
> >> <version>${log4j.version}</version>
> >> - <exclusions>
> >> - <exclusion>
> >> - <groupId>javax.mail</groupId>
> >> - <artifactId>mail</artifactId>
> >> - </exclusion>
> >> - <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>oro</groupId>
> >> - <artifactId>oro</artifactId>
> >> - </exclusion>
> >> - <exclusion>
> >> - <groupId>junit</groupId>
> >> - <artifactId>junit</artifactId>
> >> - </exclusion>
> >> - </exclusions>
> >> </dependency>
> >>
> >> <dependency>
> >> @@ -700,8 +674,8 @@
> >> <type>jar</type>
> >> <exclusions>
> >> <exclusion>
> >> - <groupId>com.sun.jmx</groupId>
> >> - <artifactId>jmxri</artifactId>
> >> + <groupId>log4j</groupId>
> >> + <artifactId>log4j</artifactId>
> >> </exclusion>
> >> </exclusions>
> >> </dependency>
> >> @@ -867,7 +841,9 @@
> >> <JLex.version>0.0</JLex.version>
> >> <!-- misc -->
> >> <junit.version>3.8.2</junit.version>
> >> - <log4j.version>1.2.15</log4j.version>
> >> + <!-- Warning: log4j versions above 1.2.14 introduce additional
> >> transitive dependencies,
> >> + some of which are not available from the Maven central
> >> repository. -->
> >> + <log4j.version>1.2.14</log4j.version>
> >> <xmlunit.version>1.1</xmlunit.version>
> >> <derby.version>10.4.2.0</derby.version>
> >> <wrapper.version>3.2.3</wrapper.version>
> >>
> >>
> >
> >
> >
> > --
> > Ruwan Linton
> > Senior Software Engineer & Product Manager; WSO2 ESB;
> http://wso2.org/esb
> > WSO2 Inc.; http://wso2.org
> > email: ruwan@wso2.com; cell: +94 77 341 3097
> > blog: http://ruwansblog.blogspot.com
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
> For additional commands, e-mail: dev-help@synapse.apache.org
>
>
--
Ruwan Linton
Senior Software Engineer & Product Manager; WSO2 ESB; http://wso2.org/esb
WSO2 Inc.; http://wso2.org
email: ruwan@wso2.com; cell: +94 77 341 3097
blog: http://ruwansblog.blogspot.com
Re: svn commit: r769518 - in /synapse/trunk/java:
modules/transports/pom.xml pom.xml
Posted by Andreas Veithen <an...@gmail.com>.
What happened is that log4j disappeared from the distribution because
it is no longer a transitive dependency. I added it as an explicit
dependency of the distribution and this solves the problem for the log
format. This makes sense because the individual JARs we include in the
distribution should only depend on commons-logging, but not on any
particular logging implementation. It is only when we assemble the
distribution that we decide which logging framework we want to use.
Andreas
On Fri, May 1, 2009 at 07:08, Ruwan Linton <ru...@gmail.com> wrote:
> Andreas,
>
> I am not clear why... but there with this version of log4j it seems like the
> console log is completely out of format and also the shutdown hook log is
> not captured. This could be due to some other issue but when I run the build
> with the 1.2.15 dependency I cannot see this issue.
>
> Anyone else seeing this or is this just local to me?
>
> Thanks,
> Ruwan
>
> On Wed, Apr 29, 2009 at 1:26 AM, <ve...@apache.org> wrote:
>>
>> Author: veithen
>> Date: Tue Apr 28 19:56:19 2009
>> New Revision: 769518
>>
>> URL: http://svn.apache.org/viewvc?rev=769518&view=rev
>> Log:
>> Downgraded log4j from 1.2.15 (latest) to 1.2.14.
>>
>> Reason: 1.2.15 introduces additional transitive dependencies, some of
>> which are not resolvable. For the build this can easily be addressed using
>> an exclusion rule in the dependencyManagement entry for log4j. However, as
>> Ruwan reported, this doesn't work with the IDEA Maven plugin, and I still
>> see issues with the Eclipse plugin.
>>
>> Note that log4j was upgraded to the latest version by Asankha in r748973,
>> along with other dependencies. Since there is probably no particular reason
>> to prefer 1.2.15 over previous versions, we should use 1.2.14 so that we can
>> avoid cluttering the POM files with exclusions rules.
>>
>> Modified:
>> synapse/trunk/java/modules/transports/pom.xml
>> synapse/trunk/java/pom.xml
>>
>> Modified: synapse/trunk/java/modules/transports/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
>>
>> ==============================================================================
>> --- synapse/trunk/java/modules/transports/pom.xml (original)
>> +++ synapse/trunk/java/modules/transports/pom.xml Tue Apr 28 19:56:19 2009
>> @@ -185,12 +185,6 @@
>> <artifactId>axis2-transport-testkit</artifactId>
>> <version>${axis2.transport.version}</version>
>> <scope>test</scope>
>> - <exclusions>
>> - <exclusion>
>> - <groupId>com.sun.jmx</groupId>
>> - <artifactId>jmxri</artifactId>
>> - </exclusion>
>> - </exclusions>
>> </dependency>
>>
>> <!-- HTTP transport dependencies -->
>>
>> Modified: synapse/trunk/java/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/synapse/trunk/java/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
>>
>> ==============================================================================
>> --- synapse/trunk/java/pom.xml (original)
>> +++ synapse/trunk/java/pom.xml Tue Apr 28 19:56:19 2009
>> @@ -448,32 +448,6 @@
>> <groupId>log4j</groupId>
>> <artifactId>log4j</artifactId>
>> <version>${log4j.version}</version>
>> - <exclusions>
>> - <exclusion>
>> - <groupId>javax.mail</groupId>
>> - <artifactId>mail</artifactId>
>> - </exclusion>
>> - <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>oro</groupId>
>> - <artifactId>oro</artifactId>
>> - </exclusion>
>> - <exclusion>
>> - <groupId>junit</groupId>
>> - <artifactId>junit</artifactId>
>> - </exclusion>
>> - </exclusions>
>> </dependency>
>>
>> <dependency>
>> @@ -700,8 +674,8 @@
>> <type>jar</type>
>> <exclusions>
>> <exclusion>
>> - <groupId>com.sun.jmx</groupId>
>> - <artifactId>jmxri</artifactId>
>> + <groupId>log4j</groupId>
>> + <artifactId>log4j</artifactId>
>> </exclusion>
>> </exclusions>
>> </dependency>
>> @@ -867,7 +841,9 @@
>> <JLex.version>0.0</JLex.version>
>> <!-- misc -->
>> <junit.version>3.8.2</junit.version>
>> - <log4j.version>1.2.15</log4j.version>
>> + <!-- Warning: log4j versions above 1.2.14 introduce additional
>> transitive dependencies,
>> + some of which are not available from the Maven central
>> repository. -->
>> + <log4j.version>1.2.14</log4j.version>
>> <xmlunit.version>1.1</xmlunit.version>
>> <derby.version>10.4.2.0</derby.version>
>> <wrapper.version>3.2.3</wrapper.version>
>>
>>
>
>
>
> --
> Ruwan Linton
> Senior Software Engineer & Product Manager; WSO2 ESB; http://wso2.org/esb
> WSO2 Inc.; http://wso2.org
> email: ruwan@wso2.com; cell: +94 77 341 3097
> blog: http://ruwansblog.blogspot.com
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org
Re: svn commit: r769518 - in /synapse/trunk/java:
modules/transports/pom.xml pom.xml
Posted by Ruwan Linton <ru...@gmail.com>.
Andreas,
I am not clear why... but there with this version of log4j it seems like the
console log is completely out of format and also the shutdown hook log is
not captured. This could be due to some other issue but when I run the build
with the 1.2.15 dependency I cannot see this issue.
Anyone else seeing this or is this just local to me?
Thanks,
Ruwan
On Wed, Apr 29, 2009 at 1:26 AM, <ve...@apache.org> wrote:
> Author: veithen
> Date: Tue Apr 28 19:56:19 2009
> New Revision: 769518
>
> URL: http://svn.apache.org/viewvc?rev=769518&view=rev
> Log:
> Downgraded log4j from 1.2.15 (latest) to 1.2.14.
>
> Reason: 1.2.15 introduces additional transitive dependencies, some of which
> are not resolvable. For the build this can easily be addressed using an
> exclusion rule in the dependencyManagement entry for log4j. However, as
> Ruwan reported, this doesn't work with the IDEA Maven plugin, and I still
> see issues with the Eclipse plugin.
>
> Note that log4j was upgraded to the latest version by Asankha in r748973,
> along with other dependencies. Since there is probably no particular reason
> to prefer 1.2.15 over previous versions, we should use 1.2.14 so that we can
> avoid cluttering the POM files with exclusions rules.
>
> Modified:
> synapse/trunk/java/modules/transports/pom.xml
> synapse/trunk/java/pom.xml
>
> Modified: synapse/trunk/java/modules/transports/pom.xml
> URL:
> http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
>
> ==============================================================================
> --- synapse/trunk/java/modules/transports/pom.xml (original)
> +++ synapse/trunk/java/modules/transports/pom.xml Tue Apr 28 19:56:19 2009
> @@ -185,12 +185,6 @@
> <artifactId>axis2-transport-testkit</artifactId>
> <version>${axis2.transport.version}</version>
> <scope>test</scope>
> - <exclusions>
> - <exclusion>
> - <groupId>com.sun.jmx</groupId>
> - <artifactId>jmxri</artifactId>
> - </exclusion>
> - </exclusions>
> </dependency>
>
> <!-- HTTP transport dependencies -->
>
> Modified: synapse/trunk/java/pom.xml
> URL:
> http://svn.apache.org/viewvc/synapse/trunk/java/pom.xml?rev=769518&r1=769517&r2=769518&view=diff
>
> ==============================================================================
> --- synapse/trunk/java/pom.xml (original)
> +++ synapse/trunk/java/pom.xml Tue Apr 28 19:56:19 2009
> @@ -448,32 +448,6 @@
> <groupId>log4j</groupId>
> <artifactId>log4j</artifactId>
> <version>${log4j.version}</version>
> - <exclusions>
> - <exclusion>
> - <groupId>javax.mail</groupId>
> - <artifactId>mail</artifactId>
> - </exclusion>
> - <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>oro</groupId>
> - <artifactId>oro</artifactId>
> - </exclusion>
> - <exclusion>
> - <groupId>junit</groupId>
> - <artifactId>junit</artifactId>
> - </exclusion>
> - </exclusions>
> </dependency>
>
> <dependency>
> @@ -700,8 +674,8 @@
> <type>jar</type>
> <exclusions>
> <exclusion>
> - <groupId>com.sun.jmx</groupId>
> - <artifactId>jmxri</artifactId>
> + <groupId>log4j</groupId>
> + <artifactId>log4j</artifactId>
> </exclusion>
> </exclusions>
> </dependency>
> @@ -867,7 +841,9 @@
> <JLex.version>0.0</JLex.version>
> <!-- misc -->
> <junit.version>3.8.2</junit.version>
> - <log4j.version>1.2.15</log4j.version>
> + <!-- Warning: log4j versions above 1.2.14 introduce additional
> transitive dependencies,
> + some of which are not available from the Maven central
> repository. -->
> + <log4j.version>1.2.14</log4j.version>
> <xmlunit.version>1.1</xmlunit.version>
> <derby.version>10.4.2.0</derby.version>
> <wrapper.version>3.2.3</wrapper.version>
>
>
>
--
Ruwan Linton
Senior Software Engineer & Product Manager; WSO2 ESB; http://wso2.org/esb
WSO2 Inc.; http://wso2.org
email: ruwan@wso2.com; cell: +94 77 341 3097
blog: http://ruwansblog.blogspot.com