From: Michael Kubacki <michael.kubacki@microsoft.com> 1. Updates edk2-pytool-library to 0.12.1 - Picks up a minor bug fix 2. Updates edk2-pytool-extensions to 0.20.0 - Picks up a major release The changes in each update are in the respective patch commit messages. CI was run against both of these patches in this pull request: https://github.com/tianocore/edk2/pull/3632 Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> Michael Kubacki (2): pip-requirements.txt: Update to edk2-pytool-library 0.12.1 pip-requirements.txt: Update to edk2-pytool-extensions 0.20.0 pip-requirements.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96315): https://edk2.groups.io/g/devel/message/96315 Mute This Topic: https://groups.io/mt/94972447/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=- From: Michael Kubacki <michael.kubacki@microsoft.com> Updates edk2-pytool-library to pick up a minor bug fix release: 0.12.0 to 0.12.1 changes: - path_utilities.py: Prevent path case modification in GetContainingModules() That change prevents the case of paths from being set to lower case when returned from the function to avoid impacting case-sensitive callers. Release notes: https://github.com/tianocore/edk2-pytool-library/releases/tag/v0.12.1 Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> --- pip-requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pip-requirements.txt b/pip-requirements.txt index XXXXXXX..XXXXXXX 100644 --- a/pip-requirements.txt +++ b/pip-requirements.txt @@ -XXX,XX +XXX,XX @@ # https://www.python.org/dev/peps/pep-0440/#version-specifiers ## -edk2-pytool-library==0.12.0 +edk2-pytool-library==0.12.1 edk2-pytool-extensions~=0.19.1 edk2-basetools==0.1.39 antlr4-python3-runtime==4.7.1 -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96316): https://edk2.groups.io/g/devel/message/96316 Mute This Topic: https://groups.io/mt/94972448/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=- From: Michael Kubacki <michael.kubacki@microsoft.com> Updates edk2-pytool-extensions to pick up a major version release: 0.19.1 to 0.20.0 changes: - .vscode/settings.json: Enable flake8 linting - Add Pydocstyle - Move dependabot.yml location - Fix typos in robot files - Pydocstyle Updates - Plugin Loader Updates - edk2_stuart_pr_eval: Improve robustness of path comparisons - edk2_pr_eval.py: Build all packages on file change outside package - Allow build wrappers Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> --- pip-requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pip-requirements.txt b/pip-requirements.txt index XXXXXXX..XXXXXXX 100644 --- a/pip-requirements.txt +++ b/pip-requirements.txt @@ -XXX,XX +XXX,XX @@ ## edk2-pytool-library==0.12.1 -edk2-pytool-extensions~=0.19.1 +edk2-pytool-extensions~=0.20.0 edk2-basetools==0.1.39 antlr4-python3-runtime==4.7.1 -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96317): https://edk2.groups.io/g/devel/message/96317 Mute This Topic: https://groups.io/mt/94972450/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=- From: Michael Kubacki <michael.kubacki@microsoft.com> 1. Updates edk2-pytool-library to 0.12.1 - Picks up a minor bug fix 2. Updates edk2-pytool-extensions to 0.20.0 - Picks up a major release The changes in each update are in the respective patch commit messages. CI was run against both of these patches in this pull request: https://github.com/tianocore/edk2/pull/3632 These changes are recommended to be included in the edk2-stable202211 tag for the following reasons: 1. edk2-pytool-library 0.12.0 to 0.12.1 includes a single bug fix that simply returns file paths from Edk2Path.GetContainingModules() with the original case used by the operating system. This allow case sensitive callers to match paths returned by the function correctly. This can impact flows such as PR Eval that use this function to determine which modules were impacted by a code change. Without this change PR eval may not return consistent/expected results across operating systems. 2. The edk2-pytool-library change requires edk2-pytool-extensions 0.20.0. For this reason, edk2-pytool-extensions setup.py was updated in 0.20.0 to specify 0.12.1 as the edk2-pytool-library requirement. The logic of determining a package in pytool-library (Edk2Path.GetContainingPackage()) needs a corresponding change in the edk2-pytool-extensions 0.20.0 release. Without this change, flows such as PR Eval will not have updated logic for calling the function and return incorrect results. These changes have been tested against edk2 in pytool integration testing, the PR linked above, and against similar code in Mu. V2 changes: 1. Add stable tag to patch subject 2. Explain why changes should be included in stable tag 3. Add Reviewed-by tags to patches from v1 Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> Michael Kubacki (2): pip-requirements.txt: Update to edk2-pytool-library 0.12.1 pip-requirements.txt: Update to edk2-pytool-extensions 0.20.0 pip-requirements.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96366): https://edk2.groups.io/g/devel/message/96366 Mute This Topic: https://groups.io/mt/95033514/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=- From: Michael Kubacki <michael.kubacki@microsoft.com> Updates edk2-pytool-library to pick up a minor bug fix release: 0.12.0 to 0.12.1 changes: - path_utilities.py: Prevent path case modification in GetContainingModules() That change prevents the case of paths from being set to lower case when returned from the function to avoid impacting case-sensitive callers. Release notes: https://github.com/tianocore/edk2-pytool-library/releases/tag/v0.12.1 Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> Reviewed-by: Sean Brogan <sean.brogan@microsoft.com> --- pip-requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pip-requirements.txt b/pip-requirements.txt index XXXXXXX..XXXXXXX 100644 --- a/pip-requirements.txt +++ b/pip-requirements.txt @@ -XXX,XX +XXX,XX @@ # https://www.python.org/dev/peps/pep-0440/#version-specifiers ## -edk2-pytool-library==0.12.0 +edk2-pytool-library==0.12.1 edk2-pytool-extensions~=0.19.1 edk2-basetools==0.1.39 antlr4-python3-runtime==4.7.1 -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96367): https://edk2.groups.io/g/devel/message/96367 Mute This Topic: https://groups.io/mt/95033521/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=- From: Michael Kubacki <michael.kubacki@microsoft.com> Updates edk2-pytool-extensions to pick up a major version release: 0.19.1 to 0.20.0 changes: - .vscode/settings.json: Enable flake8 linting - Add Pydocstyle - Move dependabot.yml location - Fix typos in robot files - Pydocstyle Updates - Plugin Loader Updates - edk2_stuart_pr_eval: Improve robustness of path comparisons - edk2_pr_eval.py: Build all packages on file change outside package - Allow build wrappers Cc: Sean Brogan <sean.brogan@microsoft.com> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Andrew Fish <afish@apple.com> Cc: Leif Lindholm <quic_llindhol@quicinc.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> Reviewed-by: Sean Brogan <sean.brogan@microsoft.com> --- pip-requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pip-requirements.txt b/pip-requirements.txt index XXXXXXX..XXXXXXX 100644 --- a/pip-requirements.txt +++ b/pip-requirements.txt @@ -XXX,XX +XXX,XX @@ ## edk2-pytool-library==0.12.1 -edk2-pytool-extensions~=0.19.1 +edk2-pytool-extensions~=0.20.0 edk2-basetools==0.1.39 antlr4-python3-runtime==4.7.1 -- 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#96368): https://edk2.groups.io/g/devel/message/96368 Mute This Topic: https://groups.io/mt/95033523/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-