[Ping][MERGE] [bug #111664] bzr rm refuses to delete renamed files

[Marius Kruger]

Hi,
I know I'm a bit hasty, but I'd like to work on it again this
weekend (if needed), so any input would be appreciated.
regards
marius

On 5/7/07, Marius Kruger <amanic at gmail.com> wrote:
> hi guys,
>
> Thank you for the abundant responses to this bundle!
> Hopefully I addressed it sufficiently..
> I had to make quite a lot of changes, so
> I rather uncommitted the last revision in my previous bundle
> and did it over. (I did get a -0 after all)
>
> I followed Aaron's suggestion to rather use _iter_changes()
> to detect changed files, which in the end worked much
> better. This might also have fixed another 'bug':
> If you specified bogus files (files that never existed and still doesn't)
> to changes_from() it considered them unknown. Thus
> in the tests we used to expect this case to throw
> an exception. But now we don't need to any more
> because _iter_changes() ignores them.
> So workingtree.remove just notes that the file(s) does
> not exist and continues.
> When there is a change though, I kept it to still use changes_from()
> to generate status output for my exception,
> as I didn't want to change too much now, and I didn't
> want to go and generate my own status output.
>
>
> regards
> marius
>
>


-- 
bazaar-vcs.org
Because I don't trust Version Control Systems with less than 6350 unit tests.