Correct, that was the error I experienced. Tommaso
2011/12/14 Raymond Feng <[email protected]> > Yes. You are right. The patch didn't contain the deletion. > > Raymond Feng > Sent from my iPhone > > On Dec 13, 2011, at 11:48 PM, Antonio Sanso <[email protected]> wrote: > > > Hi Raymond, > > > > the error that Tommaso probably refers to are contained in the > org.apache.amber.oauth2.integration.AccessTokenAssertion. > > This class still contains some reference to the no more existing > GrantType.ASSERTION. > > If I am not mistaken though this class should not be there anymore in > the first place (and probably is not anymore in your workspace, that is why > you do not see any error). > > > > Just a guess > > > > Regards > > > > Antonio > > > > > > On Dec 14, 2011, at 12:09 AM, Raymond Feng wrote: > > > > Hi, > > > > I'm not seeing test failures in my local build. Please let me know what > errors you ran into. > > > > Thanks, > > Raymond > > ________________________________________________________________ > > Raymond Feng > > [email protected]<mailto:[email protected]> > > Apache Software Foundation Member > > Apache Tuscany PMC member and committer: tuscany.apache.org > > Co-author of Tuscany SCA In Action book: www.tuscanyinaction.com > > Personal Web Site: www.enjoyjava.com > > ________________________________________________________________ > > > > On Dec 13, 2011, at 12:57 PM, Tommaso Teofili wrote: > > > > Hello Antonio, > > > > I intended to commit it some days ago but it looked like there were some > > integration tests failing due to it. > > Since I wanted to double check it I postponed it a little, I am > travelling > > these days so I'll probably be able to review it in the first days of the > > next week, if anyone of the committers is willing to do it before feel > free > > :) > > Cheers, > > Tommaso > > > > 2011/12/13 Antonio Sanso <[email protected]> > > > > Hello Amber committers, > > > > I was wondering if any of the committer might take some action regarding > > AMBER 42. I must highlight is not my patch and that there is not any > > intention from my side to push/rush anything :) > > The main reason that made me write this mail is the size of the patch > > itself. While it is a really good patch (thanks Raymond for it) it is a > > pretty big one. > > If somebody with write SVN commit would review and apply it, it would be > > really good (I have already put my non binding +1 after reviewing it ) . > > This would allow other people to do new patches in an easier way without > > overlapping with any of the several files included in the AMBER-42 patch. > > > > WDYT? > > > > Regards > > > > Antonio > > > > [0] https://issues.apache.org/jira/browse/AMBER-42 > > > > > > >
