From patchwork Thu Dec  7 15:09:24 2017
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [6,of,6] revset: make follow() accept empty startrev
From: Yuya Nishihara <yuya@tcha.org>
X-Patchwork-Id: 26000
Message-Id: <f2a19afb19c146c59956.1512659364@mimosa>
To: mercurial-devel@mercurial-scm.org
Date: Fri, 08 Dec 2017 00:09:24 +0900

# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1474540902 -32400
#      Thu Sep 22 19:41:42 2016 +0900
# Node ID f2a19afb19c146c59956d33d48ab91a7b76896b8
# Parent  394d25004f912b5b104e2f56a042b3d1a3b2ca14
revset: make follow() accept empty startrev

This is the same behavior as ancestors(emptyset).

diff --git a/mercurial/revset.py b/mercurial/revset.py
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -913,9 +913,6 @@ def _follow(repo, subset, x, name, follo
     revs = None
     if 'startrev' in args:
         revs = getset(repo, fullreposet(repo), args['startrev'])
-        if not revs:
-            raise error.RepoLookupError(
-                _("%s expected at least one starting revision") % name)
     if 'file' in args:
         x = getstring(args['file'], _("%s expected a pattern") % name)
         if revs is None:
diff --git a/tests/test-log.t b/tests/test-log.t
--- a/tests/test-log.t
+++ b/tests/test-log.t
@@ -742,8 +742,6 @@ follow files starting from multiple revi
 follow files starting from empty revision:
 
   $ hg log -T '{rev}: {files}\n' -r "follow('glob:*', startrev=.-.)"
-  abort: follow expected at least one starting revision!
-  [255]
 
 follow starting from revisions: