Skip to content

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Nov 16, 2016

I spent some time looking into FSI.EXE on CoreCLR

After this PR you can run

fsi tests\scripts\fscc.fsx
fsi tests\scripts\fsci.fsx

These are shims for CoreCLR fsc.exe and fsi.exe respectively

  • the first fsi is the .NET Framework fsi used to run the shim
  • these then respectively invoke the CoreCLR fsc.exe and fsi.exe with sensible defaults. The fscc.fsx is an evolution of CompileProj.fsx and replaces it.

This also makes more improvements to the fsharp suite.

@dsyme dsyme changed the title Make it easier to play with F# Interactive on CoreCLR [WIP] Make it easier to play with F# Interactive on CoreCLR Nov 16, 2016
@dsyme
Copy link
Contributor Author

dsyme commented Nov 17, 2016

@KevinRansom I did some more work on this

  • Enable one "fsharp" suite test that drives the CoreCLR compiler
  • Make some improvements to the fsci.fsx driver
  • Make sure we only have one copy of FSharp.Compiler.Interactive.Settings loaded through the better techniques we discussed last night, where we no longer access this component directly from fsi.fs. It turns out these changes were very similar to a set of changes in FSharp.Compiler.Service, so I incorporated those changes into fsi.fs.

I think it's ready to go through once we're green

@dsyme dsyme changed the title [WIP] Make it easier to play with F# Interactive on CoreCLR Additional testing for CoreCLR and make it easier to play with F# Compiler and Interactive on CoreCLR Nov 17, 2016
@mmitche
Copy link
Member

mmitche commented Nov 17, 2016

@dotnet-bot test this please (test ci move)

@dsyme
Copy link
Contributor Author

dsyme commented Nov 17, 2016

@mmitche Could you explain what the "test this please" thing is? Do we need to do it or are you testing something? Just curious. Also should we update the badges on README.md?

@mmitche
Copy link
Member

mmitche commented Nov 17, 2016

@dsyme it restarts all the tests on the PR. I moved you to the other CI server and wanted them to kick off again there.

It looks pretty decent so far, so I'll probably update the badges in a bit.

@dsyme
Copy link
Contributor Author

dsyme commented Nov 18, 2016

@dotnet-bot test this please (heisen failure in StructLayout test, seems unrelated)

@dsyme dsyme merged commit f31f26f into dotnet:master Nov 18, 2016
@KevinRansom
Copy link
Contributor

@dsyme can you use Squash and Merge when accepting PR's pls.

@dsyme
Copy link
Contributor Author

dsyme commented Nov 21, 2016

@KevinRansom Sorry my bad - I meant to

@KevinRansom
Copy link
Contributor

@dsyme ... lol. It's not a problem ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants