Oh yeah, I didn't really consider the default. I clearly prefer "true", so I'll change it. Maybe it's not even worth having an option. I think the only advantage of not printing this column is to have 5 more columns available for the code excerpt at the end of lines.

Alright, I think I did what you asked for.

Sorry, it took me a while to come back to this. I renamed the syntax to rootglob.

IIRC, listkeys is a newer command than lookup. If the peer doesn't support listkeys, I suspect this batch query would fail. In that case, maybe listkeys has to be skipped if the peer doesn't support it and if --bookmark is not specified.

I'd prefer not to have unpredictable aborts where, when they happen, the solution is "try again". A year or two ago, the "remote heads changed during push" errors were like that, and they were annoying.

I went this way in part because it's fairly simple change, and might be useful in other circumstances (to interpret pulling a tag as actually pulling the tag itself perhaps, though I don't know concretely how that would work).

Ok. I added the b'', as this change created a small conflict later.

Thanks. FWIW, re2 options allow to choose an encoding which is latin1 or utf8 (https://github.com/google/re2/blob/master/re2/re2.h#L609). Presumably latin1 means that "." matches a byte, which would seem more compatible with re, but python bindings don't provide the ability to choose this encoding.

will be subsumed by a more thorough change

I actually ran the tests, but didn't see the style error, because the test in question is among the several ones that never pass for me. Fixed now.

In this case, we'll have to load 'bookmarks' if --bookmarks is set. I have
no idea if default and not ui.quiet should be handled in the same way.
Can you send a followup?