Code review comment for lp:~slyguy/bzr/bug-855155

Revision history for this message
John A Meinel (jameinel) wrote :

I think the test is very good. I'm concerned that this is fixing where the exception was, when there might be a different logic error (why aren't we creating the entry for 'a' before this happens, even if it is an empty directory.)

I'll look into that side of the logic, if I can't find something better we should certainly land this.

« Back to merge proposal