[edk2-devel] [Patch] BaseTools: Remove './SecMain' from 'run' target

[Michael D Kinney]

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 <liming.gao at intel.com>; Feng, Bob C
> <bob.c.feng at intel.com>; devel at edk2.groups.io; Kinney,
> Michael D <michael.d.kinney at intel.com>
> Subject: RE: [Patch] BaseTools: Remove './SecMain' from
> 'run' target
> 
> Reviewed-by: Michael D Kinney
> <michael.d.kinney at intel.com>
> 
> If we want the 'run' target to work for the
> EmulatorPkg, we
> can add a python post build script to support that.
> Please
> enter a BZ feature request.
> 
> There is an example in the following edk2-staging
> branch that
> runs QEMU for an OvmfPkg build using a 'run' target.
> 
> https://github.com/tianocore/edk2-
> staging/tree/Bug_1525_FmpDevicePkg_MultipleControllers/
> OvmfPkg
> 
> The DSC file contains a POSTBUILD statement.  There are
> two
> python scripts in this example, but could be
> consolidated to
> a single python script for the EmulatorPkg.  Run.py
> finds the
> output directory where the EmulatorPkg executable for
> each
> supported host OS can be found.
> 
> https://github.com/tianocore/edk2-
> staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/
> OvmfPkg/OvmfPkgIa32X64.dsc
> 
> https://github.com/tianocore/edk2-
> staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/
> OvmfPkg/PostBuild.py
> 
> https://github.com/tianocore/edk2-
> staging/blob/Bug_1525_FmpDevicePkg_MultipleControllers/
> OvmfPkg/Run.py
> 
> Best regards,
> 
> Mike
> 
> > -----Original Message-----
> > From: Gao, Liming
> > Sent: Tuesday, May 21, 2019 8:06 AM
> > To: Feng, Bob C <bob.c.feng at intel.com>;
> > devel at edk2.groups.io
> > Cc: Kinney, Michael D <michael.d.kinney at intel.com>
> > Subject: RE: [Patch] BaseTools: Remove './SecMain'
> from
> > 'run' target
> >
> > Bob:
> >   Because NT32 has been removed, this change should
> be
> > OK. Reviewed-by: Liming Gao <liming.gao at intel.com>
> >
> > Thanks
> > Liming
> > > -----Original Message-----
> > > From: Feng, Bob C
> > > Sent: Friday, May 10, 2019 3:34 PM
> > > To: Gao, Liming <liming.gao at intel.com>;
> > devel at edk2.groups.io
> > > Cc: Kinney, Michael D <michael.d.kinney at intel.com>
> > > 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 <bob.c.feng at intel.com>;
> > devel at edk2.groups.io
> > > Cc: Kinney, Michael D <michael.d.kinney at intel.com>
> > > 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 at edk2.groups.io
> > > >Cc: Feng, Bob C <bob.c.feng at intel.com>; Gao,
> Liming
> > > ><liming.gao at intel.com>; Kinney, Michael D
> > <michael.d.kinney at intel.com>
> > > >Subject: [Patch] BaseTools: Remove './SecMain'
> from
> > 'run' target
> > > >
> > > >BZ:
> > https://bugzilla.tianocore.org/show_bug.cgi?id=1561
> > > >
> > > >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 <bob.c.feng at intel.com>
> > > >Cc: Liming Gao <liming.gao at intel.com>
> > > >Cc: Michael D Kinney <michael.d.kinney at intel.com>
> > > >---
> > > > 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 =
> > >
> >
> >GenMake.BuildFile(AutoGenObject)._FILE_NAME_[GenMake.g
> > MakeType]
> > > >
> > > >         # run
> > > >         if Target == 'run':
> > > >-            RunDir =
> > os.path.normpath(os.path.join(AutoGenObject.BuildDir,
> > > >GlobalData.gGlobalDefines['ARCH']))
> > > >-            Command = '.\SecMain'
> > > >-            os.chdir(RunDir)
> > > >-            LaunchCommand(Command, RunDir)
> > > >             return True
> > > >
> > > >         # build modules
> > > >         if BuildModule:
> > > >             BuildCommand = BuildCommand +
> [Target]
> > > >--
> > > >2.20.1.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41251): https://edk2.groups.io/g/devel/message/41251
Mute This Topic: https://groups.io/mt/31574316/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-