import: abort usefully if no patch name given stable
authorKevin Bullock <kbullock@ringworld.org>
Fri, 21 Oct 2011 11:10:43 -0500
branchstable
changeset 15327 67e92d29ecb5
parent 15326 8ae2900d6d9b
child 15328 9b4ab5f7ad2a
import: abort usefully if no patch name given
mercurial/commands.py
tests/test-import.t
--- a/mercurial/commands.py	Fri Oct 21 14:15:58 2011 +0200
+++ b/mercurial/commands.py	Fri Oct 21 11:10:43 2011 -0500
@@ -3356,7 +3356,7 @@
      _('use any branch information in patch (implied by --exact)'))] +
     commitopts + commitopts2 + similarityopts,
     _('[OPTION]... PATCH...'))
-def import_(ui, repo, patch1, *patches, **opts):
+def import_(ui, repo, patch1=None, *patches, **opts):
     """import an ordered set of patches
 
     Import a list of patches and commit them individually (unless
@@ -3418,6 +3418,10 @@
 
     Returns 0 on success.
     """
+
+    if not patch1:
+        raise util.Abort(_('need at least one patch to import'))
+
     patches = (patch1,) + patches
 
     date = opts.get('date')
--- a/tests/test-import.t	Fri Oct 21 14:15:58 2011 +0200
+++ b/tests/test-import.t	Fri Oct 21 11:10:43 2011 -0500
@@ -10,6 +10,11 @@
   $ echo line 2 >> a/a
   $ hg --cwd a ci -u someone -d '1 0' -m'second change'
 
+import with no args:
+
+  $ hg --cwd a import
+  abort: need at least one patch to import
+  [255]
 
 generate patches for the test