From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web10.5880.1576638075207076350 for ; Tue, 17 Dec 2019 19:01:15 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: nathaniel.l.desimone@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Dec 2019 19:01:14 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,327,1571727600"; d="scan'208";a="390041131" Received: from orsmsx102.amr.corp.intel.com ([10.22.225.129]) by orsmga005.jf.intel.com with ESMTP; 17 Dec 2019 19:01:12 -0800 Received: from orsmsx122.amr.corp.intel.com (10.22.225.227) by ORSMSX102.amr.corp.intel.com (10.22.225.129) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 17 Dec 2019 19:01:12 -0800 Received: from orsmsx114.amr.corp.intel.com ([169.254.8.106]) by ORSMSX122.amr.corp.intel.com ([169.254.11.3]) with mapi id 14.03.0439.000; Tue, 17 Dec 2019 19:01:12 -0800 From: "Nate DeSimone" To: "devel@edk2.groups.io" CC: "Feng, Bob C" , "Gao, Liming" , Leif Lindholm Subject: Re: [edk2-devel] [PATCH] BaseTools/Scripts: Add sendemail.transferEncoding to SetupGit.py Thread-Topic: [edk2-devel] [PATCH] BaseTools/Scripts: Add sendemail.transferEncoding to SetupGit.py Thread-Index: AQHVrGEr3fv/rHRdhUSiZoFmN8urhae/Rehw Date: Wed, 18 Dec 2019 03:01:11 +0000 Message-ID: <02A34F284D1DA44BB705E61F7180EF0AB5C64C6F@ORSMSX114.amr.corp.intel.com> References: <15DDDAE6FE3C410D.8264@groups.io> In-Reply-To: <15DDDAE6FE3C410D.8264@groups.io> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiM2FjMGEyZDItOTBhZS00MjZhLTk0NjYtOTFmYmEyMGJkYjFlIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiOEd0XC83a21HWWdFVDBveGlPVDY0WTk4eHc4M1wvWjU1RHVBeFJiVE9aVGF4bE1nRHMwQjZHNWdPVWtOREZsUEpHIn0= x-ctpclassification: CTP_NT x-originating-ip: [10.22.254.138] MIME-Version: 1.0 Return-Path: nathaniel.l.desimone@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Any chance I could get a code review on this? Thanks, Nate -----Original Message----- From: devel@edk2.groups.io On Behalf Of Nate DeSimo= ne Sent: Friday, December 6, 2019 10:15 AM To: devel@edk2.groups.io Cc: Feng, Bob C ; Gao, Liming = ; Leif Lindholm Subject: [edk2-devel] [PATCH] BaseTools/Scripts: Add sendemail.transferEnc= oding to SetupGit.py If git finds a '\r' character in the message, then it converts the entire = message content into Quoted-Printable encoding. It appears that when groups= .io converts the QP encoding back to text format, the '\r' characters someh= ow become '\n'. To workaround this, the SetupGit.py script will now explici= tly set the sendemail.transferEncoding git config option to '8bit' Cc: Bob Feng Cc: Liming Gao Cc: Leif Lindholm Signed-off-by: Nate DeSimone --- BaseTools/Scripts/SetupGit.py | 38 ++++++++++++++++++----------------- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git a/BaseTools/Scripts/SetupGit.py b/BaseTools/Scripts/SetupGit.py= index 3d39d3b35f..514f1c4d42 100644 --- a/BaseTools/Scripts/SetupGit.py +++ b/BaseTools/Scripts/SetupGit.py @@ -2,6 +2,7 @@ # Set up the git configuration for contributing to TianoCore projects #= # Copyright (c) 2019, Linaro Ltd. All rights reserved.
+# Copyright (c) 2019, Intel Corporation. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -53,29 +54,30 @@ MI= N_GIT_VERSION =3D (1, 9, 0) =20 # Set of options to be set identically for all repositories OPTIONS =3D = [ - {'section': 'am', 'option': 'keepcr', 'value': True}= , - {'section': 'am', 'option': 'signoff', 'value': True}= , - {'section': 'cherry-pick', 'option': 'signoff', 'value': True}= , - {'section': 'color', 'option': 'diff', 'value': True}= , - {'section': 'color', 'option': 'grep', 'value': 'auto= '}, - {'section': 'commit', 'option': 'signoff', 'value': True}= , - {'section': 'core', 'option': 'abbrev', 'value': 12}, + {'section': 'am', 'option': 'keepcr', 'value': Tr= ue}, + {'section': 'am', 'option': 'signoff', 'value': Tr= ue}, + {'section': 'cherry-pick', 'option': 'signoff', 'value': Tr= ue}, + {'section': 'color', 'option': 'diff', 'value': Tr= ue}, + {'section': 'color', 'option': 'grep', 'value': 'a= uto'}, + {'section': 'commit', 'option': 'signoff', 'value': Tr= ue}, + {'section': 'core', 'option': 'abbrev', 'value': 12= }, {'section': 'core', 'option': 'attributesFile', 'value': os.path.join(CONFDIR, 'gitattributes').replace('\\', '/')}, - {'section': 'core', 'option': 'whitespace', 'value': 'cr-a= t-eol'}, - {'section': 'diff', 'option': 'algorithm', 'value': 'pati= ence'}, + {'section': 'core', 'option': 'whitespace', 'value': 'c= r-at-eol'}, + {'section': 'diff', 'option': 'algorithm', 'value': 'p= atience'}, {'section': 'diff', 'option': 'orderFile', 'value': os.path.join(CONFDIR, 'diff.order').replace('\\', '/')}, - {'section': 'diff', 'option': 'renames', 'value': 'copi= es'}, - {'section': 'diff', 'option': 'statGraphWidth', 'value': '20'}= , - {'section': 'diff "ini"', 'option': 'xfuncname', + {'section': 'diff', 'option': 'renames', 'value': 'c= opies'}, + {'section': 'diff', 'option': 'statGraphWidth', 'value': '2= 0'}, + {'section': 'diff "ini"', 'option': 'xfuncname', 'value': '^\\\\[[A-Za-z0-9_., ]+]'}, - {'section': 'format', 'option': 'coverLetter', 'value': True}= , - {'section': 'format', 'option': 'numbered', 'value': True}= , - {'section': 'format', 'option': 'signoff', 'value': False= }, - {'section': 'notes', 'option': 'rewriteRef', 'value': 'refs= /notes/commits'}, - {'section': 'sendemail', 'option': 'chainreplyto', 'value': False= }, - {'section': 'sendemail', 'option': 'thread', 'value': True}= , + {'section': 'format', 'option': 'coverLetter', 'value': Tr= ue}, + {'section': 'format', 'option': 'numbered', 'value': Tr= ue}, + {'section': 'format', 'option': 'signoff', 'value': Fa= lse}, + {'section': 'notes', 'option': 'rewriteRef', 'value': 'r= efs/notes/commits'}, + {'section': 'sendemail', 'option': 'chainreplyto', 'value': Fa= lse}, + {'section': 'sendemail', 'option': 'thread', 'value': Tr= ue}, + {'section': 'sendemail', 'option': 'transferEncoding', 'value': '8= bit'}, ] =20 =20 -- 2.24.0.windows.2