You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rave.apache.org by Jasha Joachimsthal <j....@onehippo.com> on 2012/03/28 15:59:58 UTC

Re: svn commit: r1304156 - in /incubator/rave/trunk: ./ rave-portal-dependencies/ rave-providers/rave-opensocial-provider/ rave-providers/rave-opensocial-provider/rave-opensocial-client/ rave-providers/rave-opensocial-provider/rave-opensocial-client/

On 28 March 2012 15:53, Scott Wilson <sc...@gmail.com> wrote:

>
> On 28 Mar 2012, at 14:48, Franklin, Matthew B. wrote:
>
> >> -----Original Message-----
> >> From: Ate Douma [mailto:ate@douma.nu]
> >> Sent: Wednesday, March 28, 2012 9:09 AM
> >> To: dev@rave.apache.org; Franklin, Matthew B.
> >> Subject: Re: svn commit: r1304156 - in /incubator/rave/trunk: ./
> rave-portal-
> >> dependencies/ rave-providers/rave-opensocial-provider/ rave-
> >> providers/rave-opensocial-provider/rave-opensocial-client/ rave-
> >> providers/rave-opensocial-provider/rave-opensocial-client/sr...
> >>
> >> Matt, any particular reason why you downgraded the slf4j version?
> >>
> >> SLF4J 1.6 has some additional API features which now cause a build
> failure in
> >> rave-vanilla-extension. Nothing serious which cannot be fixed, but
> unless
> >> there
> >> is a critical reason for this downgrade, I'd like to bump it back up
> again :)
> >>
> >
> > There was a dependency collision in shindig.  Once I moved the code out
> of the overlay I was getting a ton of errors and the only way to make it
> work was to go with 1.5.  I have it on my list of TODOs to create a ticket
> to bump it back to 1.6 when we move to shindig 2.5 beta 1 and have the
> ability to create the rave-shindig-dependencies & rave-shindig-resources
> submodules.
>
> I've had the same problem before - SLF4J versions are incompatible and
> cause compile errors. Can you not just exclude slf4j in the dependencies?
>

Yes they are incompatible, but can't we have 1.5 only in the war for
rave-shindig and 1.6 in the rest of the stack?


>
> >
> > -Matt
> >
> >> Ate
> >>
> >> On 03/23/2012 02:08 AM, mfranklin@apache.org wrote:
> >>> Author: mfranklin
> >>> Date: Fri Mar 23 01:08:47 2012
> >>> New Revision: 1304156
> >>>
> >>> URL: http://svn.apache.org/viewvc?rev=1304156&view=rev
> >>> Log:
> >>> Reorganization of project to move rave-shindig and split out components
> >> (Fixes RAVE-524)
> >>>
> >>
> >>>
> >>> Modified: incubator/rave/trunk/pom.xml
> >>> URL:
> >> http://svn.apache.org/viewvc/incubator/rave/trunk/pom.xml?rev=1304156&
> >> r1=1304155&r2=1304156&view=diff
> >>>
> >> ===========================================================
> >> ===================
> >>> --- incubator/rave/trunk/pom.xml (original)
> >>> +++ incubator/rave/trunk/pom.xml Fri Mar 23 01:08:47 2012
> >>> @@ -51,7 +51,7 @@
> >>>          <javax.servlet.version>2.5</javax.servlet.version>
> >>>          <jsp-api.version>2.1</jsp-api.version>
> >>>          <el-api.version>1.0</el-api.version>
> >>> -<org.slf4j.version>1.6.1</org.slf4j.version>
> >>> +<org.slf4j.version>1.5.11</org.slf4j.version>
>
>

RE: svn commit: r1304156 - in /incubator/rave/trunk: ./ rave-portal-dependencies/ rave-providers/rave-opensocial-provider/ rave-providers/rave-opensocial-provider/rave-opensocial-client/ rave-providers/rave-opensocial-provider/rave-opensocial-client/

Posted by "Franklin, Matthew B." <mf...@mitre.org>.
>-----Original Message-----
>From: Jasha Joachimsthal [mailto:j.joachimsthal@onehippo.com]
>Sent: Wednesday, March 28, 2012 10:00 AM
>To: dev@rave.apache.org
>Subject: Re: svn commit: r1304156 - in /incubator/rave/trunk: ./ rave-portal-
>dependencies/ rave-providers/rave-opensocial-provider/ rave-
>providers/rave-opensocial-provider/rave-opensocial-client/ rave-
>providers/rave-opensocial-provider/rave-opensocial-client/
>
>On 28 March 2012 15:53, Scott Wilson <sc...@gmail.com>
>wrote:
>
>>
>> On 28 Mar 2012, at 14:48, Franklin, Matthew B. wrote:
>>
>> >> -----Original Message-----
>> >> From: Ate Douma [mailto:ate@douma.nu]
>> >> Sent: Wednesday, March 28, 2012 9:09 AM
>> >> To: dev@rave.apache.org; Franklin, Matthew B.
>> >> Subject: Re: svn commit: r1304156 - in /incubator/rave/trunk: ./
>> rave-portal-
>> >> dependencies/ rave-providers/rave-opensocial-provider/ rave-
>> >> providers/rave-opensocial-provider/rave-opensocial-client/ rave-
>> >> providers/rave-opensocial-provider/rave-opensocial-client/sr...
>> >>
>> >> Matt, any particular reason why you downgraded the slf4j version?
>> >>
>> >> SLF4J 1.6 has some additional API features which now cause a build
>> failure in
>> >> rave-vanilla-extension. Nothing serious which cannot be fixed, but
>> unless
>> >> there
>> >> is a critical reason for this downgrade, I'd like to bump it back up
>> again :)
>> >>
>> >
>> > There was a dependency collision in shindig.  Once I moved the code out
>> of the overlay I was getting a ton of errors and the only way to make it
>> work was to go with 1.5.  I have it on my list of TODOs to create a ticket
>> to bump it back to 1.6 when we move to shindig 2.5 beta 1 and have the
>> ability to create the rave-shindig-dependencies & rave-shindig-resources
>> submodules.
>>
>> I've had the same problem before - SLF4J versions are incompatible and
>> cause compile errors. Can you not just exclude slf4j in the dependencies?
>>
>
>Yes they are incompatible, but can't we have 1.5 only in the war for
>rave-shindig and 1.6 in the rest of the stack?
>

I tried exclusion, but the problem was that rave-opensocial-core is compiled against 1.6 and dropped into shindig running 1.5 and we get a No Such Method exception.  I tried making rave-opensocial-core compile explicitly against 1.5.11 but ended up with issues coming from rave-commons.

Long story short, I didn't see another way.  If someone can fix it, I would be ecstatic :)

>
>>
>> >
>> > -Matt
>> >
>> >> Ate
>> >>
>> >> On 03/23/2012 02:08 AM, mfranklin@apache.org wrote:
>> >>> Author: mfranklin
>> >>> Date: Fri Mar 23 01:08:47 2012
>> >>> New Revision: 1304156
>> >>>
>> >>> URL: http://svn.apache.org/viewvc?rev=1304156&view=rev
>> >>> Log:
>> >>> Reorganization of project to move rave-shindig and split out
>components
>> >> (Fixes RAVE-524)
>> >>>
>> >>
>> >>>
>> >>> Modified: incubator/rave/trunk/pom.xml
>> >>> URL:
>> >>
>http://svn.apache.org/viewvc/incubator/rave/trunk/pom.xml?rev=1304156&
>> >> r1=1304155&r2=1304156&view=diff
>> >>>
>> >>
>===========================================================
>> >> ===================
>> >>> --- incubator/rave/trunk/pom.xml (original)
>> >>> +++ incubator/rave/trunk/pom.xml Fri Mar 23 01:08:47 2012
>> >>> @@ -51,7 +51,7 @@
>> >>>          <javax.servlet.version>2.5</javax.servlet.version>
>> >>>          <jsp-api.version>2.1</jsp-api.version>
>> >>>          <el-api.version>1.0</el-api.version>
>> >>> -<org.slf4j.version>1.6.1</org.slf4j.version>
>> >>> +<org.slf4j.version>1.5.11</org.slf4j.version>
>>
>>