Re: [Patch 1/5] Build Spec: Add !error statement support

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

Hi Mike,

Yes, build tool will return error code when build break. This is the document patch to add '!error' statement.

Best Regards,
Zhu Yonghong

-----Original Message-----
From: Kinney, Michael D 
Sent: Tuesday, June 05, 2018 4:06 AM
To: Zhu, Yonghong <yonghong.zhu@intel.com>; edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kinney@intel.com>
Cc: Shaw, Kevin W <kevin.w.shaw@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: RE: [edk2] [Patch 1/5] Build Spec: Add !error statement support

Yonghong,

I think it should be stated that the build tool also returns an error code.  This allows scripts that invoke build to check the returned error code.

Mike

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-
> bounces@lists.01.org] On Behalf Of Yonghong Zhu
> Sent: Wednesday, May 30, 2018 5:56 PM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Shaw, Kevin W 
> <kevin.w.shaw@intel.com>; Gao, Liming <liming.gao@intel.com>
> Subject: [edk2] [Patch 1/5] Build Spec: Add !error statement support
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael Kinney <michael.d.kinney@intel.com>
> Cc: Kevin W Shaw <kevin.w.shaw@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
> ---
>  8_pre-build_autogen_stage/82_auto-generation_process.md
> | 6 ++++++
>  README.md
> | 3 ++-
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/8_pre-build_autogen_stage/82_auto-
> generation_process.md b/8_pre-
> build_autogen_stage/82_auto-generation_process.md
> index 8f8f29b..abfa55c 100644
> --- a/8_pre-build_autogen_stage/82_auto-
> generation_process.md
> +++ b/8_pre-build_autogen_stage/82_auto-
> generation_process.md
> @@ -1058,10 +1058,16 @@ hash value file into the directory specified 
> by binary-destination at the build  When --hash and --binary-source 
> are specified, build tool will try to get the binary  files from the 
> binary source directory at the build phase. If the cached binary has  
> the same hash value, it will be directly used.
> Otherwise, build tool will compile the  source files and generate the 
> binary files.
> 
> +#### 8.2.4.16 !error Statement
> +
> +The DSC and FDF file can use `!error` statement. The
> argument of this statement is an
> +error message, it causes build tool to stop at the
> location where the statement is
> +encountered and error message following the `!error`
> statement is output as a message.
> +
>  ### 8.2.5 Post processing
> 
>  Once all files are parsed, the build tools will do following work for 
> each EDK  II module:
> 
> diff --git a/README.md b/README.md
> index 18e46c9..9ca8733 100644
> --- a/README.md
> +++ b/README.md
> @@ -223,6 +223,7 @@ Copyright (c) 2008-2017, Intel Corporation. All 
> rights reserved.
>  |            |
> [#775](https://bugzilla.tianocore.org/show_bug.cgi?id=77
> 5) Build spec: Add description about auto detect thread number
> |               |
>  |            | Add Flexible PCD value format into spec
> |               |
>  |            | Update PCD value and SKU, DefaultStore
> info in build report
> |               |
>  |            | Clarify structure PCD field value
> assignment precedence
> |               |
>  |            | Update description for build handles
> PCDs for SKU support
> |               |
> -|            | Add statement about module scoped
> `<Pcd*>` section override scope
> |               |
> +|            | Add statement about module scoped
> `<Pcd*>` section override scope
> |               |
> +|            | Add !error statement section
> |               |
> --
> 2.6.1.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel