Skip to content

Fix mixed emit-multi+/- sessions#18465

Merged
KevinRansom merged 3 commits into
dotnet:mainfrom
KevinRansom:emitmulti
May 5, 2025
Merged

Fix mixed emit-multi+/- sessions#18465
KevinRansom merged 3 commits into
dotnet:mainfrom
KevinRansom:emitmulti

Conversation

@KevinRansom

@KevinRansom KevinRansom commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Fixes: #18441

The issue is caused because multi-emit+ and multi-emit- use the same name for the assembly: multi-emit- uses a dynamic assembly named FSIASSEMBLY and multi-emit+ uses multiple memory assemblies named FSIASSEMBLY with a new version for each submission.

When a type can't be found the magic resolver does a simple name search on assemblies looking for the missing type and finds the dynamic assembly.

The fix is to use a different assembly name for multi and single.

@github-actions

github-actions Bot commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

❗ Release notes required


✅ Found changes and release notes in following paths:

Change path Release notes path Description
src/Compiler docs/release-notes/.FSharp.Compiler.Service/9.0.300.md

@T-Gro

T-Gro commented Apr 11, 2025

Copy link
Copy Markdown
Member

/run fantomas

@github-actions

Copy link
Copy Markdown
Contributor

@T-Gro

T-Gro commented Apr 25, 2025

Copy link
Copy Markdown
Member

/run fantomas

@KevinRansom KevinRansom force-pushed the emitmulti branch 2 times, most recently from 8a756ec to 521d8c4 Compare May 3, 2025 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

FSI multi-emit unstable

3 participants