Skip to content

Add scripts to set the environment up to use just-built artifacts#2578

Merged
StephanTLavavej merged 2 commits into
microsoft:mainfrom
barcharcraz:environment
Feb 23, 2022
Merged

Add scripts to set the environment up to use just-built artifacts#2578
StephanTLavavej merged 2 commits into
microsoft:mainfrom
barcharcraz:environment

Conversation

@barcharcraz
Copy link
Copy Markdown
Contributor

These are intended for development,
because I kept forgetting to add in the arch prefix

This is a github-only change, there's nothing that will touch the internal build, and
the files are copied over outside the "output" directory.

These are intended for development,
because I kept forgetting to add in the arch prefix
@barcharcraz barcharcraz added the build Related to the build system label Feb 17, 2022
@barcharcraz barcharcraz requested a review from a team as a code owner February 17, 2022 22:06
Copy link
Copy Markdown
Member

@StephanTLavavej StephanTLavavej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks - these scripts will make life more convenient.

Comment thread stl/set_environment.bat.in Outdated
Comment thread stl/set_environment.bat.in Outdated
Comment thread stl/set_environment.bat.in Outdated
Comment thread stl/set_environment.ps1.in Outdated
@StephanTLavavej
Copy link
Copy Markdown
Member

Verified that both the BAT and PS1 produce the same paths that my local incantations do. FYI @CaseyCarter as significant changes were pushed.

@StephanTLavavej StephanTLavavej self-assigned this Feb 22, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit ff54450 into microsoft:main Feb 23, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

Thanks again for this quality-of-life improvement! 😸 🎉 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Related to the build system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants