[edk2-devel] [PATCH] edksetup.bat: Simplify the step to use CLANGPDB

Zhiguang Liu posted 1 patch 4 years, 3 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
edksetup.bat | 11 +++++++++++
1 file changed, 11 insertions(+)
[edk2-devel] [PATCH] edksetup.bat: Simplify the step to use CLANGPDB
Posted by Zhiguang Liu 4 years, 3 months ago
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2404

Set the below two environment variables in edksetup.bat:
  set CLANG_HOST_BIN=n
  set CLANG_BIN=C:\Program Files\LLVM\bin\
The CLANG_BIN is only be set if it is not defined.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
---
 edksetup.bat | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/edksetup.bat b/edksetup.bat
index 024f57a4b7..bd057d1241 100755
--- a/edksetup.bat
+++ b/edksetup.bat
@@ -113,6 +113,17 @@ if not defined NASM_PREFIX (
     @if not exist "C:\nasm\nasm.exe" echo   Attempting to build modules that require NASM will fail.
 )
 
+:check_CLANGPDB
+@set CLANG_HOST_BIN=n
+if not defined CLANG_BIN (
+    @echo.
+    @echo !!! WARNING !!! CLANG_BIN environment variable is not set
+    @if exist "C:\Program Files\LLVM\bin\clang.exe" (
+        @set CLANG_BIN=C:\Program Files\LLVM\bin\
+        @echo   Found LLVM, setting the environment variable to C:\Program Files\LLVM\bin\
+    )
+)
+
 :check_cygwin
 if defined CYGWIN_HOME (
   if not exist "%CYGWIN_HOME%" (
-- 
2.16.2.windows.1


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

View/Reply Online (#52451): https://edk2.groups.io/g/devel/message/52451
Mute This Topic: https://groups.io/mt/68845983/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH] edksetup.bat: Simplify the step to use CLANGPDB
Posted by Ni, Ray 4 years, 3 months ago
> +:check_CLANGPDB

1. Would be great if there is comment to explain why "n" is set to CLANG_HOST_BIN.

> +@set CLANG_HOST_BIN=n
> +if not defined CLANG_BIN (
> +    @echo.
> +    @echo !!! WARNING !!! CLANG_BIN environment variable is not set
> +    @if exist "C:\Program Files\LLVM\bin\clang.exe" (
> +        @set CLANG_BIN=C:\Program Files\LLVM\bin\
> +        @echo   Found LLVM, setting the environment variable to C:\Program
> Files\LLVM\bin\

2. How about "Found LLVM, setting CLANG_BIN environment variable to C:\Program Files\LLVM\bin\."?

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

View/Reply Online (#52453): https://edk2.groups.io/g/devel/message/52453
Mute This Topic: https://groups.io/mt/68845983/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-