Skip to content

Fix crossgen2 comp jobs#52949

Merged
davidwrighton merged 14 commits intodotnet:mainfrom
davidwrighton:fix_crossgen2_comp_jobs
May 20, 2021
Merged

Fix crossgen2 comp jobs#52949
davidwrighton merged 14 commits intodotnet:mainfrom
davidwrighton:fix_crossgen2_comp_jobs

Conversation

@davidwrighton
Copy link
Copy Markdown
Member

@davidwrighton davidwrighton commented May 18, 2021

There were several problems with running these tests

  • They attempted to compile the Utf8 experimental assembly which no longer exists
  • It used the wrong CORE_ROOT
  • Errors were inscrutable and very hard to analyze. The tests now report using the standard mechanism of testresults.xml

There remains some evidence that the behavior is still a bit flaky, but the last test pass was completely green.

@davidwrighton davidwrighton marked this pull request as ready for review May 19, 2021 21:57
@davidwrighton davidwrighton requested a review from trylek May 19, 2021 21:57
Copy link
Copy Markdown
Member

@trylek trylek left a comment

Choose a reason for hiding this comment

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

Looks great to me, thank you!

--target_os $(target_crossgen2_os)
--target_arch $(targetarch);
--target_arch $(targetarch)
--compiler_arch_os $(compiler_arch_os);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is the semicolon appropriate here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes. Its the separator between different commands in bash. Just like the '&' for cmd scripts below.

@davidwrighton davidwrighton merged commit 478571c into dotnet:main May 20, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
@davidwrighton davidwrighton deleted the fix_crossgen2_comp_jobs branch April 13, 2023 18:47
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.

3 participants