public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-platofrms] [patch v2] Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
@ 2019-06-28  2:55 Dandan Bi
  2019-06-28  2:57 ` [edk2-devel] " Liming Gao
       [not found] ` <15AC3D5D2C07B354.14856@groups.io>
  0 siblings, 2 replies; 3+ messages in thread
From: Dandan Bi @ 2019-06-28  2:55 UTC (permalink / raw)
  To: devel; +Cc: Shifei A Lu, Xiaohu Zhou, Isaac W Oram, Liming Gao

V2: Remove nmake -f %BASE_TOOLS_PATH%\Makefile, since it
has been covered by edksetup.bat Rebuild option.

Since Nt32Pkg has been removed from edk2 repo, and
we will also remove the --nt32 flag in edksetup.bat.
So here update platform to pass build.

Cc: Shifei A Lu <shifei.a.lu@intel.com>
Cc: Xiaohu Zhou <bowen.zhou@intel.com>
Cc: Isaac W Oram <isaac.w.oram@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
 .../BoardMtOlympus/GitEdk2MinMtOlympus.bat                  | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOlympus.bat b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOlympus.bat
index 02ed458236..6b4e90294b 100644
--- a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOlympus.bat
+++ b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOlympus.bat
@@ -46,20 +46,16 @@ set EDK_TOOLS_BIN=%WORKSPACE%\edk2-BaseTools-win32
 
 set EDK_SETUP_OPTION=
 @rem if python is installed, disable the binary base tools.
 if defined PYTHON_HOME (
   set EDK_TOOLS_BIN=
-  set EDK_SETUP_OPTION=--nt32
+  set EDK_SETUP_OPTION=Rebuild
 )
 pushd %WORKSPACE%\edk2
 call edksetup.bat %EDK_SETUP_OPTION%
 popd
 pushd %WORKSPACE%
-@rem if python is installed, nmake BaseTools source and enable BaseTools source build
-@if defined PYTHON_HOME (
-  nmake -f %BASE_TOOLS_PATH%\Makefile
-)
 popd
 
 set openssl_path=%WORKSPACE%
 
 popd
-- 
2.18.0.windows.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [edk2-devel] [edk2-platofrms] [patch v2] Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
  2019-06-28  2:55 [edk2-platofrms] [patch v2] Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup Dandan Bi
@ 2019-06-28  2:57 ` Liming Gao
       [not found] ` <15AC3D5D2C07B354.14856@groups.io>
  1 sibling, 0 replies; 3+ messages in thread
From: Liming Gao @ 2019-06-28  2:57 UTC (permalink / raw)
  To: devel@edk2.groups.io, Bi, Dandan; +Cc: Lu, Shifei A, Zhou, Bowen, Oram, Isaac W

Dandan:
  Don't need to add version info in the commit log. And, please also remove pushd %WORKSPACE% & popd. They are not used after nmake is removed. With those changes, Reviewed-by: Liming Gao <liming.gao@intel.com>

Thanks
Liming
>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Dandan Bi
>Sent: Friday, June 28, 2019 10:55 AM
>To: devel@edk2.groups.io
>Cc: Lu, Shifei A <shifei.a.lu@intel.com>; Zhou, Bowen
><bowen.zhou@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>; Gao,
>Liming <liming.gao@intel.com>
>Subject: [edk2-devel] [edk2-platofrms] [patch v2]
>Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
>
>V2: Remove nmake -f %BASE_TOOLS_PATH%\Makefile, since it
>has been covered by edksetup.bat Rebuild option.
>
>Since Nt32Pkg has been removed from edk2 repo, and
>we will also remove the --nt32 flag in edksetup.bat.
>So here update platform to pass build.
>
>Cc: Shifei A Lu <shifei.a.lu@intel.com>
>Cc: Xiaohu Zhou <bowen.zhou@intel.com>
>Cc: Isaac W Oram <isaac.w.oram@intel.com>
>Cc: Liming Gao <liming.gao@intel.com>
>Signed-off-by: Dandan Bi <dandan.bi@intel.com>
>---
> .../BoardMtOlympus/GitEdk2MinMtOlympus.bat                  | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
>diff --git
>a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>mpus.bat
>b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>mpus.bat
>index 02ed458236..6b4e90294b 100644
>---
>a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>mpus.bat
>+++
>b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>mpus.bat
>@@ -46,20 +46,16 @@ set EDK_TOOLS_BIN=%WORKSPACE%\edk2-
>BaseTools-win32
>
> set EDK_SETUP_OPTION=
> @rem if python is installed, disable the binary base tools.
> if defined PYTHON_HOME (
>   set EDK_TOOLS_BIN=
>-  set EDK_SETUP_OPTION=--nt32
>+  set EDK_SETUP_OPTION=Rebuild
> )
> pushd %WORKSPACE%\edk2
> call edksetup.bat %EDK_SETUP_OPTION%
> popd
> pushd %WORKSPACE%
>-@rem if python is installed, nmake BaseTools source and enable BaseTools
>source build
>-@if defined PYTHON_HOME (
>-  nmake -f %BASE_TOOLS_PATH%\Makefile
>-)
> popd
>
> set openssl_path=%WORKSPACE%
>
> popd
>--
>2.18.0.windows.1
>
>
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [edk2-devel] [edk2-platofrms] [patch v2] Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
       [not found] ` <15AC3D5D2C07B354.14856@groups.io>
@ 2019-06-28  7:14   ` Liming Gao
  0 siblings, 0 replies; 3+ messages in thread
From: Liming Gao @ 2019-06-28  7:14 UTC (permalink / raw)
  To: devel@edk2.groups.io, Gao, Liming, Bi, Dandan
  Cc: Lu, Shifei A, Zhou, Bowen, Oram, Isaac W

Push @e7fc97a3f9045c35a077ab0151c696aeba722417

>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Liming Gao
>Sent: Friday, June 28, 2019 10:57 AM
>To: devel@edk2.groups.io; Bi, Dandan <dandan.bi@intel.com>
>Cc: Lu, Shifei A <shifei.a.lu@intel.com>; Zhou, Bowen
><bowen.zhou@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>
>Subject: Re: [edk2-devel] [edk2-platofrms] [patch v2]
>Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
>
>Dandan:
>  Don't need to add version info in the commit log. And, please also remove
>pushd %WORKSPACE% & popd. They are not used after nmake is removed.
>With those changes, Reviewed-by: Liming Gao <liming.gao@intel.com>
>
>Thanks
>Liming
>>-----Original Message-----
>>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>>Dandan Bi
>>Sent: Friday, June 28, 2019 10:55 AM
>>To: devel@edk2.groups.io
>>Cc: Lu, Shifei A <shifei.a.lu@intel.com>; Zhou, Bowen
>><bowen.zhou@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>; Gao,
>>Liming <liming.gao@intel.com>
>>Subject: [edk2-devel] [edk2-platofrms] [patch v2]
>>Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup
>>
>>V2: Remove nmake -f %BASE_TOOLS_PATH%\Makefile, since it
>>has been covered by edksetup.bat Rebuild option.
>>
>>Since Nt32Pkg has been removed from edk2 repo, and
>>we will also remove the --nt32 flag in edksetup.bat.
>>So here update platform to pass build.
>>
>>Cc: Shifei A Lu <shifei.a.lu@intel.com>
>>Cc: Xiaohu Zhou <bowen.zhou@intel.com>
>>Cc: Isaac W Oram <isaac.w.oram@intel.com>
>>Cc: Liming Gao <liming.gao@intel.com>
>>Signed-off-by: Dandan Bi <dandan.bi@intel.com>
>>---
>> .../BoardMtOlympus/GitEdk2MinMtOlympus.bat                  | 6 +-----
>> 1 file changed, 1 insertion(+), 5 deletions(-)
>>
>>diff --git
>>a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>>mpus.bat
>>b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>>mpus.bat
>>index 02ed458236..6b4e90294b 100644
>>---
>>a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>>mpus.bat
>>+++
>>b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/GitEdk2MinMtOly
>>mpus.bat
>>@@ -46,20 +46,16 @@ set EDK_TOOLS_BIN=%WORKSPACE%\edk2-
>>BaseTools-win32
>>
>> set EDK_SETUP_OPTION=
>> @rem if python is installed, disable the binary base tools.
>> if defined PYTHON_HOME (
>>   set EDK_TOOLS_BIN=
>>-  set EDK_SETUP_OPTION=--nt32
>>+  set EDK_SETUP_OPTION=Rebuild
>> )
>> pushd %WORKSPACE%\edk2
>> call edksetup.bat %EDK_SETUP_OPTION%
>> popd
>> pushd %WORKSPACE%
>>-@rem if python is installed, nmake BaseTools source and enable BaseTools
>>source build
>>-@if defined PYTHON_HOME (
>>-  nmake -f %BASE_TOOLS_PATH%\Makefile
>>-)
>> popd
>>
>> set openssl_path=%WORKSPACE%
>>
>> popd
>>--
>>2.18.0.windows.1
>>
>>
>>
>
>
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-06-28  7:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-28  2:55 [edk2-platofrms] [patch v2] Platform/PurleyOpenBoardPkg: Remove --nt32 flag of edkseup Dandan Bi
2019-06-28  2:57 ` [edk2-devel] " Liming Gao
     [not found] ` <15AC3D5D2C07B354.14856@groups.io>
2019-06-28  7:14   ` Liming Gao

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox