Skip to content
This repository was archived by the owner on Oct 31, 2021. It is now read-only.

Conversation

dungpa
Copy link
Contributor

@dungpa dungpa commented Mar 12, 2015

It turns out that we can use assembly resolution from F# Project System to ensure that assembly ordering is resolved correctly by default. We still keep the revised ordering if users decide to change it.

It isn't obvious to me until this conversation fsprojects/Paket#677.

@dungpa
Copy link
Contributor Author

dungpa commented Mar 14, 2015

@vasily-kirichenko Could you take a look?

@vasily-kirichenko
Copy link
Contributor

I'll take a look later today. The idea is great.

@vasily-kirichenko
Copy link
Contributor

  1. Generate from scratch
// Warning: generated file; your changes could be lost when a new file is generated.
#I __SOURCE_DIRECTORY__
#r @"..\..\packages\ExtCore\lib\net45\ExtCore.dll"
#r @"..\..\packages\FSharpx.Core\lib\40\FSharpx.Core.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.Core.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.dll"
#r @"..\..\packages\Hopac.Extras\lib\net45\Hopac.Extras.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.Platform.dll"
#r @"..\..\Libs\Microsoft.ConcurrencyVisualizer.Markers.dll"
#r @"System.Core.dll"
#r @"System.dll"
#r @"System.Numerics.dll"
#r @"System.Xml.Linq.dll"
  1. Add a reference at end (or any other place)
// Warning: generated file; your changes could be lost when a new file is generated.
#I __SOURCE_DIRECTORY__
#r @"..\..\packages\ExtCore\lib\net45\ExtCore.dll"
#r @"..\..\packages\FSharpx.Core\lib\40\FSharpx.Core.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.Core.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.dll"
#r @"..\..\packages\Hopac.Extras\lib\net45\Hopac.Extras.dll"
#r @"..\..\packages\Hopac\lib\net45\Hopac.Platform.dll"
#r @"..\..\Libs\Microsoft.ConcurrencyVisualizer.Markers.dll"
#r @"System.Core.dll"
#r @"System.dll"
#r @"System.Numerics.dll"
#r @"System.Xml.Linq.dll"
#r "System.Runtime"
  1. Compile the project
  2. The added reference disappeared.

@dungpa
Copy link
Contributor Author

dungpa commented Mar 18, 2015

It is the expected behaviour (that isn't affected by this change). We don't care about new entries; they will be removed if they don't appear in the list of references.

vasily-kirichenko added a commit that referenced this pull request Mar 18, 2015
Implement assembly resolution for FsiReferenceCommand
@vasily-kirichenko vasily-kirichenko merged commit b004049 into fsprojects-archive:master Mar 18, 2015
@dungpa dungpa deleted the assembly-resolution branch March 19, 2015 17:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants