From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.132.183.28; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 1F14E2117FD47 for ; Tue, 6 Nov 2018 06:39:00 -0800 (PST) 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 mx1.redhat.com (Postfix) with ESMTPS id D9DC03D3; Tue, 6 Nov 2018 14:38:59 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-95.rdu2.redhat.com [10.10.120.95]) by smtp.corp.redhat.com (Postfix) with ESMTP id 14C6960C46; Tue, 6 Nov 2018 14:38:56 +0000 (UTC) To: "Fu, Siyuan" , Leif Lindholm Cc: "edk2-devel@lists.01.org" References: <20181030073328.62924-1-siyuan.fu@intel.com> <20181030105036.sz6opnxgvt4amg75@bivouac.eciton.net> <20181031105911.wr3x4crlu2fnfcti@bivouac.eciton.net> From: Laszlo Ersek Message-ID: Date: Tue, 6 Nov 2018 15:38:50 +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: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Tue, 06 Nov 2018 14:38:59 +0000 (UTC) Subject: Re: [PATCH v1 0/7] Delete TCP, PXE, iSCSI driver in MdeModulePkg. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Nov 2018 14:39:01 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 11/06/18 01:59, Fu, Siyuan wrote: > Hi, Leif and Laszlo > > >> -----Original Message----- >> From: Laszlo Ersek [mailto:lersek@redhat.com] >> Sent: Tuesday, November 6, 2018 6:12 AM >> To: Leif Lindholm ; Fu, Siyuan >> >> Cc: edk2-devel@lists.01.org >> Subject: Re: [edk2] [PATCH v1 0/7] Delete TCP, PXE, iSCSI driver in >> MdeModulePkg. >> >> Hi Leif, >> >> On 10/31/18 11:59, Leif Lindholm wrote: >>> Hi Fu Siyuan, >>> >>> Hmm, strange. >>> >>> Ming Huang (cc) has also had issues with this in his latest set (but >>> not before). Can you guys work together to try to figure out what is >>> going wrong? >>> >>> Can you start by testing with an older version of git? >> >> this is indeed a git regression. I ran into it myself recently, when I >> upgraded git from v2.14.1 to v2.19.1. I dug a little into the git source >> code, but couldn't spend much time on it. Ultimately I rolled back the >> upgrade. >> >> The diffstats are still well-formed (not truncated) in the individual >> patch emails, under v2.19.1, as long as the "recommended" options are >> used on the cmdline. However, the cumulative diffstat is truncated >> regardless. I couldn't convince git v2.19.1 to keep the previous >> (desired) behavior. > > So should I rollback my Git to v2.14.1, or the new truncated cumulative > diffstat is also acceptable? Leif identified the culprit git patch as commit 43662b23abbd ("format-patch: keep cover-letter diffstat wrapped in 72 columns", 2018-01-25), and that commit was part of release v2.17.0. Therefore, if you could roll back to v2.16.5 (the latest stable release of v2.16), that should mitigate this issue. (I haven't checked whether running upstream v2.16.* binaries is advisable from a security perspective.) Thanks Laszlo