[IPython-dev] Shutdown __del__ methods bug
Fri Oct 22 03:08:34 CDT 2010
On Sat, Oct 16, 2010 at 8:29 AM, Thomas Kluyver <firstname.lastname@example.org> wrote:
> Unfortunately, this commit appears to have undone Fernando's fix for the bug
> with __del__ methods that I found:
> I can reproduce the bug in trunk. This also highlights that the doctest to
> catch it evidently doesn't do so. Running iptest IPython.core shows the
> error message on the console (among the ... of passed tests), but it doesn't
> fail. I'm not a console ninja, but could it be that the message goes to
> stderr, and the evaluation only checks stdout?
Thomas, thanks a lot for this fix, I've merged it since it was a total
no-brainer. Great detective work on finding where it got reverted,
but I'm quite bothered by that having happened: do you have any idea
why that commit would have reverted those changes? That commit:
appears as a simple merge commit, but its diff is gigantic (the entire
merge). I don't understand why the merge Min did reverted that
And I am actually *very* worried that there might have been other
damage done in that commit that we didn't catch simply because it
didn't trigger any test failure...
Any thoughts? I really don't want to leave dangling the notion that
we might have had a commit that undid a lot of other stuff we thought
was already in there... I really have no idea what happened here, any
clarity will be welcome.
More information about the IPython-dev