bookmarks: adjust exception type so present(bookmark(.)) works as expected

mercurial/bookmarks.py

@@ -240,7 +240,7 b' 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):

tests/test-bookmarks-pushpull.t

@@ -345,7 +345,7 b' 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

tests/test-bookmarks.t

@@ -172,12 +172,9 b' 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 b' 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 b' 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