cachedTypes in AssemblyNamesTypeResolutionService is unreachable

[Liminiens]

Problem description:

Here the field cachedTypes is currently unreachable because it relies on Assembly.GlobalAssemblyCache which is false in .NET Core and IsNetFrameworkAssembly, which is also always false because it tests if assembly Location path is .NET Framework path, while (on Windows) .NET Core assemblies are on C:\Program Files\dotnet\shared\Microsoft.NETCore.App\... path.

Actual behavior:

Types are not cached

Expected behavior:

.NET Core types are cached (?)

[zsd4yr]

@Liminiens do you have a proposed change to rely on other values which are not always false?

[Liminiens]

@zsd4yr i am not sure if it's a correct way, but maybe checking if assembly Location is on ..\dotnet\shared\.. path will do the job?

[zsd4yr]

@Liminiens it's been awhile here; I am wondering exactly what your scenario is in order to determine a viable solution / replacement? Could you provide a little more information about your needs here?

[zsd4yr]

@ericstj do you happen to know what the replacement for Assembly.GlobalAssemblyCache is?

[ericstj]

/cc @jeffschwMSFT
It looks like @hughbe
fixed the GAC checking here: 5e0710c#diff-a2cbb3dd218c63ebe4e00266618b9ef5R828, but this still checks IsNetFrameworkPath:

winforms/src/System.Windows.Forms/src/System/Resources/ResXDataNode.cs

Line 791 in f3ae8f6

private static string NetFrameworkPath = Path.Combine(Environment.GetEnvironmentVariable("SystemRoot"), "Microsoft.Net\\Framework");

.

Since .NETCore is an xcopy-able framework, you shouldn't really be checking any specific path dotnet\shared.

I think to really answer this question you need to understand the reasoning behind this statement:

winforms/src/System.Windows.Forms/src/System/Resources/ResXDataNode.cs

Line 927 in f3ae8f6

// Only cache types from .Net framework because they don't need to update.

I'm not sure what this code is imaging might change over the lifetime of these statics: it's not like it can unload assemblies once they've been loaded so future requests for the same type must result in the same type even if it is not cached (since the assembly is effectively cached by the runtime).

[weltkante]

@ericstj

I'm not sure what this code is imaging might change over the lifetime of these statics: it's not like it can unload assemblies once they've been loaded

Might be related to VS usage of ITypeResolutionService, after a build the WinForms Designer will load the newly built version so maybe its referring to that, to avoid having cached the old version of the assembly.