Re: [PATCH 3/3] commit-graph: error out on invalid commit oids in 'write --stdin-commits'
- From
Jeff King <peff@peff.net>
- Date
- Apr 3, 2020, 23:10 UTC
- Message-ID
- <20200403231021.GA672258@coredump.intra.peff.net>
- In-Reply-To
- <20200403204013.GC7859@szeder.dev>
On Fri, Apr 03, 2020 at 10:40:13PM +0200, SZEDER Gábor wrote:
Show 13 quoted lines
> > Do you care about complaining about:
> >
> > git rev-parse HEAD^{tree} | git commit-graph write --stdin-commits
> >
> > ? That's the case that's much more interesting, I think.
>
> Hm, are you trying to go in the direction where '--stdin-commits'
> would keep erroring out on any non-full-hex-oid, but would accept and
> silently ignore any hex oids that are not commits (perhaps even when
> there is no such object, dunno)? I think that would support the use
> cases you mentioned, while it would still save me when I do the 'echo
> <ref>' thing (somehow I regularly do that, remember doing it the day
> before yesterday!).Yes, exactly. The case you care about and the case I care about are different ones, so there's no inherent conflict between them.
-Peff