Re: [PATCH] ls-remote: create option to sort by versions
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 2, 2018, 18:32 UTC
- Message-ID
- <xmqqbmf1zcfb.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <87605aw1ue.fsf@evledraar.gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 5 quoted lines
> This is a sensible thing to want, but why not follow the UI we have for > this with git-tag? I.e. --sort=<key> & -i (or --ignore-case)? Of course > ls-remote doesn't just show tags, so maybe we'd want --tag-sort=<key> > and --ignore-tag-case or something, but the rest should be equivalent, > no?
Yeah, and if we can reuse more of ref-filter.c machinery (which was factored out of for-each-ref and tag you suggested borrows from), that would be even better. In the context of ls-remote, however, we cannot inspect the object (we typically do not have them yet), so it may not be practical, but I agree with your suggestion---we should match the behaviour at the UI level at least when we can.