From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web08.27991.1610934766263218390 for ; Sun, 17 Jan 2021 17:52:47 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Mon, 18 Jan 2021 09:52:44 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: "'Matthew Carlson'" , References: <004401d6eb1c$97ecab90$c7c602b0$@byosoft.com.cn> <29608.1610766920854152845@groups.io> In-Reply-To: <29608.1610766920854152845@groups.io> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0g5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIHYzIDEvMV0gQmFzZVRvb2xzOiBVc2UgcGlwIG1vZHVsZSBpZiBhdmFpbGFibGUsIENJIHVzZXMgaXQgYnkgZGVmYXVsdA==?= Date: Mon, 18 Jan 2021 09:52:42 +0800 Message-ID: <003e01d6ed3c$9cfc00f0$d6f402d0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQGXals3p5NwYSenYqNAr0MC7xPwigKpS13DqpZXdyA= Content-Type: multipart/alternative; boundary="----=_NextPart_000_003F_01D6ED7F.AB202B50" Content-Language: zh-cn ------=_NextPart_000_003F_01D6ED7F.AB202B50 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Matthew: I can get your patch from matthewfcarlson/edk2: EDK II (github.com) = . = There are still some file format issues in this patch.=20 =20 1. BaseTools/BinPipWrappers/PosixLike/Brotli can be removed. It is not = required any more.=20 2. BaseTools/BinPipWrappers/PosixLike/build file mode should be 100755 = instead of 100644, because this file needs executable attribute. You can = use git update-index --chmod=3D+x file to set the file mode. All files = in BaseTools/BinPipWrappers/PosixLike have the same problem.=20 3. BaseTools/BinPipWrappers/WindowsLike/build.bat is the windows batch = script. It should be DOS style instead of UNIX. All files in = BaseTools/BinPipWrappers/WindowsLike have the same problem. 4. BaseTools/BuildEnv is Linux script. It should be UNIX style. Please = keep it. =20 Thanks Liming =E5=8F=91=E4=BB=B6=E4=BA=BA: tosubbounce+27952+0+0+0@groups.io = =E4=BB=A3=E8=A1=A8 macarl via [] =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B41=E6=9C=8816=E6=97=A5 = 11:15 =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming ; = devel@edk2.groups.io =E4=B8=BB=E9=A2=98: Re: [edk2-devel] =E5=9B=9E=E5=A4=8D: [edk2-devel] = [PATCH v3 1/1] BaseTools: Use pip module if available, CI uses it by = default =20 Sure. Is this the forked repo you mean? tianocore/edk2-basetools: EDK II = BaseTools Python tools as a PIP module (github.com) = =20 Or this one: matthewfcarlson/edk2: EDK II (github.com) = =20 --=20 - Matthew Carlson=20 ------=_NextPart_000_003F_01D6ED7F.AB202B50 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

Matthew:

=C2=A0I can = get your patch from matthewfcarlson/edk2: EDK II (github.com). There are still some = file format issues in this patch.

 

1.  BaseTools/Bin= PipWrappers/PosixLike/Brotli can be removed. It is not required any = more.

2.  BaseTools/Bin= PipWrappers/PosixLike/build file mode should be 100755 instead of = 100644, because this file needs executable attribute. You can use git = update-index --chmod=3D+x file to set the file mode. All files in = BaseTools/BinPipWrappers/PosixLike have the same problem. =

3.  BaseTools/Bin= PipWrappers/WindowsLike/build.bat is the windows batch script. It should = be DOS style instead of UNIX. All files in = BaseTools/BinPipWrappers/WindowsLike have the same = problem.

4.  BaseTools/Bui= ldEnv is Linux script. It should be UNIX style. Please keep = it.

 

Thanks

Liming

=E5=8F=91=E4=BB= =B6=E4=BA=BA: = tosubbounce+27952+0+0+0@groups.io = <tosubbounce+27952+0+0+0@groups.io> =E4=BB=A3=E8=A1= =A8 macarl via = []
=E5=8F=91=E9=80= =81=E6=97=B6=E9=97=B4: = 2021=E5=B9=B41=E6=9C=8816=E6=97=A5 11:15
=E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming = <gaoliming@byosoft.com.cn>; = devel@edk2.groups.io
=E4=B8=BB=E9=A2=98: Re: [edk2-devel] = =E5=9B=9E=E5=A4=8D: [edk2-devel] [PATCH v3 = 1/1] BaseTools: Use pip module if available, CI uses it by = default

 

Sure. Is this the forked repo you = mean? tianocore/edk2-baset= ools: EDK II BaseTools Python tools as a PIP module = (github.com)
Or this one: matthewfcarlson/edk2: EDK II (github.com)
--
- Matthew = Carlson

------=_NextPart_000_003F_01D6ED7F.AB202B50--