[PATCH V1 1/1] .pytool: UncrustifyCheck: Set IgnoreFiles path relative to package path

Kun Qin

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3942

`IgnoreFiles` tag is specified in the CI YAML files in each individual
packages. The current logic for UncrustifyCheck script bases specified
file paths from workspace, which requires the package name to be included
in each entry.

This change updates the ignore checking logic to be based on current
package path in order to reduce redundancy. It also keeps the consistency
of `IgnoreFiles` field other pytools such as SpellCheck and EccCheck.

Cc: Sean Brogan <sean.brogan@...>
Cc: Bret Barkelew <Bret.Barkelew@...>
Cc: Michael D Kinney <michael.d.kinney@...>
Cc: Liming Gao <gaoliming@...>
Cc: Michael Kubacki <mikuback@...>

Signed-off-by: Kun Qin <kuqin12@...>
.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py b/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py
index 00d78864656f..8dc9ffe6945a 100644
--- a/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py
+++ b/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py
@@ -290,7 +290,7 @@ class UncrustifyCheck(ICiBuildPlugin):
# This information is only used for reporting (not used here) and
# the ignore lines are being passed directly as they are given to
# this plugin.
- return parse_gitignore_lines(ignored_files, "Package configuration file", self._abs_workspace_path)
+ return parse_gitignore_lines(ignored_files, "Package configuration file", self._abs_package_path)

def _get_git_ignored_paths(self) -> List[str]:

Join {devel@edk2.groups.io to automatically receive all group messages.