Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PERF] Fix "x$PERF_PREREQS_INSTALLED was unexpected at this time" Error for Microbenchmarks #73584

Merged
merged 4 commits into from
Aug 9, 2022

Conversation

LoopedBard3
Copy link
Member

Fix "x$PERF_PREREQS_INSTALLED was unexpected at this time" Error for Microbenchmarks from changes in #72431. This splits the changed command between a Windows and non-Windows version as Windows does not support Bash if statements.

@ghost
Copy link

ghost commented Aug 8, 2022

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Fix "x$PERF_PREREQS_INSTALLED was unexpected at this time" Error for Microbenchmarks from changes in #72431. This splits the changed command between a Windows and non-Windows version as Windows does not support Bash if statements.

Author: LoopedBard3
Assignees: LoopedBard3
Labels:

area-Infrastructure-libraries

Milestone: -

@radical
Copy link
Member

radical commented Aug 8, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LoopedBard3
Copy link
Member Author

I have a test runtime perf run of this here: https://dev.azure.com/dnceng/internal/_build/results?buildId=1930401. It looks like we should be good to merge this.

@radical
Copy link
Member

radical commented Aug 9, 2022

The changes are perf pipeline specific, and the remaining builds are completely unrelated. I'm merging this to unblock the pipeline.

@radical radical merged commit b8554be into dotnet:main Aug 9, 2022
@LoopedBard3 LoopedBard3 deleted the FixPerfVarNotExpectedYet branch August 9, 2022 15:16
@ghost ghost locked as resolved and limited conversation to collaborators Sep 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants