From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mx.groups.io with SMTP id smtpd.web12.12826.1593514729029071500 for ; Tue, 30 Jun 2020 03:58:49 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=JSsH8Pwq; spf=pass (domain: intel.com, ip: 192.55.52.115, mailfrom: bob.c.feng@intel.com) IronPort-SDR: VhTs/BicuBZBAQx44lfpqJSAmJKaBy/kXvdXxWnkf61kvsMal4HbcTXxVzoxksK194FopHMm3z bcdOuRJW8brA== X-IronPort-AV: E=McAfee;i="6000,8403,9666"; a="145271444" X-IronPort-AV: E=Sophos;i="5.75,297,1589266800"; d="scan'208";a="145271444" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jun 2020 03:58:48 -0700 IronPort-SDR: SWHYqzC18+uj1C3CWlNX7uOmew9Bc7WtoPCNTXFvWcpdSfsNwLBn8xsh7dPys5X+KXhlVj+ccx ARJRFeo+y7Qw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,297,1589266800"; d="scan'208";a="265085403" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by fmsmga007.fm.intel.com with ESMTP; 30 Jun 2020 03:58:48 -0700 Received: from FMSEDG002.ED.cps.intel.com (10.1.192.134) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 30 Jun 2020 03:58:47 -0700 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.169) by edgegateway.intel.com (192.55.55.69) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 30 Jun 2020 03:58:47 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fNv/10CIrce0a7Ths1bDrOz3SDmABtjcNslOXkRKDw/g1Xnx9iFzT/hnq/cIJGytduwVo8vseJhxbdm21Fc8GVguHbvTTrMiY7d/s/1ra6xnh8GX7Bbp1J+wlUfngIsWZiAabMKiJX/i0AC8PVS7mdNvYIMDBAC1y60dSm9jzpwjQ8dfbNPPRwdojgCIJWrEaflz86+YQD/NQBRniQBgcrpf2BSq7zqZskTkfMMt+BYZ31GdGLdPzH93XK0gXKGjOX8fsfQBx5KmKmNpPWvVfuN0aocG9tWBIjPQx9D1tSjJhFoJT4zqjdFqZwX3mJkp6qKxiwvsjEw4sPoaSnBzNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FuX9+AEObTYEDkPmbfUABsMKkv7utpUXbYUIZ+mQHY0=; b=boOrurAyBecjeyWlmxqGJk/auU4j+A2xK58VO0MKVSuE59yXTYOhZwf7C6uFWybCLwPt75y6QXiuEaitL8ho9pZtjk46qeGRoK8y5d92LnZP+8L9Fbaokym7m3lwSJIudDZg8W3FPesfPe85egpWTARj6JEkFvNJQHHkwdJErPFSZA7mD+szpaLD6eai37uhM4qL2PEL+2Sm/p7Dp3gTgXsXjAVAU1MPZ7KLvPw80HbymElC23068lyN4zuymEFWcCoKAXg0BxHdGxexp5Mlgr129S1r9Pyx5khLtqn02Te4oB/Vp2uorSv3x0ONUiF6Wk3O9CMMUGb21twiG/Xx5A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FuX9+AEObTYEDkPmbfUABsMKkv7utpUXbYUIZ+mQHY0=; b=JSsH8PwqlcvJ7ZwTaU7a7Re/1hxYc0A4Q/G5gZgveHkkeEmJ2OjROEbfGxJqwCCWy3/LCDxAViTVupMcBlfHITZZ9H1rYi6TVnK7k09poAGhCLw5yuKKlR/9jp3WALBWeZ7qGTIRuxw3XrLoM9LY0RnP8tKX8UPSZMBEQiLi+WU= Received: from BN6PR11MB0068.namprd11.prod.outlook.com (2603:10b6:405:69::17) by BN7PR11MB2723.namprd11.prod.outlook.com (2603:10b6:406:b9::25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.21; Tue, 30 Jun 2020 10:58:45 +0000 Received: from BN6PR11MB0068.namprd11.prod.outlook.com ([fe80::6870:82d3:5bde:4153]) by BN6PR11MB0068.namprd11.prod.outlook.com ([fe80::6870:82d3:5bde:4153%6]) with mapi id 15.20.3131.028; Tue, 30 Jun 2020 10:58:45 +0000 From: "Bob Feng" To: "devel@edk2.groups.io" , "pierre.gondois@arm.com" CC: Sami Mujawar , Tomas Pilar , "Gao, Liming" , nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ file Thread-Topic: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ file Thread-Index: AQHWSgcjA7iQ/v72okSjnvFg4182Xajn4DtwgAEyUgCABJT+4IACCKSAgAFVvJA= Date: Tue, 30 Jun 2020 10:58:45 +0000 Message-ID: References: <20200624090837.124392-1-pierre.gondois@arm.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.102.204.45] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: d294d56c-a903-497a-91fd-08d81ce48f6b x-ms-traffictypediagnostic: BN7PR11MB2723: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:10000; x-forefront-prvs: 0450A714CB x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: AibDaigcUfTfQcA6sqSSlytDOL0YmqTIN3Zmvf+neQWbjamFpDj816xoD5uzj9Kb6MYCF8M2Vu0EmmprSg8qS+EX+7zmtAGEd/+znR4oimyxaXm8mhgK47hc3FZmFp1xHbkGd9n4txTs0DiZTFYzavgPuRekqboPnWNm2PcvjiBa65XkF9wW0aYuBadG2xzfBvjhDMMuTBxsbtkpzkAHZyOHVGF8H9AWl+PnyeMoP+xeFCvfTaHeKiQKVgrVEqhs912zSD3lbPqd6nhbVhsWW9J0Gc/IcgbBHFAIaFgMAzorqcpPwgfD9pGTXf0uw//klDbmqglsl+tLKOm90S/3R2Vn68hgqkm0G3SiN7bPpDphhXwd81otsL9BnnxA0wFnXDyADxKmtr1j3T7sXKJx3g== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN6PR11MB0068.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(136003)(39860400002)(366004)(396003)(376002)(346002)(66556008)(64756008)(66476007)(66446008)(55016002)(71200400001)(8676002)(316002)(9686003)(86362001)(8936002)(76116006)(52536014)(478600001)(26005)(186003)(7696005)(54906003)(110136005)(6506007)(53546011)(4326008)(966005)(2906002)(30864003)(66946007)(33656002)(5660300002)(83380400001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: NqCZdsXcAdaJi2ZZYY+uU3k8KQknuNJd8H8j3VCAhbdMluFk80HB3rlrMe3VojT/prfckctD5c1f4tIhk2T6c6Rg8LAZI4jfMwj2dacy4StlKnDLnRj8Xw8uNIrmSrX9TnRMPoLPQ7a0G4KB2wskaDyHVZiex9z/Tbd1U6HgvGmLWhOp5RgpS0RMF9FzOVWM4T/Antqn76W7nCSW+aoP4L5BVXVVXC1FLtc7iZevBlsJKVp66DlmemhS4OROyGRgJZpmFGUGmPkaaweFyxJrYoAPAn5xyBQAqFnVsL/916l+9ofmXl88U4m76KpFS/1SbYnFbcO+GIKqml/5Ho0Tw53VHmSIP2vCZCRAWPd2bJ/YhPFdWWeSVscC3syGIDUYCXaktPFA3q1JKLy6pohOLSZtkYrrKwBKErtQbCVqj4h3u1CtCJS3SRV4KheraY+AUjPhO1YMnxlXPDHdzmUiX3Jd5FGEUsfCsHndYiZkBUboViBblNp0/H2YvkRj40A3 MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN6PR11MB0068.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: d294d56c-a903-497a-91fd-08d81ce48f6b X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Jun 2020 10:58:45.6248 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: eaUJJZuTtlrVVcS/C/w1tylGSzMAW7nU09vQ4IjWGQcEvZMGKE4kEgLKDDqWuuctrA/ys05pnXCpQDwjT3Aizw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR11MB2723 Return-Path: bob.c.feng@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Pierre, I think we need to update the PatchCheck.py script. The comments says, "Do= not enforce CR/LF line endings for linux shell scripts.", but that only ch= eck the file ext is .sh is not enough. if self.filename.endswith('.sh'): # # Do not enforce CR/LF line endings for linux shell sc= ripts. # self.force_crlf =3D False Thanks, Bob -----Original Message----- From: devel@edk2.groups.io On Behalf Of PierreGondo= is Sent: Monday, June 29, 2020 10:33 PM To: Feng, Bob C ; devel@edk2.groups.io Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hello Bob, I actually found more files having a LF line ending by running the followi= ng command: for file in `git ls-files`; do if [ "${file##*.}" !=3D bin ] && [ "${file#= #*.}" !=3D bmp ] && [ "${file##*.}" !=3D a ] && [ "${file##*.}" !=3D lib ] = && [ "${file##*.}" !=3D cer ] && [ "${file##*.}" !=3D pyd ] &&[ "${file##*.= }" !=3D pem ] && [ "${file##*.}" !=3D raw ]; then file $file | grep -v "CRL= F" | grep -v ": directory"; fi; done The command list files referenced in g= it, then excludes files with specific extensions, excludes directories, and= excludes files with CRLF line endings. I believe some of them should have CRLF line endings if this is the defaul= t in edk2. I am mainly thinking about the ".rtf", ".py" and ".txt" files. F= or the actual bash scripts, maybe a ".gitattributes" file can be created in= edk2 to explicitly make them use LF line endings. This can be done by sett= ing this attribute: "eol=3Dlf". This ".gitattributes" file would then be parsed by the PatchCHeck.py scrip= t not to trigger an error for files having the later attribute. Regards, Pierre List of files obtained by running the command above, (maybe it needs some = more filters): BaseTools/Bin/CYGWIN_NT-5.1-i686/BootSectImage: Bourne-Again shell script,= ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/BuildEnv: Bourne-Again shell script, ASCI= I text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/Ecc: Bourne-Again shell script, ASCII tex= t executable BaseTools/Bin/CYGWIN_NT-5.1-i686/EfiLdrImage: Bourne-Again shell script, A= SCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/EfiRom: Bourne-Again shell script, ASCII = text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenCrc32: Bourne-Again shell script, ASCI= I text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenDepex: Bourne-Again shell script, ASCI= I text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenFds: Bourne-Again shell script, ASCII = text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenFfs: Bourne-Again shell script, ASCII = text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenFv: Bourne-Again shell script, ASCII t= ext executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenFw: Bourne-Again shell script, ASCII t= ext executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenPage: Bourne-Again shell script, ASCII= text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenSec: Bourne-Again shell script, ASCII = text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GenVtf: Bourne-Again shell script, ASCII = text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/GnuGenBootSector: Bourne-Again shell scri= pt, ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/LzmaCompress: Bourne-Again shell script, = ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/LzmaF86Compress: Bourne-Again shell scrip= t, ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/RunBinToolFromBuildDir: Bourne-Again shel= l script, ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/RunToolFromSource: Bourne-Again shell scr= ipt, ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/Split: Bourne-Again shell script, ASCII t= ext executable BaseTools/Bin/CYGWIN_NT-5.1-i686/TargetTool: Bourne-Again shell script, AS= CII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/TianoCompress: Bourne-Again shell script,= ASCII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/Trim: Bourne-Again shell script, ASCII te= xt executable BaseTools/Bin/CYGWIN_NT-5.1-i686/VfrCompile: Bourne-Again shell script, AS= CII text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/VolInfo: Bourne-Again shell script, ASCII= text executable BaseTools/Bin/CYGWIN_NT-5.1-i686/build: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/BPDG: Bourne-Again shell script, ASCII tex= t executable BaseTools/BinWrappers/PosixLike/Brotli: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/BrotliCompress: Bourne-Again shell script,= ASCII text executable BaseTools/BinWrappers/PosixLike/DevicePath: Bourne-Again shell script, ASC= II text executable BaseTools/BinWrappers/PosixLike/Ecc: Bourne-Again shell script, ASCII text= executable BaseTools/BinWrappers/PosixLike/EfiRom: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/GenCrc32: Bourne-Again shell script, ASCII= text executable BaseTools/BinWrappers/PosixLike/GenDepex: Bourne-Again shell script, ASCII= text executable BaseTools/BinWrappers/PosixLike/GenFds: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/GenFfs: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/GenFv: Bourne-Again shell script, ASCII te= xt executable BaseTools/BinWrappers/PosixLike/GenFw: Bourne-Again shell script, ASCII te= xt executable BaseTools/BinWrappers/PosixLike/GenPatchPcdTable: Bourne-Again shell scrip= t, ASCII text executable BaseTools/BinWrappers/PosixLike/GenSec: Bourne-Again shell script, ASCII t= ext executable BaseTools/BinWrappers/PosixLike/GenerateCapsule: Bourne-Again shell script= , ASCII text executable BaseTools/BinWrappers/PosixLike/LzmaCompress: Bourne-Again shell script, A= SCII text executable BaseTools/BinWrappers/PosixLike/LzmaF86Compress: Bourne-Again shell script= , ASCII text executable BaseTools/BinWrappers/PosixLike/PatchPcdValue: Bourne-Again shell script, = ASCII text executable BaseTools/BinWrappers/PosixLike/Pkcs7Sign: Bourne-Again shell script, ASCI= I text executable BaseTools/BinWrappers/PosixLike/Rsa2048Sha256GenerateKeys: Bourne-Again sh= ell script, ASCII text executable BaseTools/BinWrappers/PosixLike/Rsa2048Sha256Sign: Bourne-Again shell scri= pt, ASCII text executable BaseTools/BinWrappers/PosixLike/Split: Bourne-Again shell script, ASCII te= xt executable BaseTools/BinWrappers/PosixLike/TargetTool: Bourne-Again shell script, ASC= II text executable BaseTools/BinWrappers/PosixLike/TianoCompress: Bourne-Again shell script, = ASCII text executable BaseTools/BinWrappers/PosixLike/Trim: Bourne-Again shell script, ASCII tex= t executable BaseTools/BinWrappers/PosixLike/UPT: Bourne-Again shell script, ASCII text= executable BaseTools/BinWrappers/PosixLike/VfrCompile: Bourne-Again shell script, ASC= II text executable BaseTools/BinWrappers/PosixLike/VolInfo: Bourne-Again shell script, ASCII = text executable BaseTools/BinWrappers/PosixLike/build: Bourne-Again shell script, ASCII te= xt executable BaseTools/BuildEnv: ASCII text BaseTools/Source/C/VfrCompile/Pccts/KNOWN_PROBLEMS.txt: data BaseTools/Source/C/VfrCompile/Pccts/history.ps: PostScript document text c= onforming DSC level 3.0 BaseTools/Source/Python/Ecc/CParser3/__init__.py: empty BaseTools/Source/Python/Ecc/CParser4/__init__.py: empty BaseTools/Source/Python/Eot/CParser3/__init__.py: empty BaseTools/Source/Python/Eot/CParser4/__init__.py: empty BaseTools/Source/Python/Pkcs7Sign/TestRoot.cer.gEfiSecurityPkgTokenSpaceGu= id.PcdPkcs7CertBuffer.inc: ASCII text, with very long lines, with no line t= erminators BaseTools/Source/Python/Pkcs7Sign/TestRoot.cer.gFmpDevicePkgTokenSpaceGuid= .PcdFmpDevicePkcs7CertBufferXdr.inc: ASCII text, with very long lines, with= no line terminators BaseTools/UserManuals/Build_Utility_Man_Page.rtf: Rich Text Format data, v= ersion 1, unknown character set BaseTools/UserManuals/EfiRom_Utility_Man_Page.rtf: Rich Text Format data, = version 1, unknown character set BaseTools/UserManuals/GenCrc32_Utility_Man_Page.rtf: Rich Text Format data= , version 1, unknown character set BaseTools/UserManuals/GenDepex_Utility_Man_Page.rtf: Rich Text Format data= , version 1, unknown character set BaseTools/UserManuals/GenFds_Utility_Man_Page.rtf: Rich Text Format data, = version 1, unknown character set BaseTools/UserManuals/GenFfs_Utility_Man_Page.rtf: Rich Text Format data, = version 1, unknown character set BaseTools/UserManuals/GenFv_Utility_Man_Page.rtf: Rich Text Format data, v= ersion 1, unknown character set BaseTools/UserManuals/GenFw_Utility_Man_Page.rtf: Rich Text Format data, v= ersion 1, unknown character set BaseTools/UserManuals/GenPatchPcdTable_Utility_Man_Page.rtf: Rich Text For= mat data, version 1, unknown character set BaseTools/UserManuals/GenSec_Utility_Man_Page.rtf: Rich Text Format data, = version 1, unknown character set BaseTools/UserManuals/Intel_UEFI_Packaging_Tool_Man_Page.rtf: Rich Text Fo= rmat data, version 1, unknown character set BaseTools/UserManuals/LzmaCompress_Utility_Man_Page.rtf: Rich Text Format = data, version 1, unknown character set BaseTools/UserManuals/PatchPcdValue_Utility_Man_Page.rtf: Rich Text Format= data, version 1, unknown character set BaseTools/UserManuals/SplitFile_Utility_Man_Page.rtf: Rich Text Format dat= a, version 1, unknown character set BaseTools/UserManuals/TargetTool_Utility_Man_Page.rtf: Rich Text Format da= ta, version 1, unknown character set BaseTools/UserManuals/TianoCompress_Utility_Man_Page.rtf: Rich Text Format= data, version 1, unknown character set BaseTools/UserManuals/Trim_Utility_Man_Page.rtf: Rich Text Format data, ve= rsion 1, unknown character set BaseTools/UserManuals/UtilityManPage_template.rtf: Rich Text Format data, = version 1, unknown character set BaseTools/UserManuals/VfrCompiler_Utility_Man_Page.rtf: Rich Text Format d= ata, version 1, unknown character set BaseTools/UserManuals/VolInfo_Utility_Man_Page.rtf: Rich Text Format data,= version 1, unknown character set CryptoPkg/Library/Include/internal/dso_conf.h: C source, ASCII text CryptoPkg/Library/OpensslLib/process_files.pl: Perl script text executable EmbeddedPkg/Library/FdtLib/TODO: ASCII text EmbeddedPkg/Library/FdtLib/fdt_overlay.c: C source, ASCII text EmulatorPkg/Unix/.gdbinit: ASCII text EmulatorPkg/Unix/GdbRun.sh: ASCII text EmulatorPkg/Unix/Host/X11IncludeHack: ASCII text, with no line terminators EmulatorPkg/Unix/Xcode/xcode_project32/XcodeBuild.sh: Bourne-Again shell s= cript, ASCII text executable EmulatorPkg/Unix/Xcode/xcode_project32/xcode_project.xcodeproj/default.pbx= user: ASCII text EmulatorPkg/Unix/Xcode/xcode_project32/xcode_project.xcodeproj/project.pbx= proj: ASCII text EmulatorPkg/Unix/Xcode/xcode_project64/XcodeBuild.sh: Bourne-Again shell s= cript, ASCII text executable EmulatorPkg/Unix/Xcode/xcode_project64/xcode_project.xcodeproj/default.pbx= user: ASCII text EmulatorPkg/Unix/Xcode/xcode_project64/xcode_project.xcodeproj/project.pbx= proj: ASCII text EmulatorPkg/Unix/lldbinit: ASCII text EmulatorPkg/build.sh: Bourne-Again shell script, ASCII text executable OvmfPkg/QemuVideoDxe/VbeShim.sh: POSIX shell script, ASCII text executable OvmfPkg/build.sh: Bourne-Again shell script, ASCII text executable edksetup.sh: ASCII text -----Original Message----- From: Feng, Bob C Sent: 28 June 2020 08:34 To: devel@edk2.groups.io; Pierre Gondois Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: RE: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hi Pierre, I met a problem when I push your patch set. If I change the CRLF to a unix format EOL, the patch can't pass PatchCheck= .py. If I don't change the CRLF, The build on Linux will fail. So I can't make the patch set pass the CI. Could you share me how did you do to make your PR 729 pass? Thanks, Bob -----Original Message----- From: devel@edk2.groups.io On Behalf Of PierreGondo= is Sent: Thursday, June 25, 2020 5:31 PM To: devel@edk2.groups.io; Feng, Bob C Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hello Bob, I believe the line endings of the BaseTools/BinWrappers/PosixLike/AmlToC s= cript have been modified to CRLF when I sent the patch. I created a pull re= quest from the linked github branch noted in the patches. It is available a= t https://github.com/PierreARM/edk2/commits/803_Compile_AML_bytecode_array_= into_OBJ_file_v3 . The pull request is available here (to show the result o= f the CI tests) https://github.com/tianocore/edk2/pull/729 . Do you want a v4 or is it possible to pull the patches from the github rep= ository? Sorry for the inconvenience. Regards, Pierre -----Original Message----- From: devel@edk2.groups.io On Behalf Of Bob Feng vi= a groups.io Sent: 24 June 2020 16:16 To: devel@edk2.groups.io; Pierre Gondois Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hi Pierre, There are some build failed in OpenCI. Would you please check it? https://github.com/tianocore/edk2/pull/727 Thanks, Bob -----Original Message----- From: devel@edk2.groups.io On Behalf Of PierreGondo= is Sent: Wednesday, June 24, 2020 5:09 PM To: devel@edk2.groups.io Cc: Pierre Gondois ; sami.mujawar@arm.com; tomas.p= ilar@arm.com; Feng, Bob C ; Gao, Liming ; nd@arm.com Subject: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ f= ile Following the BZ at https://bugzilla.tianocore.org/show_bug.cgi?id=3D2425 This patch serie is a another way to solve the dependency of C files over = ASL files. With this new method, the dependency is resolved at the linking = stage. The last method to solve this dependency was to add the possibility to mod= ify INF files to depict such a dependency. This method was not accepted. Th= e discussion is available at https://edk2.groups.io/g/devel/topic/72655342#= 56658 The last patch modifying the INF specification and INF parsing are availab= le at: https://edk2.groups.io/g/devel/topic/72655342#56658 https://edk2.groups.io/g/devel/topic/72656060#56662 Pierre Gondois (4): BaseTools: Generate multiple rules when multiple output files BaseTools: Rename AmlToHex script to AmlToC BaseTools: Compile AML bytecode arrays into .obj file BaseTools: Fix string concatenation BaseTools/BinWrappers/PosixLike/{AmlToHex =3D> AmlToC} | 28= +++---- BaseTools/BinWrappers/WindowsLike/{AmlToHex.bat =3D> AmlToC.bat} | 0 BaseTools/Conf/build_rule.template | 15 += ++- BaseTools/Source/Python/{AmlToHex/AmlToHex.py =3D> AmlToC/AmlToC.py} | 82= ++++++++------------ BaseTools/Source/Python/AutoGen/BuildEngine.py | 2 += - BaseTools/Source/Python/AutoGen/GenMake.py | 6 += + BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 38 += ++++---- 7 files changed, 89 insertions(+), 82 deletions(-) rename BaseTools/BinW= rappers/PosixLike/{AmlToHex =3D> AmlToC} (97%) rename BaseTools/BinWrapper= s/WindowsLike/{AmlToHex.bat =3D> AmlToC.bat} (100%) rename BaseTools/Sourc= e/Python/{AmlToHex/AmlToHex.py =3D> AmlToC/AmlToC.py} (52%) -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'