Skip to content

Disable DllImportSearchPathsTest and StartupHookTests on mobile - #86269

Merged
akoeplinger merged 6 commits into
dotnet:mainfrom
akoeplinger:fix-tests
May 18, 2023
Merged

Disable DllImportSearchPathsTest and StartupHookTests on mobile#86269
akoeplinger merged 6 commits into
dotnet:mainfrom
akoeplinger:fix-tests

Conversation

@akoeplinger

@akoeplingerakoeplinger commented May 15, 2023

Copy link
Copy Markdown
Member

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.

@ghostghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 15, 2023
@akoeplinger

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
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
Copy Markdown
MemberAuthor

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

It needs to match with what the rest of the repo is using.
@akoeplinger

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@akoeplinger

Copy link
Copy Markdown
MemberAuthor

Failures are unrelated.

@akoeplinger
akoeplinger merged commit 9f39cf0 into dotnet:mainMay 18, 2023
@akoeplinger
akoeplinger deleted the fix-tests branch May 18, 2023 09:35
@ghostghost locked as resolved and limited conversation to collaborators Jun 17, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn 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

@akoeplinger@lateralusX