Skip to content

Conversation

akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented May 15, 2023

Also try to fix the StartupHookTests on LLVM FullAOT by making sure we also AOT dlls in subdirectories.

This uncovered an issue in src/tests/Common/external/external.csproj where an old version of Microsoft.CodeAnalysis.Compilers was referenced which made AOT fail, so I upgraded it.
That uncovered a problem AOTing Microsoft.CodeAnalysis.VisualBasic.dll so had to exclude it from AOTing.

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 15, 2023
@ghost ghost assigned akoeplinger May 15, 2023
@akoeplinger akoeplinger requested a review from lateralusX May 15, 2023 18:51
@akoeplinger
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Also try to fix the StartupHookTests on LLVM FullAOT by making sure we also AOT dlls in subdirectories.
This could be a problem once we run the test in-proc.
@akoeplinger
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

It needs to match with what the rest of the repo is using.
@akoeplinger
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger
Copy link
Member Author

/azp run runtime-extra-platforms

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger
Copy link
Member Author

Failures are unrelated.

@akoeplinger akoeplinger merged commit 9f39cf0 into dotnet:main May 18, 2023
@akoeplinger akoeplinger deleted the fix-tests branch May 18, 2023 09:35
@ghost ghost locked as resolved and limited conversation to collaborators Jun 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants