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

Fix executable bitness for testhost.x86 #4652

Merged
merged 7 commits into from
Aug 14, 2023
Merged

Fix executable bitness for testhost.x86 #4652

merged 7 commits into from
Aug 14, 2023

Conversation

nohwnd
Copy link
Member

@nohwnd nohwnd commented Aug 11, 2023

Fixes those issues in 17.8.0 but we need to backport and release 17.7.1:

Related #4650
Related AB#1866640'
Related #4645

Remove + from the final version.
Fix the bitness of our executables. Will plug that script later into build, IDK if both those tools are on hosted images, and on everyones computers.

@nohwnd nohwnd changed the title No binlog from Test.cmd -integrationTest Fix executable bitness for testhost.x86 Aug 11, 2023
@@ -0,0 +1,8 @@
Get-ChildItem S:\p\vstest3\artifacts\packages\Debug\Shipping -Filter vstest.console.exe -Recurse -Force | ForEach-Object {
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this path linked to your machine setup?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes I am aware of that, answer to all the same questions below is in the PR description. I just did not want to lose that code.

Will plug that script later into build, IDK if both those tools are on hosted images, and on everyones computers.

eng/verify-nupkgs-exe.ps1 Show resolved Hide resolved
eng/verify-nupkgs-exe.ps1 Show resolved Hide resolved
eng/verify-nupkgs-exe.ps1 Show resolved Hide resolved
Co-authored-by: Amaury Levé <amauryleve@microsoft.com>
@nohwnd nohwnd enabled auto-merge (squash) August 14, 2023 07:17
@nohwnd
Copy link
Member Author

nohwnd commented Aug 14, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nohwnd
Copy link
Member Author

nohwnd commented Aug 14, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nohwnd nohwnd merged commit 5e505c8 into main Aug 14, 2023
@nohwnd nohwnd deleted the nobl branch August 14, 2023 13:46
nohwnd added a commit to nohwnd/vstest that referenced this pull request Aug 14, 2023
Co-authored-by: Amaury Levé <amauryleve@microsoft.com>
cvpoienaru pushed a commit that referenced this pull request Aug 14, 2023
Co-authored-by: Amaury Levé <amauryleve@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants