Reviewed-by: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com> > -----Original Message----- > From: Grant Likely <grant.lik...@arm.com> > Sent: Thursday, February 11, 2021 12:46 PM > To: devel@edk2.groups.io; G Edhaya Chandran > <edhaya.chand...@arm.com>; Barton Gao <gao...@byosoft.com.cn> > Cc: nd <n...@arm.com>; Samer El-Haj-Mahmoud <Samer.El-Haj- > mahm...@arm.com>; Grant Likely <grant.lik...@arm.com>; Samer El-Haj- > Mahmoud <samer.el-haj-mahm...@arm.com> > Subject: [edk2-sct PATCH 1/3] edk2-test: Add support for building extra > packages > > The build.sh script is very useful for setting up the build environment before > calling the package build. Sometimes additional packages are needed when > building the SCT. (e.g., it is useful to build ShellPkg). > Refactor the build code to allow additional DSCs to be added to the build. > > This patch is useful when building a full SCT image that includes the > EDK2 shell. > > Signed-off-by: Grant Likely <grant.lik...@arm.com> > Cc: G Edhaya Chandran <edhaya.chand...@arm.com> > Cc: Barton Gao <gao...@byosoft.com.cn> > Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com> > --- > uefi-sct/SctPkg/build.sh | 34 +++++++++++++--------------------- > 1 file changed, 13 insertions(+), 21 deletions(-) > > diff --git a/uefi-sct/SctPkg/build.sh b/uefi-sct/SctPkg/build.sh index > 37667711..22cf9667 100755 > --- a/uefi-sct/SctPkg/build.sh > +++ b/uefi-sct/SctPkg/build.sh > @@ -249,28 +249,20 @@ mkdir -p $DEST_DIR cp > $EDK_TOOLS_PATH/Source/C/bin/GenBin $DEST_DIR/GenBin > > # > -# Build the SCT package > +# Build the packages needed for the SCT # Set $DSC_EXTRA to any extra > +packages needed for the build > # > -build -p SctPkg/UEFI/UEFI_SCT.dsc -a $SCT_TARGET_ARCH -t > $TARGET_TOOLS -b $SCT_BUILD $3 $4 $5 $6 $7 $8 $9 > - > -# Check if there is any error > -status=$? > -if test $status -ne 0 > -then > - echo Could not build the UEFI SCT package > - exit -1 > -fi > - > -build -p SctPkg/UEFI/IHV_SCT.dsc -a $SCT_TARGET_ARCH -t > $TARGET_TOOLS -b $SCT_BUILD $3 $4 $5 $6 $7 $8 $9 > - > -# Check if there is any error > -status=$? > -if test $status -ne 0 > -then > - echo Could not build the IHV SCT package > - exit -1 > -fi > - > +for DSC in SctPkg/UEFI/UEFI_SCT.dsc SctPkg/UEFI/IHV_SCT.dsc > $DSC_EXTRA > +do > + build -p $DSC -a $SCT_TARGET_ARCH -t $TARGET_TOOLS -b > $SCT_BUILD $3 $4 $5 $6 $7 $8 $9 > + # Check if there is any error > + status=$? > + if test $status -ne 0 > + then > + echo Could not build package $DSC > + exit -1 > + fi > +done > > # > # If the argument is clean, then don't have to generate Sct binary. > -- > 2.20.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#71876): https://edk2.groups.io/g/devel/message/71876 Mute This Topic: https://groups.io/mt/80563679/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-