On Wed, Feb 1, 2023 at 2:48 PM Marvin Häuser <mhaeu...@posteo.de> wrote: > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4326 > > EccBuild currently assumes all packages reside in WORKSPACE. However, > this is obviously not the case for many setups. Most notably, Ext4Pkg > is located in edk2-platforms/Features and thus cannot be in WORKSPACE > in any reasonable setup. > > Use Edk2Path to locate the package in WORKSPACE and PACKAGES_PATH. > > Cc: Sean Brogan <sean.bro...@microsoft.com> > Cc: Michael Kubacki <mikub...@linux.microsoft.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Pedro Falcato <pedro.falc...@gmail.com> > Signed-off-by: Marvin Häuser <mhaeu...@posteo.de> > --- > .pytool/Plugin/EccCheck/EccCheck.py | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/.pytool/Plugin/EccCheck/EccCheck.py > b/.pytool/Plugin/EccCheck/EccCheck.py > index 4fbc67765fdf..7235fcb55cea 100644 > --- a/.pytool/Plugin/EccCheck/EccCheck.py > +++ b/.pytool/Plugin/EccCheck/EccCheck.py > @@ -69,6 +69,13 @@ class EccCheck(ICiBuildPlugin): > env.set_shell_var('PACKAGES_PATH', > os.pathsep.join(Edk2pathObj.PackagePathList)) > self.ECC_PASS = True > > + abs_pkg_path = > Edk2pathObj.GetAbsolutePathOnThisSystemFromEdk2RelativePath(packagename) > + > + if abs_pkg_path is None: > + tc.SetSkipped() > + tc.LogStdError("No Package folder {0}".format(abs_pkg_path)) > + return 0 > + > # Create temp directory > temp_path = os.path.join(workspace_path, 'Build', '.pytool', > 'Plugin', 'EccCheck') > try: > @@ -77,7 +84,7 @@ class EccCheck(ICiBuildPlugin): > shutil.rmtree(temp_path) > # Copy package being scanned to temp_path > shutil.copytree ( > - os.path.join(workspace_path, packagename), > + abs_pkg_path, > os.path.join(temp_path, packagename), > symlinks=True > ) > -- > 2.39.1 >
FWIW, Acked-by: Pedro Falcato <pedro.falc...@gmail.com> Please merge this so we can get edk2-platforms CI on Ext4Pkg. -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#99413): https://edk2.groups.io/g/devel/message/99413 Mute This Topic: https://groups.io/mt/96677674/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
