[issue2214] qimport with a 2nd bogus filename doesn't cleanup properly

[Justin Dolske]

New submission from Justin Dolske <dolske at mozilla.com>:

A coworker new to mq discovered this. If you specify a 2nd file for qimport
which is non-existent, the first file is left in the .hg/patches directory
but it's not added to the series file, so attempting to qimport it again fails.

$ hg qseries
$ hg qimport ~/Desktop/newpatch oops
adding newpatch to series file
abort: unable to read oops
$ hg qimport ~/Desktop/newpatch
abort: patch "newpatch" already exists
$ hg qseries
$ ls .hg/patches/
newpatch series status

I'd expect either:
 1) remove .hg/patches/newpatch when the command fails
 2) add newpatch to series when that step succeeds, and keep both the
    patch and series entry when the qimport of "oops" fails.

----------
messages: 12678
nosy: dolske
priority: bug
status: unread
title: qimport with a 2nd bogus filename doesn't cleanup properly

____________________________________________________
Mercurial issue tracker <bugs at mercurial.selenic.com>
<http://mercurial.selenic.com/bts/issue2214>
____________________________________________________