Because if statement is always FALSE.

On 19 авг. 2014 г., at 19:49, Olivier Martin <olivier.mar...@arm.com> wrote:

> I confirm this error too. It looks the recent BaseTools requires a ‘!else’ to 
> interpret the file correctly ...
>  
> From: Sergey Isakov [mailto:isakov...@bk.ru] 
> Sent: 19 August 2014 12:44
> To: edk2-devel@lists.sourceforge.net
> Subject: Re: [edk2] [Patch][Basetools]Fix a build failure in Linux
>  
> Sorry, it was with old build.py. With all new files all works but as I see 
> the syntax of fdf files become more strict
> Previously I have
> ----------
> !if ($(ARCH) == X64)
>  
> [FV.DuetEfiMainFvX64]
>  
> !elseif ($(ARCH) == IA32)
>  
> [FV.DuetEfiMainFvIA32]
>  
> !endif
> ----------
> now this is a syntax error
> --------
>  warning: Suspicious expression: == Comparison between Operand of string type 
> and Boolean/Number Type always return False.
>     !if ($(ARCH) == X64)
>  
> ---------
> When I use
> --------
> !if ("$(ARCH)" == "X64")
> --------
> This is good but error at !endif
> Then I add Else section
> ---------------
> !if ("$(ARCH)" == "X64")
>  
> [FV.DuetEfiMainFvX64]
>  
> !elseif ("$(ARCH)" == "IA32")
>  
> [FV.DuetEfiMainFvIA32]
>  
> !else 
> [FV.DuetEfiMainFvX64]
>  
> !endif
> ---------------
> and got successful compilation... but a result is questionable.
> How to do it right way?
>  
> On 19.08.2014, at 13:14, Gao, Liming wrote:
> 
> 
> Could you show the error message?
>  
>  
>  
> From: Sergey Isakov [mailto:isakov...@bk.ru] 
> Sent: Tuesday, August 19, 2014 5:05 PM
> To: edk2-devel@lists.sourceforge.net
> Subject: Re: [edk2] [Patch][Basetools]Fix a build failure in Linux
>  
> No,
> revision 15831 is still not working in OSX.
>  
> On 19.08.2014, at 12:07, Gao, Liming wrote:
> 
> 
> 
> Hess:
>   The patch is good.
> Reviewed-by: Gao, Liming <liming....@intel.com>
>  
> From: Chen, Hesheng [mailto:hesheng.c...@intel.com] 
> Sent: Tuesday, August 19, 2014 2:44 PM
> To: edk2-devel@lists.sourceforge.net
> Subject: [edk2] [Patch][Basetools]Fix a build failure in Linux
>  
> Hello all,
> Could you help review this patch?
>  
> This patch is going to fix a build failure in Linux system.
>  
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Hess Chen <hesheng.c...@intel.com>
>  
>  
> Best Regards,
> Chen, Hess
> Intel China Software Center
> Tel: +86-21-6116-6740
> Email: hesheng.c...@intel.com
>  
> ------------------------------------------------------------------------------
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/edk2-devel
>  
> ------------------------------------------------------------------------------
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/edk2-devel
>  
> ------------------------------------------------------------------------------
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/edk2-devel

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to