Sean,

I agree the hard coded paths are not good.  This was a quick workaround to see 
if I could get the Azure Pipeline agents to function.  It we continue this 
path, we would have to migrate those out to something like toolsdef.txt or 
other mechanism that is more flexible.

There is a fundamental assumption that all edk2 build environments support a 
host POSIX build.  It would be better to update Azure Pipelines config to meet 
this assumption, and I can abandon this patch with the hard coded paths.

Do you have a suggestion for a patch to the Azure Pipelines YAML files to setup 
the VS host env?

Thanks,

Mike

From: sean.brogan via [] <sean.brogan=microsoft.com@[]>
Sent: Friday, January 31, 2020 4:33 PM
To: Kinney, Michael D <michael.d.kin...@intel.com>; devel@edk2.groups.io
Subject: Re: [edk2-devel] [Patch] BaseTools/Build: Fix Structured PCD app host 
env issues

Mike,
The hardcoded vs paths are not a safe assumption.
I would rather see agreement of how the environment should be configured prior 
to calling edk2 build and if being capable of building host os specific 
binaries is the requirement then that should be clarified and the scripts can 
be updated.  That way this type of stuff is contained within environment 
specific scripts rather than the edk2 build system.

thanks
Sean

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#53630): https://edk2.groups.io/g/devel/message/53630
Mute This Topic: https://groups.io/mt/70263619/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to