Re: [edk2-devel] [PATCH v2 1/6] EDK2 Code First: PI Specification: EFI_MM_COMMUNICATE_HEADER Update

["Laszlo Ersek" <lersek@redhat.com>]

On 06/18/21 11:02, Kun Qin wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3430
> 
> This change includes specification update markdown file that describes
> the proposed PI Specification v1.7 Errata A in detail and potential
> impact to the existing codebase.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> 
> Signed-off-by: Kun Qin <kuqin12@gmail.com>
> ---
> 
> Notes:
>     v2:
>     - Updated change impact analysis regarding SmmLockBoxDxeLib [Hao]
> 
>  BZ3430-SpecChange.md | 90 ++++++++++++++++++++
>  1 file changed, 90 insertions(+)

Placing such a document in the edk2 root directory looks very strange to me.

https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Code-First-Process#tianocore-github

    [...] Specification text changes are held within the affected source
    repository, using the GitHub flavor of markdown, in a file (or split
    across several files) with .md suffix [...]

The wiki does not seem to specify what directory should contain the spec
change document.

Should we create a "CodeFirst" top-level directory?

Thanks,
Laszlo