From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.20, mailfrom: bob.c.feng@intel.com) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by groups.io with SMTP; Wed, 22 May 2019 18:54:37 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 May 2019 18:54:36 -0700 X-ExtLoop1: 1 Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga008.jf.intel.com with ESMTP; 22 May 2019 18:54:36 -0700 Received: from fmsmsx152.amr.corp.intel.com (10.18.125.5) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 22 May 2019 18:54:36 -0700 Received: from shsmsx106.ccr.corp.intel.com (10.239.4.159) by FMSMSX152.amr.corp.intel.com (10.18.125.5) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 22 May 2019 18:54:36 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.129]) by SHSMSX106.ccr.corp.intel.com ([169.254.10.213]) with mapi id 14.03.0415.000; Thu, 23 May 2019 09:54:34 +0800 From: "Bob Feng" To: "Kinney, Michael D" , "Gao, Liming" , "devel@edk2.groups.io" Subject: Re: [Patch] BaseTools: Remove './SecMain' from 'run' target Thread-Topic: [Patch] BaseTools: Remove './SecMain' from 'run' target Thread-Index: AQHVBvpWjYakGzkbJEGDZvTEKnWOq6ZjbxaAgACG3tCAEUNyAIAACsgAgAIDwYCAAL6zMA== Date: Thu, 23 May 2019 01:54:33 +0000 Message-ID: <08650203BA1BD64D8AD9B6D5D74A85D160114023@SHSMSX101.ccr.corp.intel.com> References: <20190510063336.27928-1-bob.c.feng@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4466B8@SHSMSX104.ccr.corp.intel.com> <08650203BA1BD64D8AD9B6D5D74A85D16010A5F3@SHSMSX101.ccr.corp.intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E44EF2C@SHSMSX104.ccr.corp.intel.com> In-Reply-To: Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: bob.c.feng@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Mike, I missed the second 'SecMain', I'll send another patch to fix it. Bob -----Original Message----- From: Kinney, Michael D=20 Sent: Thursday, May 23, 2019 6:31 AM To: Gao, Liming ; Feng, Bob C ;= devel@edk2.groups.io; Kinney, Michael D Subject: RE: [Patch] BaseTools: Remove './SecMain' from 'run' target Hi Bob, I just noticed that there is a 2nd reference to 'SecMain' in build.py that was missed in this patch review. Is there a reason both were not removed? Mike > -----Original Message----- > From: Kinney, Michael D > Sent: Tuesday, May 21, 2019 8:45 AM > To: Gao, Liming ; Feng, Bob C=20 > ; devel@edk2.groups.io; Kinney, Michael D=20 > > Subject: RE: [Patch] BaseTools: Remove './SecMain' from 'run' target >=20 > Reviewed-by: Michael D Kinney > >=20 > If we want the 'run' target to work for the EmulatorPkg, we can add a=20 > python post build script to support that. > Please > enter a BZ feature request. >=20 > There is an example in the following edk2-staging branch that runs=20 > QEMU for an OvmfPkg build using a 'run' target. >=20 > https://github.com/tianocore/edk2- > staging/tree/Bug_1525_FmpDevicePkg_MultipleControllers/ > OvmfPkg >=20 > The DSC file contains a POSTBUILD statement. There are two python=20 > scripts in this example, but could be consolidated to a single python=20 > script for the EmulatorPkg. Run.py finds the output directory where=20 > the EmulatorPkg executable for each supported host OS can be found. >=20 > https://github.com/tianocore/edk2- > staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/ > OvmfPkg/OvmfPkgIa32X64.dsc >=20 > https://github.com/tianocore/edk2- > staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/ > OvmfPkg/PostBuild.py >=20 > https://github.com/tianocore/edk2- > staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/ > OvmfPkg/Run.py >=20 > Best regards, >=20 > Mike >=20 > > -----Original Message----- > > From: Gao, Liming > > Sent: Tuesday, May 21, 2019 8:06 AM > > To: Feng, Bob C ; devel@edk2.groups.io > > Cc: Kinney, Michael D > > Subject: RE: [Patch] BaseTools: Remove './SecMain' > from > > 'run' target > > > > Bob: > > Because NT32 has been removed, this change should > be > > OK. Reviewed-by: Liming Gao > > > > Thanks > > Liming > > > -----Original Message----- > > > From: Feng, Bob C > > > Sent: Friday, May 10, 2019 3:34 PM > > > To: Gao, Liming ; > > devel@edk2.groups.io > > > Cc: Kinney, Michael D > > > Subject: RE: [Patch] BaseTools: Remove './SecMain' > > from 'run' target > > > > > > I think NT32Pkg need a POSTBUILD scripts to execute > > its run command. > > > > > > > > > -----Original Message----- > > > From: Gao, Liming > > > Sent: Friday, May 10, 2019 3:26 PM > > > To: Feng, Bob C ; > > devel@edk2.groups.io > > > Cc: Kinney, Michael D > > > Subject: RE: [Patch] BaseTools: Remove './SecMain' > > from 'run' target > > > > > > Bob: > > > Please make sure run command still works on NT32 > > platform. > > > > > > >-----Original Message----- > > > >From: Feng, Bob C > > > >Sent: Friday, May 10, 2019 2:34 PM > > > >To: devel@edk2.groups.io > > > >Cc: Feng, Bob C ; Gao, > Liming > > > >; Kinney, Michael D > > > > > >Subject: [Patch] BaseTools: Remove './SecMain' > from > > 'run' target > > > > > > > >BZ: > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D1561 > > > > > > > >When a target of 'run' is passed into build.py, > > BaseTools > > > >unconditionally attempts to execute the > application > > called './SecMain' > > > >in the build output directory. > > > > > > > >This behavior applies to the Nt32Pkg which is > being > > replaced with > > > >features in the EmulatorPkg. > > > > > > > >Signed-off-by: Bob Feng > > > >Cc: Liming Gao > > > >Cc: Michael D Kinney > > > >--- > > > > BaseTools/Source/Python/build/build.py | 4 ---- > > > > 1 file changed, 4 deletions(-) > > > > > > > >diff --git > a/BaseTools/Source/Python/build/build.py > > > >b/BaseTools/Source/Python/build/build.py > > > >index 7271570d29..04f266abf5 100644 > > > >--- a/BaseTools/Source/Python/build/build.py > > > >+++ b/BaseTools/Source/Python/build/build.py > > > >@@ -1225,14 +1225,10 @@ class Build(): > > > > > > > > makefile =3D > > > > > > >GenMake.BuildFile(AutoGenObject)._FILE_NAME_[GenMake.g > > MakeType] > > > > > > > > # run > > > > if Target =3D=3D 'run': > > > >- RunDir =3D > > os.path.normpath(os.path.join(AutoGenObject.BuildDir, > > > >GlobalData.gGlobalDefines['ARCH'])) > > > >- Command =3D '.\SecMain' > > > >- os.chdir(RunDir) > > > >- LaunchCommand(Command, RunDir) > > > > return True > > > > > > > > # build modules > > > > if BuildModule: > > > > BuildCommand =3D BuildCommand + > [Target] > > > >-- > > > >2.20.1.windows.1