On Thu, Oct 28, 2021 at 10:09:07AM -0700, Sean Brogan wrote:
> one thing that jumped out quickly (comment below).
> Do you have a branch that i can look at and compare online?

https://github.com/kraxel/edk2/commits/ovmf-ci

> > +sys.path.append(os.path.join(os.getcwd(), 'OvmfPkg', 'PlatformCI'))
> 
> I would strongly suggest you base your path off your current file location
> rather than CWD.  This makes invocation much more flexible. See how
> workspace root is determined for similar solution.

Yep, that'll be more robust indeed.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#82908): https://edk2.groups.io/g/devel/message/82908
Mute This Topic: https://groups.io/mt/86435028/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to