# HG changeset patch # User Matt Mackall # Date 1130369570 25200 # Node ID 214f42f23a3bbf04efb96ec1c02097cad11d018d # Parent 407bd229f00399aedf542aebff107e60947f35d5 Remove copy/rename --parents option This is no longer needed now that we have more standard UNIX-like copy semantics. diff -r 407bd229f003 -r 214f42f23a3b doc/hg.1.txt --- a/doc/hg.1.txt Wed Oct 26 16:24:10 2005 -0700 +++ b/doc/hg.1.txt Wed Oct 26 16:32:50 2005 -0700 @@ -201,7 +201,6 @@ -I, --include include names matching the given patterns -X, --exclude exclude names matching the given patterns -f, --force forcibly copy over an existing managed file - -p, --parents append source path to dest aliases: cp @@ -493,7 +492,6 @@ Options: -A, --after record a rename that has already occurred -f, --force forcibly copy over an existing managed file - -p, --parents append source path to dest aliases: mv diff -r 407bd229f003 -r 214f42f23a3b mercurial/commands.py --- a/mercurial/commands.py Wed Oct 26 16:24:10 2005 -0700 +++ b/mercurial/commands.py Wed Oct 26 16:32:50 2005 -0700 @@ -788,7 +788,7 @@ pats = list(pats) dest = pats.pop() sources = [] - dir2dir = not opts['parents'] and len(pats) == 1 and os.path.isdir(pats[0]) + dir2dir = len(pats) == 1 and os.path.isdir(pats[0]) def okaytocopy(abs, rel, exact): reasons = {'?': _('is not managed'), @@ -814,12 +814,10 @@ destisfile = not dir2dir and (len(sources) == 1 or repo.dirstate.state(absdest) != '?') - if destisfile: - if opts['parents']: - raise util.Abort(_('with --parents, destination must be a directory')) - elif len(sources) > 1: - raise util.Abort(_('with multiple sources, destination must be a ' - 'directory')) + if destisfile and len(sources) > 1: + raise util.Abort(_('with multiple sources, destination must be a ' + 'directory')) + srcpfxlen = 0 if dir2dir: srcpfx = util.pathto(cwd, util.canonpath(repo.root, cwd, pats[0])) @@ -831,9 +829,7 @@ errs, copied = 0, [] for abs, rel, exact in sources: - if opts['parents']: - mydest = os.path.join(dest, rel) - elif destisfile: + if destisfile: mydest = reldest elif dir2dir: mydest = os.path.join(dest, rel[srcpfxlen:]) @@ -847,7 +843,7 @@ mydestdir = os.path.dirname(myreldest) or '.' if not opts['after']: try: - if opts['parents'] or dir2dir: os.makedirs(mydestdir) + if dir2dir: os.makedirs(mydestdir) elif not destisfile: os.mkdir(mydestdir) except OSError, inst: if inst.errno != errno.EEXIST: raise @@ -2213,8 +2209,7 @@ [('I', 'include', [], _('include names matching the given patterns')), ('X', 'exclude', [], _('exclude names matching the given patterns')), ('A', 'after', None, _('record a copy that has already occurred')), - ('f', 'force', None, _('forcibly copy over an existing managed file')), - ('p', 'parents', None, _('append source path to dest'))], + ('f', 'force', None, _('forcibly copy over an existing managed file'))], _('hg copy [OPTION]... [SOURCE]... DEST')), "debugancestor": (debugancestor, [], _('debugancestor INDEX REV1 REV2')), "debugcheckstate": (debugcheckstate, [], _('debugcheckstate')), @@ -2336,8 +2331,7 @@ [('I', 'include', [], _('include names matching the given patterns')), ('X', 'exclude', [], _('exclude names matching the given patterns')), ('A', 'after', None, _('record a rename that has already occurred')), - ('f', 'force', None, _('forcibly copy over an existing managed file')), - ('p', 'parents', None, _('append source path to dest'))], + ('f', 'force', None, _('forcibly copy over an existing managed file'))], _('hg rename [OPTION]... [SOURCE]... DEST')), "^revert": (revert,