Re: [Patch] BaseTools: Fix the bug to support generate FFS by Align = Auto

["Gao, Liming" <liming.gao@intel.com>]

Yonghong:
  If Align is Auto, it means FFS alignment will inherit the alignment from its child section. Right?
  If Align is not specifies, it also means FFS alignment will inherit the alignment from its child section.

Thanks
Liming
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Yonghong Zhu
> Sent: Thursday, November 03, 2016 9:58 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming <liming.gao@intel.com>
> Subject: [edk2] [Patch] BaseTools: Fix the bug to support generate FFS by
> Align = Auto
> 
> FDF spec support to use 'Auto' as <FfsAlignmentValues>, but current Tool
> report error about -a=Auto is invalid option.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
> ---
>  BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py
> b/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py
> index c3f3624..05c4070 100644
> --- a/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py
> +++ b/BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py
> @@ -432,10 +432,12 @@ class GenFdsGlobalVariable:
>          if Fixed == True:
>              Cmd += ["-x"]
>          if CheckSum:
>              Cmd += ["-s"]
>          if Align not in [None, '']:
> +            if Align == 'Auto':
> +                Align = '1'
>              Cmd += ["-a", Align]
> 
>          Cmd += ["-o", Output]
>          for I in range(0, len(Input)):
>              Cmd += ("-i", Input[I])
> --
> 2.6.1.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel