diff --git a/mercurial/bookmarks.py b/mercurial/bookmarks.py --- a/mercurial/bookmarks.py +++ b/mercurial/bookmarks.py @@ -240,7 +240,7 @@ class bmstore(object): if self.active: return self.active else: - raise error.Abort(_("no active bookmark")) + raise error.RepoLookupError(_("no active bookmark")) return bname def checkconflict(self, mark, force=False, target=None): diff --git a/tests/test-bookmarks-pushpull.t b/tests/test-bookmarks-pushpull.t --- a/tests/test-bookmarks-pushpull.t +++ b/tests/test-bookmarks-pushpull.t @@ -345,7 +345,7 @@ demand that one of the bookmarks is acti 1 files updated, 0 files merged, 0 files removed, 0 files unresolved (leaving bookmark V) $ hg push -B . ../a - abort: no active bookmark + abort: no active bookmark! [255] $ hg update -r V 0 files updated, 0 files merged, 1 files removed, 0 files unresolved diff --git a/tests/test-bookmarks.t b/tests/test-bookmarks.t --- a/tests/test-bookmarks.t +++ b/tests/test-bookmarks.t @@ -172,12 +172,9 @@ but "literal:." is not since "." seems n $ hg bookmark --inactive $ hg log -r 'bookmark(.)' - abort: no active bookmark + abort: no active bookmark! [255] -BUG: this should be resolved to an empty set: $ hg log -r 'present(bookmark(.))' - abort: no active bookmark - [255] $ hg log -r 'bookmark(unknown)' abort: bookmark 'unknown' does not exist! @@ -263,7 +260,7 @@ rename bookmark using . with no active b $ hg book rename-me $ hg book -i rename-me $ hg book -m . renamed - abort: no active bookmark + abort: no active bookmark! [255] $ hg up -q Y $ hg book -d rename-me @@ -283,7 +280,7 @@ delete bookmark using . with no active b $ hg book delete-me $ hg book -i delete-me $ hg book -d . - abort: no active bookmark + abort: no active bookmark! [255] $ hg up -q Y $ hg book -d delete-me