From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.9 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id 2400F1F5AD for ; Thu, 9 Apr 2020 18:24:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726627AbgDISY4 (ORCPT ); Thu, 9 Apr 2020 14:24:56 -0400 Received: from cloud.peff.net ([104.130.231.41]:38420 "HELO cloud.peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1726582AbgDISY4 (ORCPT ); Thu, 9 Apr 2020 14:24:56 -0400 Received: (qmail 20503 invoked by uid 109); 9 Apr 2020 18:24:56 -0000 Received: from Unknown (HELO peff.net) (10.0.1.2) by cloud.peff.net (qpsmtpd/0.94) with SMTP; Thu, 09 Apr 2020 18:24:56 +0000 Authentication-Results: cloud.peff.net; auth=none Received: (qmail 21193 invoked by uid 111); 9 Apr 2020 18:35:26 -0000 Received: from coredump.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.2) by peff.net (qpsmtpd/0.94) with (TLS_AES_256_GCM_SHA384 encrypted) ESMTPS; Thu, 09 Apr 2020 14:35:26 -0400 Authentication-Results: peff.net; auth=none Date: Thu, 9 Apr 2020 14:24:55 -0400 From: Jeff King To: Derrick Stolee Cc: Junio C Hamano , Philip Oakley , Derrick Stolee via GitGitGadget , git@vger.kernel.org, me@ttaylorr.com, Derrick Stolee , "brian m. carlson" , Johannes Schindelin Subject: Re: [PATCH] revision: --include-diversions adds helpful merges Message-ID: <20200409182455.GA4166316@coredump.intra.peff.net> References: <74562b04-b1ce-cad4-da18-4af030a3dc29@gmail.com> <20200408200526.GA2270445@coredump.intra.peff.net> <9eab6c60-3c99-d6ac-9206-02ce37f7a479@gmail.com> <91fafa17-c58f-dcd0-320d-48c4512561f6@gmail.com> <7d84fc45-786f-7a2e-5889-ef26916627ef@iee.email> <63667043-dbca-0092-209d-865c2f60efc8@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <63667043-dbca-0092-209d-865c2f60efc8@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Thu, Apr 09, 2020 at 01:20:57PM -0400, Derrick Stolee wrote: > In conclusion, I think "--show-pulls" provides the right context for these > extra merges to show in the history view. It also roots these merges in a > Git-native name (that also happens to evoke the "pull request" concept that > is _not_ native to Git). > > What do you think? Yeah, after reading more of the thread, I think the simplest way to think about is "keep merges that pulled in something" with the implication of "(even if the other side didn't touch anything)". And "something you pulled" is a sensible way to think of that. So --show-pulls makes sense to me. Or if we really want to tie it in to simplification, --no-simplify-pulls. But that's more awkward to type, and none of the existing simplification options use the word simplify. ;) -Peff