I'll ping Chris to ask about it... we could also use a new jarjar release to fix http://code.google.com/p/jarjar/issues/detail?id=21 (we're using a locally built jarjar snapshot to fix it right now).
sam On Mon, Nov 22, 2010 at 9:45 AM, Stuart McCulloch <[email protected]> wrote: > 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]<google-guice%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/google-guice?hl=en. > -- 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.
