[PATCH 1 of 6 path intent] ui: add an intent to expandpath
Mads Kiilerich
mads at kiilerich.com
Tue Sep 30 11:53:33 CDT 2014
On 09/30/2014 02:30 AM, Gregory Szorc wrote:
> outgoing is read-only despite the association with push. In Mozilla's
> use case, only people with push access (a subset) have access to the
> push URL (SSH). Everybody has access to the HTTP endpoint. People
> without SSH access should still be able to use outgoing. So using the
> push/SSH URL for outgoing would be wrong in our case, even though it
> is logically the most appropriate URL to use.
I think you put it backwards and describe your current setup as a
requirement.
Yes, outgoing is a read-only (and very cheap - especially when using
ssh) subset of push. It is intended to show exactly what will be pushed.
Trying to treat them differently would be a confusing misfeature. I
don't think Mercurial itself should do anything to encourage that
misfeature - quite the opposite.
In the setup you describe, people with push access should use ssh for
outgoing, just like they do for push. People without push access should
just use http for everything.
> To further complicate matters, our HTTP layer is currently rejecting
> some Mercurial HTTP requests because the encoding of 100+ heads during
> discovery is exceeding HTTP header limits.
We only see that problem when people have full clones with all the
gazillion of heads that they never will have any use for. Do you support
that case or do you see it in other cases as well?
/Mads
More information about the Mercurial-devel
mailing list