On 21 November 2010 22:40, Sam Berlin <[email protected]> wrote: > Stuart, > > I did a little detective work, and I think Guice's current cglib snapshot > is based off the current cglib CVS HEAD.
Thanks Sam - do you know if the cglib folks are planning a new release... or at least a new tag? [ fwiw the sisu-cglib artifact used by the maven build is built from the same code in CVS HEAD, I'm just using it as a placeholder until we can get an official cglib release onto maven central ] Here's the history: > > * Guice initially began with cglib 2.2-beta1 (as mentioned in the thread > at [1]). > * On April 1st, 2008, Matt Bastress posted a message to the guice list > saying he discovered a race condition in cglib [1]. > * On April 15th, 2008, Matt added the bug to cglib [2]. > * On May 26, 2008, Chris N updated the cglib source to fix the bug Matt > reported [3]. The fixed version of the file is tagged RELEASE_2_2 > * On Oct 10, 2008, Daniel Martin added a new bug to cglib [4]. > * On Jan 11th, 2009, Chris N marked the bug Matt opened in cglib as fixed > [2]. > * Also on Jan 11th, 2009, Chris N marked the bug Daniel Martin added as > fixed [4]. > * Also on Jan 11th, 2009, Chris N committed a fix for Daniel Martin's bug > [5]. > * Also on Jan 11th, 2009, Chris N updated guice's cglib snapshot [6]. > * None of the fixes Chris committed to cglib on the 11th appear to be > tagged in cglib, so I don't think any new official release was ever created. > > [1] > http://groups.google.com/group/google-guice/browse_thread/thread/94e681aa7201dc0b/8eb5ca1e91b18298 > [2] > http://sourceforge.net/tracker/index.php?func=detail&aid=1943116&group_id=56933&atid=482370 > [3] > http://cglib.cvs.sourceforge.net/viewvc/cglib/cglib/src/proxy/net/sf/cglib/proxy/MethodProxy.java?r1=1.13&r2=1.14 > [4] > http://sourceforge.net/tracker/?func=detail&aid=2157542&group_id=56933&atid=482368 > [5] http://www.ohloh.net/p/cglib/commits/33948862 > [6] http://code.google.com/p/google-guice/source/detail?spec=svn1422&r=804 > > sam > > On Wed, Nov 17, 2010 at 6:59 PM, Stuart McCulloch <[email protected]>wrote: > >> Hi, >> >> Guice (when building with Ant) currently builds against a local snapshot >> binary of CGLIB that includes no source :/ >> >> Is it possible to get an official release of CGLIB with the fixes used by >> Guice (assuming they've been pushed upstream) >> >> Or at the very least can we add the exact source that went into this >> binary to the Guice build? >> >> This will a) help when debugging and b) let people verify what exactly >> went into the final jar (since CGLIB is embedded) >> >> -- >> Cheers, Stuart >> > -- You received this message because you are subscribed to the Google Groups "google-guice" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/google-guice?hl=en.
