[Building Sakai] Alert: Moving spring-webmvc out of shared

[David Horwitz]

> I hate to -1 one my own suggestion, just looking for others to verify
> this works correctly.
>
> How much testing has been done with this, I've been seeing a few weird
> things, but I'm trying to shake out if they are solely things
> affecting rsmart specific tools or not.  I just want to make sure
> someone else other than me does some testing with this.  One issue was
> that we had a component that was including spring-mvc and pushing
> something into the spring context, which is really bad and crossing
> streams that shouldn't be crossed.  But even now that I've resolved
> that, I'm running into some other weird errors that I haven't been
> able to resolve.  Core sakai tools seems fine.
>
> I'd personally like to do some more testing, but if others have done
> some and it looks good then I'm fine with this.

I think I've done slightly less testing than you. The reason I'd 
recommend doing this now is that it will be slightly disruptive at best 
this way we can weed out any weird issues well before 2.9 and if 
completely disaster is the result revert the change without threatening 
the 2.9 timeline.

D


> +1 for change!
>
> On Wed, Jun 29, 2011 at 8:30 AM, csev<csev at umich.edu>  wrote:
>> +1 - sounds good - and he contrib folks will catch on pretty quickly since the error is nice and obvious.
>>
>> /Chuck
>>
>> On Jun 29, 2011, at 5:54 AM, David Horwitz wrote:
>>
>>> Hi All,
>>>
>>> Please see https://jira.sakaiproject.org/browse/KNL-757
>>>
>>> This concerns moving spring-webmvc out of shared and allowing webapps to
>>> package their own  versions and thereby allow more flexibility in UI
>>> work. Testing indicates that the change will is fairly trouble free with
>>> one caveat. That is that a small number of tools may have an undeclared
>>> runtime requirement for spring-webmvc, these webapps will fail on start
>>> up with a Classnotfound error relating to a spring class. Adding
>>> spring-webmvc to the tools pom so its packaged in the webapp fixes this.
>>>
>>> Unless I hear otherwise from the list I will commit this on Saturday
>>> morning GMT+2
>>>
>>> David
>>> _______________________________________________
>>> sakai-dev mailing list
>>> sakai-dev at collab.sakaiproject.org
>>> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>>
>>> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"
>>>
>>>
>> _______________________________________________
>> sakai-dev mailing list
>> sakai-dev at collab.sakaiproject.org
>> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>>
>> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"
>>
> _______________________________________________
> sakai-dev mailing list
> sakai-dev at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>
> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"