Reviewed-by: Eric Dong
> -Original Message-
> From: Ni, Ray
> Sent: Wednesday, June 17, 2020 5:03 PM
> To: devel@edk2.groups.io
> Cc: Ray Ni ; Dong, Eric
> ; Chan, Amy ; Chaganty,
> Rangasai V ; Oram, Isaac W
>
> Subject: [PATCH] Features/Intel/Readme.md: clarify feature package
> dependency rule
>
> From: Ray Ni
>
> Original statement only says depending on another feature package
> is not allowed. It gives developers impression that board level
> packages are allowed to depend on.
>
> This patch clarifies the package dependency rule.
>
> Signed-off-by: Ray Ni
> Cc: Eric Dong
> Cc: Amy Chan
> Cc: Rangasai V Chaganty
> Cc: Isaac W Oram
> ---
> Features/Intel/Readme.md | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/Features/Intel/Readme.md b/Features/Intel/Readme.md
> index 9729f90a41..1a03a4c0d2 100644
> --- a/Features/Intel/Readme.md
> +++ b/Features/Intel/Readme.md
> @@ -17,8 +17,10 @@ document as needed.
> ### Advanced Feature Attributes
>
> Advanced features should be:
>
> * _Cohesive_, the feature should not contain any functionality unrelated to
> the feature.
>
> -* _Complete_, the feature must have a complete design that minimizes
> dependencies. A feature package cannot directly
>
> - depend on another feature package.
>
> +* _Complete_, the feature must have a complete design that minimizes
> dependencies. A feature package can only depend
>
> + on following packages:
>
> + * edk2 repo: `MdePkg`, `MdeModulePkg`, `UefiCpuPkg`, `FmpDevicePkg`,
> `SecurityPkg`, `NetworkPkg`, `ShellPkg`.
>
> + * edk2-platforms repo: `MinPlatformPkg`, `IntelSiliconPkg`.
>
> * _Easy to Integrate_, the feature should expose well-defined software
> interfaces to use and configure the feature.
>
>* It should also present a set of simple and well-documented standard EDK
> II configuration options such as PCDs to
>
>configure the feature.
>
> --
> 2.25.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#61448): https://edk2.groups.io/g/devel/message/61448
Mute This Topic: https://groups.io/mt/74933867/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-