Re: [Patch 4/4 V3] BaseTools: Update Makefile to support FFS file generation

["Zhu, Yonghong" <yonghong.zhu@intel.com>]

Hi Leif,

Please help to check whether this patch " [edk2] [PATCH] BaseTools: Fix GenSec can't found the depex file" can fix it. thanks a lot.

Best Regards,
Zhu Yonghong

-----Original Message-----
From: Zhu, Yonghong 
Sent: Thursday, December 07, 2017 9:00 AM
To: Leif Lindholm <leif.lindholm@linaro.org>
Cc: edk2-devel@lists.01.org; Feng, YunhuaX <yunhuax.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Zhu, Yonghong <yonghong.zhu@intel.com>
Subject: RE: [edk2] [Patch 4/4 V3] BaseTools: Update Makefile to support FFS file generation

Thanks Leif.  I will provide patch to fix it soon.

Best Regards,
Zhu Yonghong


-----Original Message-----
From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
Sent: Thursday, December 07, 2017 2:24 AM
To: Zhu, Yonghong <yonghong.zhu@intel.com>
Cc: edk2-devel@lists.01.org; Feng, YunhuaX <yunhuax.feng@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: Re: [edk2] [Patch 4/4 V3] BaseTools: Update Makefile to support FFS file generation

Hi Yonghong,

I'm afraid I have come across an issue with this one as well:
When trying to build OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b
into a platform (edk2-platforms/Platform/Hisilicon/HiKey/), GenSec fails with:

Generating FVMAIN FV
####################################
########################################
########Return Value = 2
GenSec: ERROR 0001: Error opening file
  /work/git/tianocore/Build/HiKey/RELEASE_GCC5/AARCH64/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b/OUTPUT/Ax88772b.depex
GenSec: ERROR 2000: Status is not successful

If I revert commit 37de70b7, the platform builds successfully again.
D03 and D05 platforms also appear to have issues with this, but in that case due to drivers in edk2-non-osi.

/
    Leif

On Wed, Nov 29, 2017 at 10:02:06PM +0800, Yonghong Zhu wrote:
> Update Makefile to support FFS file generation with new build option 
> --genfds-multi-thread.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
> Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com>
> ---
>  BaseTools/Source/Python/AutoGen/AutoGen.py         |  23 ++-
>  BaseTools/Source/Python/AutoGen/GenMake.py         |  90 ++++++++-
>  BaseTools/Source/Python/Common/GlobalData.py       |   1 +
>  BaseTools/Source/Python/GenFds/AprioriSection.py   |  14 +-
>  BaseTools/Source/Python/GenFds/CompressSection.py  |  10 +-
>  BaseTools/Source/Python/GenFds/DataSection.py      |  37 ++--
>  BaseTools/Source/Python/GenFds/DepexSection.py     |   6 +-
>  BaseTools/Source/Python/GenFds/EfiSection.py       |  78 +++++---
>  BaseTools/Source/Python/GenFds/Fd.py               |  24 ++-
>  BaseTools/Source/Python/GenFds/FfsFileStatement.py |   4 +-
>  BaseTools/Source/Python/GenFds/FfsInfStatement.py  | 202 ++++++++++++---------
>  BaseTools/Source/Python/GenFds/Fv.py               | 189 ++++++++++---------
>  BaseTools/Source/Python/GenFds/FvImageSection.py   |   8 +-
>  BaseTools/Source/Python/GenFds/GenFds.py           |  20 ++
>  .../Source/Python/GenFds/GenFdsGlobalVariable.py   | 174 ++++++++++++++----
>  BaseTools/Source/Python/GenFds/GuidSection.py      | 168 +++++++++--------
>  .../Source/Python/GenFds/OptRomFileStatement.py    |   4 +-
>  .../Source/Python/GenFds/OptRomInfStatement.py     |  12 +-
>  BaseTools/Source/Python/GenFds/OptionRom.py        |  25 +--
>  BaseTools/Source/Python/GenFds/Region.py           |  49 +++--
>  BaseTools/Source/Python/GenFds/Section.py          |   4 +-
>  BaseTools/Source/Python/GenFds/UiSection.py        |   5 +-
>  BaseTools/Source/Python/GenFds/VerSection.py       |   9 +-
>  BaseTools/Source/Python/build/build.py             |  48 ++++-
>  24 files changed, 793 insertions(+), 411 deletions(-)