Hello Eric/Supreeth,

Could you please review this patch.

Thanks,
Lokesh

-----Original Message-----
From: Lokesh B V <lokesh...@arm.com>
Sent: Tuesday, November 20, 2018 12:21 PM
To: edk2-devel@lists.01.org; Supreeth Venkatesh <supreeth.venkat...@arm.com>; 
eric....@intel.com
Cc: Lokesh Belathur Veerappa <lokesh...@arm.com>
Subject: [edk2-test][PATCH] SctPkg/build: Add support for GenBin tool build

As the GenBin tool is necessary for SCT build, it is appropriate to support 
it's build in the SCT build procedure.

Signed-off-by: Lokesh B V <lokesh...@arm.com>
---
 uefi-sct/SctPkg/build.sh | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/uefi-sct/SctPkg/build.sh b/uefi-sct/SctPkg/build.sh index 
73581c9..e070ad5 100755
--- a/uefi-sct/SctPkg/build.sh
+++ b/uefi-sct/SctPkg/build.sh
@@ -228,21 +228,26 @@ else
   echo using prebuilt tools
 fi

-# Copy GenBin file to Base tools directory
+if  [[ ! -e $EDK_TOOLS_PATH/Source/C/bin/GenBin ]] then
+  # build the GenBin if it doesn't yet exist
+  echo Building GenBin
+  make -C $EDK_TOOLS_PATH/../SctPkg/Tools/Source/GenBin
+  status=$?
+  if test $status -ne 0
+  then
+  echo Error while building GenBin
+    exit -1
+  fi
+else
+  echo using prebuilt GenBin
+fi
+
+# Copy GenBin file to Base tools bin directory
 DEST_DIR=`GetEdkToolsPathBinDirectory`
 # Ensure the directory exist
 mkdir -p $DEST_DIR
-case `uname -m` in
-x86_64)
-cp SctPkg/Tools/Bin/GenBin_lin_64 $DEST_DIR/GenBin
-;;
-x86_32)
-cp SctPkg/Tools/Bin/GenBin_lin_32 $DEST_DIR/GenBin
-;;
-*)
-cp SctPkg/Tools/Bin/GenBin_lin_32 $DEST_DIR/GenBin
-;;
-esac
+cp $EDK_TOOLS_PATH/Source/C/bin/GenBin $DEST_DIR/GenBin

 #
 # Build the SCT package
--
2.7.4

IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to