On Sun, Dec 5, 2010 at 7:55 AM, Stefan Behnel <[email protected]> wrote:
> mark florisson, 05.12.2010 15:11:
>> On 5 December 2010 14:15, Stefan Behnel wrote:
>>> I haven't tested it yet, mainly because I don't have the latest gdb version
>>> installed. When I tried building lxml with that branch, I ran into a couple
>>> of compiler crashes that Mark resolved, but I can't tell if there aren't
>>> any left. I also didn't review the code in any way.
>>>
>>> AFAIC, I'm not sure I want this right in 0.14, but that's more because I
>>> can't really comment on it. I certainly agree that it needs more testing.
>> [...]
>> But I agree that it should be tested more widely before merging it. The
>> documentation (currently only on hudson but I presume it will make it
>> to docs.cython.org with the new release) already contains a link to my
>> git branch so users that are interested could still use it.
>
> Then I would suggest we keep the docs as they are now to make your gdb
> support visible, but with a big note that this hasn't been merged yet and
> keep your branch alive for a bit longer after the release. I wouldn't mind
> merging it for 0.14.1 when it seems mature enough, and I hope that release
> isn't too far off. 0.14 is a release with many, many new features, so it's
> quite likely that people find bugs quickly once it's out in the wild.

OTOH, when the flag is off, the changes made are quite noninvasive, so
I would be fine merging this as an experimental feature to get more
widespread testing and for what utility it already has, even if it's
not fully polished yet (not that I have any bugs to point at...).

- Robert
_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to