[IPython-dev] Shutdown __del__ methods bug
Fri Oct 22 13:55:09 CDT 2010
Sure, we can meet to look into it. If I recall correctly, I exactly
followed the merge flow provided by github at the bottom of the pull
On Fri, Oct 22, 2010 at 10:59, Fernando Perez <email@example.com> wrote:
> On Fri, Oct 22, 2010 at 2:49 AM, Thomas Kluyver <firstname.lastname@example.org> wrote:
> > A couple of other things strike me about it:
> > - In the ipython network graph on github, it doesn't show up as a merge
> > (there's no other branch leading into it). You have to scroll back a bit
> > to see it, but it's the first in a cluster of three commits on trunk just
> > before the number 11 on the date line.
> > - Other merges usually mention branches in the commit message. This one
> > mentions a file.
> > Perhaps Min can shed more light, if he remembers how he made the commit.
> > does seem that something didn't go to plan. I've noticed a few places
> > it's reverting to older forms of code (e.g. using exceptions.Exception),
> > I've been tidying them up in my cleanup-old-code branch. I'll put in a
> > request.
> > If you're worried about possible regressions, I think the best thing is
> > you and Min to go over the diff for that commit and work out which
> > were intentional.
> Yes, I am worried. Min, let's try to get together sometime next week
> and do a little forensics on this one, to make sure nothing else
> slipped by. I'd also like to understand *what happened* so we don't
> repeat this in the future, but I admit I'm puzzled right now.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the IPython-dev