[edk2-devel] [Patch] BaseTools: Remove './SecMain' from 'run' target
Bob Feng
bob.c.feng at intel.com
Wed May 22 01:05:52 UTC 2019
I entered a new BZ to create the EmulatorPkg postbuild script for "build run" support.
https://bugzilla.tianocore.org/show_bug.cgi?id=1828
Thanks,
Bob
-----Original Message-----
From: Kinney, Michael D
Sent: Tuesday, May 21, 2019 11:45 PM
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 (#41179): https://edk2.groups.io/g/devel/message/41179
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]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list