Hi£¬


    (Sorry for the previous incomplete email!)
    For the second one:  Dai already pointed out the problem, a ending slash(\) should be added to the nasm path variable. You can find the how nasm path is processed in edksetup.bat:


:check_NASM
if not defined NASM_PREFIX (
    @echo.
    @echo !!! WARNING !!! NASM_PREFIX environment variable is not set
    @if exist "C:\nasm\nasm.exe" @set "NASM_PREFIX=C:\nasm\"
    @if exist "C:\nasm\nasm.exe" @echo Found nasm.exe, setting the environment variable to C:\nasm\
    @if not exist "C:\nasm\nasm.exe" echo Attempting to build modules that require NASM will fail.
)


    And iasl path variable(IASL_PREFIX) is handled similarly. Maybe they should be adopted to handle with and without ending slash(\).


Best Regards,
Hiber
On 06/30/2019 21:22, Ni, Ray wrote:

Bob,

Any comments?

 

 

From:´÷£¬•ÔÕþ [mailto:cloud0907@yahoo.com.tw]
Sent: Sunday, June 30, 2019 3:37 PM
To: Justen, Jordan L <jordan.l.justen@intel.com>; afish@apple.com; Ni, Ray <ray.ni@intel.com>
Subject: May I become a contributor in EmulatorPkg ?

 

Dears,

 

I live in Taiwan so please forgive my English pool. I download edk2 source code from Github recently and I encounter two build error when I build EmulatorPkg (I use Visual Studio 2017 ). First is in GenPcdDb.py, this error means parameter used to unpack is zero, please refer below :

 

 

Second is in tools_def.txt, this error means there is missing slash before nasm.exe, please refer below :

 

 

You can search ¡°clouddai-test¡± in Attachment so that you will easy understand what I say. If I need submit more official document for become a contributor please let me know, or if my modification need obey some rule please also let me know, I am waiting your response