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: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.8 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id D797C1FD99 for ; Wed, 10 Aug 2016 19:31:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933600AbcHJTbN (ORCPT ); Wed, 10 Aug 2016 15:31:13 -0400 Received: from relay4-d.mail.gandi.net ([217.70.183.196]:49706 "EHLO relay4-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933195AbcHJTbK (ORCPT ); Wed, 10 Aug 2016 15:31:10 -0400 Received: from x (unknown [IPv6:2605:e000:d5ce:1e00:c5f5:b7bf:c731:5d3f]) (Authenticated sender: josh@joshtriplett.org) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id 7A9581720A4; Wed, 10 Aug 2016 21:31:01 +0200 (CEST) Date: Wed, 10 Aug 2016 09:30:58 -1000 From: Josh Triplett To: Jakub =?utf-8?B?TmFyxJlic2tp?= Cc: Eric Wong , Michael Haggerty , Johannes Schindelin , Stefan Beller , Junio C Hamano , Git Mailing List , Eric Sunshine , Jeff King , Johannes Sixt , Duy Nguyen , Richard Ipsum , Lars Schneider , Philip Oakley Subject: Re: patch submission process, was Re: [PATCH v6 06/16] merge_recursive: abort properly upon errors Message-ID: <20160810193057.s36wfcivlfm3xmh2@x> References: <6c937f79-2b82-619d-51fe-adccbe09bd66@alum.mit.edu> <20160809182800.GA19044@dcvr> <20160810005548.gee6ontd33ck5vej@x> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.6.2-neo (2016-07-23) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Wed, Aug 10, 2016 at 09:30:01AM +0200, Jakub Narębski wrote: > On 10 August 2016 at 02:55, Josh Triplett wrote: > > On Tue, Aug 09, 2016 at 06:28:00PM +0000, Eric Wong wrote: > >> Some of these problems I hope public-inbox (or something like > >> it) can fix and turn the tide towards email, again. > > > > This really seems like the dichotomy that drives people towards central > > services like GitHub or GitLab. We need an alternative that doesn't > > involve email, or at the very least, doesn't require people to use email > > directly. Half of the pain in the process comes from coaxing email > > clients that don't treat mail text as sacrosanct to leave it alone and > > not mangle it. (Some of that would go away if we accepted attachments > > with inline disposition, but not all of it. All of it would go away if > > the submission process just involved "git push" to an appropriate > > location.) > > But submission is less important than review. And for review it is > usually better (except gigantic series) to have patch text for review > with the review. Agreed. However, submission typically requires more work than review, because the patch text must remain applicable. For review, as long as the email client you use to respond doesn't do something horrible like *re-wrap* the quoted patch text, the result will work as a review. Ideally, I'd love to see 1) a review UI that stores line-by-line reviews into a common format and can translate those to email, and 2) a mechanism to translate reviews written by email and quoting into the review format and store them with the repository. > And (meta)-versioning of series. I've got a documented format for that. :) > And place for proof-of-concept / weather-balon patches... Same place as all other patches, just with an "RFC" tag on them.