static http and bookmarks?

[Giorgos Keramidas]

I've recently bumped against this exception when pulling from a static
http repository:

,-----------------------------------------------------------------------
| keramida at kobe:/hg/doc/typaldos$ hg pull
| pulling from http://typal-doc.superuser.gr
| αναζήτηση αλλαγών
| δεν υπάρχουν αλλαγές
| ** unknown exception encountered, details follow
| ** report bug details to http://mercurial.selenic.com/bts/
| ** or mercurial at selenic.com
| ** Python 2.6.5 (r265:79063, Jul 22 2010, 11:03:11) [GCC 4.2.1 20070719  [FreeBSD]]
| ** Mercurial Distributed SCM (version 1.6.1+4-c20dcfbb02f5)
| ** Extensions loaded: bookmarks, color, convert, extdiff, gpg, graphlog, mq, notify, pager, patchbomb, record, relink, transplant, purge, rebase
| Traceback (most recent call last):
|   File "/usr/local/bin/hg", line 27, in <module>
|     mercurial.dispatch.run()
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 16, in run
|     sys.exit(dispatch(sys.argv[1:]))
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 36, in dispatch
|     return _runcatch(u, args)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 58, in _runcatch
|     return _dispatch(ui, args)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 516, in _dispatch
|     cmdpats, cmdoptions)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 377, in runcommand
|     ret = _runcommand(ui, options, cmd, d)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/extensions.py", line 174, in wrap
|     return wrapper(origfn, *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/hgext/pager.py", line 95, in pagecmd
|     return orig(ui, options, cmd, cmdfunc)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/extensions.py", line 174, in wrap
|     return wrapper(origfn, *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/hgext/color.py", line 211, in colorcmd
|     return orig(ui_, opts, cmd, cmdfunc)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 567, in _runcommand
|     return checkargs()
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 521, in checkargs
|     return cmdfunc()
|   File "/usr/local/lib/python2.6/site-packages/mercurial/dispatch.py", line 514, in <lambda>
|     d = lambda: util.checksignature(func)(ui, *args, **cmdoptions)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/extensions.py", line 129, in wrap
|     util.checksignature(origfn), *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/hgext/rebase.py", line 530, in pullrebase
|     orig(ui, repo, *args, **opts)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/extensions.py", line 129, in wrap
|     util.checksignature(origfn), *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/hgext/mq.py", line 2826, in mqcommand
|     return orig(ui, repo, *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/extensions.py", line 129, in wrap
|     util.checksignature(origfn), *args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/hgext/bookmarks.py", line 409, in pull
|     result = oldpull(ui, repo, source, **opts)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/util.py", line 426, in check
|     return func(*args, **kwargs)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/commands.py", line 2793, in pull
|     modheads = repo.pull(other, heads=revs, force=opts.get('force'))
|   File "/usr/local/lib/python2.6/site-packages/hgext/bookmarks.py", line 301, in pull
|     rb = remote.listkeys('bookmarks')
|   File "/usr/local/lib/python2.6/site-packages/mercurial/localrepo.py", line 1788, in listkeys
|     return pushkey.list(self, namespace)
|   File "/usr/local/lib/python2.6/site-packages/mercurial/pushkey.py", line 30, in list
|     return lk(repo)
|   File "/usr/local/lib/python2.6/site-packages/hgext/bookmarks.py", line 374, in listbookmarks
|     for k, v in repo._bookmarks.iteritems():
| AttributeError: 'statichttprepository' object has no attribute '_bookmarks'
| keramida at kobe:/hg/doc/typaldos$
`-----------------------------------------------------------------------

Is this supposed to work?  If not, can we skip repo types that have no
'_bookmarks' attribute in listbookmarks()?