From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web11.2109.1578307318225982311 for ; Mon, 06 Jan 2020 02:41:58 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=irjzgvIs; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1578307317; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=E4JLpHIJy/oH0lxpZELdsJqzgRhfoSygQ4pVJ6TEb7U=; b=irjzgvIsiKn0Kc7unOSH3kXnH3VVgPcG6yDlYz5FI40hSb95omQ6rbt1Z6NbwGIWhT4Llg 4WaLrhK5uZm5W64mkYq6e1cGKmCEPYKp/JuEfuXh9ESiMarGmn//LK47FR8TlUvubwT1MD 5BGuNm29lo/3zUnqXawxC1PwvWUVGr4= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-387--JT-hClqMWux3rmFm29dVw-1; Mon, 06 Jan 2020 05:41:55 -0500 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id AD4FC477; Mon, 6 Jan 2020 10:41:54 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-24.ams2.redhat.com [10.36.117.24]) by smtp.corp.redhat.com (Postfix) with ESMTP id B1D3560E1C; Mon, 6 Jan 2020 10:41:50 +0000 (UTC) Subject: Re: [PATCH v3 00/32] .mailmap: Add mailmap file to have a cleaner git history To: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , devel@edk2.groups.io Cc: Leif Lindholm , Michael D Kinney , Andrew Fish References: <20200102192345.14600-1-philmd@redhat.com> <07a65a33-8b5c-6479-90f6-c0ef8cd15c9a@redhat.com> <69a63257-d4b7-a715-2db2-457376dd45de@redhat.com> From: "Laszlo Ersek" Message-ID: <5f6d5901-9c19-f5b2-08ec-f7482d868b6b@redhat.com> Date: Mon, 6 Jan 2020 11:41:49 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <69a63257-d4b7-a715-2db2-457376dd45de@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-MC-Unique: -JT-hClqMWux3rmFm29dVw-1 X-Mimecast-Spam-Score: 0 Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 01/03/20 14:56, Philippe Mathieu-Daud=C3=A9 wrote: > On 1/3/20 2:48 PM, Laszlo Ersek wrote: >> Apparently, there is a patch that >> prevents the series from being merged? Which one? > > This comment was about patch 13/32, but Jiewen Yao already reviewed > it, OK, thanks. > so this series is fully reviewed by each developer. I've now tried applying this series locally (for a github.com pull request). But git-am fails, on patch#6 ("corrupted patch"). I've checked the patch email (as saved from Thunderbird) and I can see it uses quoted-printable transfer encoding. That has basically never worked with edk2. Can you please send a v4, with 8-bit encoding? Please see: [edk2-devel] [PATCH] BaseTools/Scripts: Add sendemail.transferEncoding to SetupGit.py https://edk2.groups.io/g/devel/message/52323 http://mid.mail-archive.com/02A34F284D1DA44BB705E61F7180EF0AB5C64C6F@ORSM= SX114.amr.corp.intel.com >> Can you please monitor the remaining feedback for this series? If >> everyone is happy with the v3 updates, and you determine that the >> patch set is fully reviewed, I can help pick up the new tags, and >> merge the series. > > Since the change is a one-line, I'd prefer avoid respining the whole > series. I think you mean the change is a one-liner *for a commit message* (namely, Jiewen's R-b). I would normally agree (I'd just pick up that R-b from the list), but again, I can't apply the v3 patches in the first place. > IMHO the series is ready for step #4 "stewards go through the backlog, > and decide one by one whether, for person X, the proposed mappings are > cross-domain or not, and ACK or NACK" which I don't really get. > I suppose this will be resolved in the open on the mailing list, so > individuals will get notified their patch got refused. Let me quote the original suggestion in-line, from here: https://edk2.groups.io/g/devel/message/51906 http://mid.mail-archive.com/9e212952-a114-57d0-9880-6686edc4d8ae@redhat.c= om On 12/06/19 13:23, Laszlo Ersek wrote: > - Collect feedback (from individual reviewers / contributors) for one > or two weeks > > - Accept cross-domain mappings in any patch where the person subject > to the patch is ACKing the patch from the mapped-to (i.e. "current") > email address. This is now a *lot* easier to do, because the patch > series is structured by person. > > - After one or two weeks, push all the patches that have been approved > like explained above. This is where we are now. We're about to push the patches with the direct contributor approvals. This covers 32 patches (v3) out of your original 105 (v2). > - Repost the rest of the series, CC'd solely to the stewards. We will > then have to go through the backlog, and decide one by one whether, > for person X, the proposed mappings are cross-domain or not, and ACK > or NACK accordingly. This is "step 4", and for that, you should please post the *rest* of the patches: approximately 105-32 =3D 73 patches. That is the "backlog". The idea is that the stewards should review those, because we should be permitted to add email mappings even for such contributors that do not personally comment on the proposed mappings, as long as those mappings are not cross-domain. > > - Push the second round, drop the rest for good. And here the "rest" (to be dropped for good) stands for such patches that even the stewards don't approve (because the mappings appear cross-domain, which cannot be approved without direct contributor feedback). Thanks! Laszlo