From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by mx.groups.io with SMTP id smtpd.web11.142448.1680728064755490751 for ; Wed, 05 Apr 2023 13:54:25 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm3 header.b=cpdH/bq+; spf=pass (domain: bsdio.com, ip: 66.111.4.27, mailfrom: rebecca@bsdio.com) Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 06B885C0065 for ; Wed, 5 Apr 2023 16:54:22 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Wed, 05 Apr 2023 16:54:22 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :content-transfer-encoding:content-type:content-type:date:date :from:from:in-reply-to:message-id:mime-version:reply-to:sender :subject:subject:to:to; s=fm3; t=1680728062; x=1680814462; bh=sD 30LqFYNiXuGF0QBHdOU7U44qkhMcl/Q0Vu6lWnRIU=; b=cpdH/bq+yRB1Xyd+iG 58JNiiXCTqv8OTSBAaZrvsBVli5Nw+WPiHrbVh/DohJOGg3DGQ6JYoHsZcnlfRLw Sdi2+JgL5A8x6vakiTw9cHCvzqTKGW03+Ff6x/rpZW20FPd6YHg2W++O5r+shftW XresIyLciOSm3goBM8jix7+6lT3PD+n7c7xilacM147T0JnugbRkxHBPtqq29jDv PFOE/86POlodsrlj6aOO0irmCo6wXPDEyZyzFxqrd1pe4meVyUPAM0HPLinDvhNb G0/eaPb9zbo7hY0qL7Cfx/IE2ezxqS85w9ZPkgy49oIGXIaschdW66rSWsIpVv9v NCAQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:message-id:mime-version:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1680728062; x=1680814462; bh=sD30LqFYNiXuG F0QBHdOU7U44qkhMcl/Q0Vu6lWnRIU=; b=oyqCVKGUcjbPfCDYUhkz7iOqHFrst W0tgTg+zC16+NVPNbNgwtRia2ODIuCZq+jISCKEG2NFcx8TnqF5t0t69wblQQypL /0ksWutkF5n4PV1vgpKIT1BgJ9J6q3nHlrHrdsluaUa49uAg7OKQ5pemIGDjv05n s8Cg0hWmL7wyWLYY5O118Xf8VboT3WaT6Gvp8Rc1ZMrenmsSLzquCxuesb6S1sIF DJuL10qLzMYwM5LAbsBMygcoqaGG+ioFgg1Ko5ooP/oo0z1sZ5f5czZLnToCjfx7 9raNRqdL++tECAqKhMlGR/aSlOXMpL6hsZ/0FNuwPqgK3M1YHSm5m1Nyw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrvdejuddgudehiecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecuogfuuhhsphgvtghtffhomhgrihhnucdlgeelmd enucfjughrpefkffggfgfvhffutgfgsehtkeertddtfeejnecuhfhrohhmpeftvggsvggt tggrucevrhgrnhcuoehrvggsvggttggrsegsshguihhordgtohhmqeenucggtffrrghtth gvrhhnpeetgeehgeeluefgheehjedufeevvdekveefgfeiveffteefhfelgfetffekjedt geenucffohhmrghinhepghhithhhuhgsrdgtohhmpdhgihhthhhusgdrihhonecuvehluh hsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomheprhgvsggvtggtrges sghsughiohdrtghomh X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA for ; Wed, 5 Apr 2023 16:54:21 -0400 (EDT) Message-ID: <1d84af79-cd13-3d6a-1406-01f5206bb928@bsdio.com> Date: Wed, 5 Apr 2023 14:54:20 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.9.1 To: "devel@edk2.groups.io" From: "Rebecca Cran" Subject: RFC: Adding an alias to format patches with --stat= and --stat-graph-width= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit I've noticed several patches recently which has ellipses at the start of the diffstat filenames. When I started contributing to edk2 I remember being asked to always use --stat=1000 --stat-graph-width=20 to avoid that problem: it's mentioned in https://github.com/tianocore/tianocore.github.io/wiki/Laszlo%27s-unkempt-git-guide-for-edk2-contributors-and-maintainers. I was wondering if we might want to introduce an alias via SetupGit.py to add those parameters automatically? For example: diff --git a/BaseTools/Scripts/SetupGit.py b/BaseTools/Scripts/SetupGit.py index 91814199bf..901d2025ab 100644 --- a/BaseTools/Scripts/SetupGit.py +++ b/BaseTools/Scripts/SetupGit.py @@ -82,6 +82,7 @@ OPTIONS = [      {'section': 'sendemail',   'option': 'chainreplyto', 'value': False},      {'section': 'sendemail',   'option': 'thread', 'value': True},      {'section': 'sendemail',   'option': 'transferEncoding', 'value': '8bit'}, +    {'section': 'alias',       'option': 'fp', 'value': 'format-patch -M --stat=1000 --stat-graph-width=20'},^M      ] -- Rebecca Cran