New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Referencing Paket.Core / Exe problems. #1958

Closed
isaacabraham opened this Issue Oct 11, 2016 · 15 comments

Comments

Projects
None yet
3 participants
@isaacabraham
Contributor

isaacabraham commented Oct 11, 2016

Referencing both Paket.Core or Paket from the NuGet feed has issues related to (I believe) the fact that we ILMerge stuff in to both versions.

With Paket.Core, you'll get errors regarding Chessie. With Paket.exe, you have issues with FSharp.Core duplicates and invalid casts.

Perhaps someone can better explain this than me :-) What's the best way to solve this though?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

Would you want to have a special package? Without ILMerged stuff?

Member

forki commented Oct 11, 2016

Would you want to have a special package? Without ILMerged stuff?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

but paket.core is not ILMerged. What issue do you see with chessie?

Member

forki commented Oct 11, 2016

but paket.core is not ILMerged. What issue do you see with chessie?

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 11, 2016

Contributor
#r @"packages\Paket.Core.3.23.1\lib\net45\Paket.Core.dll"
#r @"packages\Chessie.0.0.1\lib\net40\Chessie.dll"
open Paket
open Chessie
let dependenciesFile = Dependencies(@"paket.dependencies")

gives an error

error FS0074: The type referenced through 'Chessie.ErrorHandling.Result`2' is defined in an assembly that is not referenced. You must add a reference to assembly
 'Chessie'.

Although in the editor window it does not show any errors.

Contributor

isaacabraham commented Oct 11, 2016

#r @"packages\Paket.Core.3.23.1\lib\net45\Paket.Core.dll"
#r @"packages\Chessie.0.0.1\lib\net40\Chessie.dll"
open Paket
open Chessie
let dependenciesFile = Dependencies(@"paket.dependencies")

gives an error

error FS0074: The type referenced through 'Chessie.ErrorHandling.Result`2' is defined in an assembly that is not referenced. You must add a reference to assembly
 'Chessie'.

Although in the editor window it does not show any errors.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

is that the correct chessie version?

Member

forki commented Oct 11, 2016

is that the correct chessie version?

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 11, 2016

Contributor

That's the one that Nuget pulled down. ;-)

Contributor

isaacabraham commented Oct 11, 2016

That's the one that Nuget pulled down. ;-)

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 11, 2016

Contributor

Perhaps there's a minimum required version of Chessie that should be specified in Paket.Core?

Contributor

isaacabraham commented Oct 11, 2016

Perhaps there's a minimum required version of Chessie that should be specified in Paket.Core?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

I guess that's the root cause. https://www.nuget.org/packages/Paket.Core doesn't specify a version.

brb

Member

forki commented Oct 11, 2016

I guess that's the root cause. https://www.nuget.org/packages/Paket.Core doesn't specify a version.

brb

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

please retry

Member

forki commented Oct 11, 2016

please retry

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 11, 2016

Contributor

Same.

Contributor

isaacabraham commented Oct 11, 2016

Same.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

against chessie 0.6?

Member

forki commented Oct 11, 2016

against chessie 0.6?

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 11, 2016

Contributor

Correct

Contributor

isaacabraham commented Oct 11, 2016

Correct

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 11, 2016

Member

wtf

Member

forki commented Oct 11, 2016

wtf

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Oct 12, 2016

Member

can you zip that sampe and add it here?

Member

forki commented Oct 12, 2016

can you zip that sampe and add it here?

@isaacabraham

This comment has been minimized.

Show comment
Hide comment
@isaacabraham

isaacabraham Oct 13, 2016

Contributor

@forki let's review tomorrow / over the weekend together.

Contributor

isaacabraham commented Oct 13, 2016

@forki let's review tomorrow / over the weekend together.

@enricosada

This comment has been minimized.

Show comment
Hide comment
@enricosada

enricosada Jan 24, 2018

Collaborator

Closing, nupkg now specify Chessie version

Collaborator

enricosada commented Jan 24, 2018

Closing, nupkg now specify Chessie version

@enricosada enricosada closed this Jan 24, 2018

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