Skip to content

Commit

Permalink
Fix Mono.Linker.Tests build (#102102)
Browse files Browse the repository at this point in the history
And ensure changes to ProjectReferences trigger linker tests
  • Loading branch information
sbomer committed May 11, 2024
1 parent dc98263 commit 31527d1
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 6 deletions.
2 changes: 2 additions & 0 deletions eng/pipelines/common/evaluate-default-paths.yml
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,8 @@ jobs:
- subset: tools_illink
include:
- src/tools/illink/*
- src/coreclr/tools/aot/ILCompiler.DependencyAnalysisFramework/*
- src/coreclr/tools/ILVerification/*
- global.json

- subset: tools_cdacreader
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ public ILVerifier (NPath[] searchDirectories, string systemModuleName)
IncludeMetadataTokensInErrorMessages = true
});

_verifier.SetSystemModuleName (new AssemblyName (systemModuleName));
_verifier.SetSystemModuleName (new AssemblyNameInfo (systemModuleName));
}

public ILVerifierResult[] VerifyByName (string assemblyName)
Expand Down Expand Up @@ -132,10 +132,10 @@ bool TryLoadAssemblyFromFolder (string assemblyName, NPath folder, [NotNullWhen
return null;
}

PEReader? IResolver.ResolveAssembly (AssemblyName assemblyName)
PEReader? IResolver.ResolveAssembly (AssemblyNameInfo assemblyName)
=> Resolve (assemblyName.Name ?? assemblyName.FullName);

PEReader? IResolver.ResolveModule (AssemblyName referencingModule, string fileName)
PEReader? IResolver.ResolveModule (AssemblyNameInfo referencingModule, string fileName)
=> Resolve (Path.GetFileNameWithoutExtension (fileName));

public void Dispose ()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
using System.IO;
using System.Linq;
using System.Reflection;
using System.Reflection.Metadata;
using System.Reflection.PortableExecutable;
using System.Runtime.Loader;
using ILVerify;
Expand Down Expand Up @@ -42,7 +43,7 @@ public ILVerifier (NPath assemblyPath)
SanityChecks = true,
IncludeMetadataTokensInErrorMessages = true
});
_verifier.SetSystemModuleName (new AssemblyName ("mscorlib"));
_verifier.SetSystemModuleName (new AssemblyNameInfo ("mscorlib"));

var allResults = _verifier.Verify (Resolve (assemblyName))
?? Enumerable.Empty<VerificationResult> ();
Expand Down Expand Up @@ -118,10 +119,10 @@ bool TryLoadAssemblyFromFolder (string assemblyName, NPath folder, [NotNullWhen
return null;
}

PEReader? ILVerify.IResolver.ResolveAssembly (AssemblyName assemblyName)
PEReader? ILVerify.IResolver.ResolveAssembly (AssemblyNameInfo assemblyName)
=> Resolve (assemblyName.Name ?? assemblyName.FullName);

PEReader? ILVerify.IResolver.ResolveModule (AssemblyName referencingModule, string fileName)
PEReader? ILVerify.IResolver.ResolveModule (AssemblyNameInfo referencingAssembly, string fileName)
=> Resolve (Path.GetFileNameWithoutExtension (fileName));

public static string GetErrorMessage (VerificationResult result)
Expand Down

0 comments on commit 31527d1

Please sign in to comment.