Re: [PATCH v2 4/6] transport-helper: warn when refspec is not used
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Apr 18, 2013, 19:12 UTC
- Message-ID
- <CAMP44s3329nhkpekg4szbYZBuKUNmVScx1uFWaNm+Xg_0=s86w@mail.gmail.com>
- In-Reply-To
- <7vvc7j6917.fsf@alter.siamese.dyndns.org>
On Thu, Apr 18, 2013 at 12:30 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 20 quoted lines
> Felipe Contreras <felipe.contreras@gmail.com> writes: > >> For the modes that need it. In the future we should probably error out, >> instead of providing half-assed support. >> >> The reason we want to do this is because if it's not present, the remote >> helper might be updating refs/heads/*, or refs/remotes/origin/*, >> directly, and in the process fetch will get confused trying to update >> refs that are already updated, or older than what they should be. We >> shouldn't be messing with the rest of git. > > So that answers my question in the response to an earlier one in > this series. We expect the ref updates to be done by the fetch or > push that drives the helper, and do not want the helper to interfere > with its ref updates. > > So it is not just 'refspec' _allows_ the refs to be constrained to a > private namespace, like the earlier updates made the documentation > say; it _is_ mandatory to use refspecs to constrain them to avoid > touching refs/heads and refs/remotes namespace.
Yeah, it was not stated that it was mandatory, but I think it's in everybody's best interest that it is.
-- Felipe Contreras