summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKun Qin <kuqin12@gmail.com>2022-06-06 12:24:54 -0700
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2022-06-07 01:53:24 +0000
commit4f89e4b3e80329b9a445500009c658d2ebce8475 (patch)
tree6a144c83e09f6457aac13327b73ba2fe4d4c50a3
parent0b36dea3f8b71ddddc4fdf3a5d2edf395115568b (diff)
downloadedk2-4f89e4b3e80329b9a445500009c658d2ebce8475.zip
edk2-4f89e4b3e80329b9a445500009c658d2ebce8475.tar.gz
edk2-4f89e4b3e80329b9a445500009c658d2ebce8475.tar.bz2
.pytool: UncrustifyCheck: Set IgnoreFiles path relative to package path
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@microsoft.com> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Michael Kubacki <mikuback@linux.microsoft.com> Signed-off-by: Kun Qin <kuqin12@gmail.com> Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
-rw-r--r--.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py b/.pytool/Plugin/UncrustifyCheck/UncrustifyCheck.py
index 00d7886..8dc9ffe 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]:
""""