Re: [PATCH] tag: Implicitly supply --list given another list-like option
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 12, 2017, 03:00 UTC
- Message-ID
- <xmqqa88rw656.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <xmqqefy3w6k0.fsf@gitster.mtv.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 7 quoted lines
> I haven't looked at the patch text of this one closely yet, but I > think the goals of both make sense, so we would eventually want to > have them both. > ... > So perhaps you would want this applied first, so that existing three > can already benefit from "implicit --list" before waiting for the > other one?
Ah, I see that you already have the other topic polished to v4 (sorry, but I am a bit behind the list traffic). If it makes your life easier, it is OK to make the current three to four by adding "--no-contains" first and then make all/any of them imply "--list" on top.
IOW, either is fine. Personally, I have a suspicion that sending both at the same time, expecting the maintainer to be able to resolve the potential conflicts correctly, would also be fine ;-)
Thanks.