> On Mon,  3 Jun 2013 12:53:18 +0900 (JST)
> [email protected] (YAMAMOTO Takashi) wrote:
> 
>> hi,
>> 
>>> On Thu, May 23, 2013 at 08:07:48PM +0900, FUJITA Tomonori wrote:
>>>> On Thu, 23 May 2013 15:30:39 +0900
>>>> Isaku Yamahata <[email protected]> wrote:
>>>> 
>>>> > On Thu, May 23, 2013 at 08:53:09AM +0900, FUJITA Tomonori wrote:
>>>> > > On Fri, 17 May 2013 11:52:07 +0900
>>>> > > YAMAMOTO Takashi <[email protected]> wrote:
>>>> > > 
>>>> > > > 0.3.0 (commit df6449f17366b9d608895766cc1a151167460ae0)
>>>> > > > 
>>>> > > > only pure python code for now because:
>>>> > > > - we don't want to worry about cython setup.py stuff
>>>> > > > - cython version has a known bug
>>>> > > >   cf. https://github.com/msgpack/msgpack-python/pull/64
>>>> > > 
>>>> > > Can you describe why we need this in a commit message? However, a fix
>>>> > > for this issue was merged into grizzly/stable and master. If so, if we
>>>> > > ignore folsom, we don't need to have this in contrib directory, right?
>>>> > 
>>>> > If the next grizzly/stable is released, no, we don't.
>>>> > It's about two weeks.
>>>> 
>>>> Then I'm inclined not to put this into contrib. Objections?
>>> 
>>> No objection from me.
>> 
>> cython code in msgpack-python has the above mentioned bug.
>> some of unit tests in this patchset fail due to this.
>> although i pushed fixes into the upstream, it would be safer
>> for us to have it in contrib for a while.
> 
> The above bug is about a value over LONG_MAX? Then unlikely we would
> hit it.

no.
it's about values which don't fit in msgpack 32-bit integer format
but fits in C long.

YAMAMOTO Takashi

> 
> 
> ------------------------------------------------------------------------------
> Get 100% visibility into Java/.NET code with AppDynamics Lite
> It's a free troubleshooting tool designed for production
> Get down to code-level detail for bottlenecks, with <2% overhead.
> Download for free and get started troubleshooting in minutes.
> http://p.sf.net/sfu/appdyn_d2d_ap2
> _______________________________________________
> Ryu-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ryu-devel

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite
It's a free troubleshooting tool designed for production
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap2
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to