From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS27357 104.130.224.0/20 X-Spam-Status: No, score=-3.6 required=3.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD, SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from cloud.peff.net (cloud.peff.net [104.130.231.41]) by dcvr.yhbt.net (Postfix) with SMTP id BBFEE1F859 for ; Tue, 16 Aug 2016 17:20:25 +0000 (UTC) Received: (qmail 20722 invoked by uid 109); 16 Aug 2016 17:20:25 -0000 Received: from Unknown (HELO peff.net) (10.0.1.2) by cloud.peff.net (qpsmtpd/0.84) with SMTP; Tue, 16 Aug 2016 17:20:25 +0000 Received: (qmail 28653 invoked by uid 111); 16 Aug 2016 17:20:26 -0000 Received: from sigill.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.7) by peff.net (qpsmtpd/0.84) with SMTP; Tue, 16 Aug 2016 13:20:26 -0400 Received: by sigill.intra.peff.net (sSMTP sendmail emulation); Tue, 16 Aug 2016 13:20:22 -0400 Date: Tue, 16 Aug 2016 13:20:22 -0400 From: Jeff King To: Junio C Hamano Cc: Stefan Beller , meta@public-inbox.org, "git@vger.kernel.org" , Johannes Schindelin , Eric Wong Subject: Re: Working with public-inbox.org [Was: [PATCH] rev-parse: respect core.hooksPath in --git-path] Message-ID: <20160816172022.juqvfn5vwt2pdcyd@sigill.intra.peff.net> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: List-Id: On Tue, Aug 16, 2016 at 10:10:42AM -0700, Junio C Hamano wrote: > Stefan Beller writes: > > > So as a discussion starter: > > * Should git am skip a patch 00/XX automatically ? > > No. My preference is to add "--initial-skip=", though. > > When I receive a patch series to reroll another series, I somehow > know and verify that earlier N patches have not changed, I detach > the HEAD at the last unchanged commit from the previous round and > apply the remainder of the new series, so that I can preserve the > author timestamps of earlier steps from the previous series. By > the time I "know and verify" where the first step that was updated, > I have a full series in a single mbox; having "--initial-skip=" > would help with that use case, too, and "skipping the first" is a > narrow special case of giving N=1. For my workflow, it is not about "initial skip", but rather just "skip emails that don't have patches in them at all". My MUA makes it easy to tag a whole thread (or subthread), cover letter and discussion included, and then dump it all to git-am. And I think that would be the same for a public-inbox workflow (if it learns to grab sub-threads; otherwise you end up with earlier iterations of the series attached to the same thread). That is solving a different problem than you, though, where you want to skip actual patches because you know they are unchanged. -Peff