paket.references gets overwritten by paket install #1344

Closed
ForNeVeR opened this Issue Jan 5, 2016 · 10 comments

Comments

Projects
None yet
3 participants
@ForNeVeR
Contributor

ForNeVeR commented Jan 5, 2016

I've created new console project with Visual Studio (and closed it, so no any interference between VS and Paket), let's call it ConsoleApplication8, and done the following:

.\.paket\paket.exe init
.\.paket\paket.exe add nuget Tesla.Csxcad version 0.0.1-alpha3

Then written the following to ConsoleApplication8\paket.references:

Tesla.Csxcad

After that I've ran .paket\paket install. It overwritten my paket.references with the following:

FSharp.Data

I've set up the test repository here. That is the state I got after paket install.

Is it a bug or not? Is it okay that it removes my Tesla.Csxcad from paket.references?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jan 5, 2016

Member

something is fucky here. I'll investigate

Member

forki commented Jan 5, 2016

something is fucky here. I'll investigate

@forki forki added the bug label Jan 5, 2016

@ForNeVeR

This comment has been minimized.

Show comment
Hide comment
@ForNeVeR

ForNeVeR Jan 5, 2016

Contributor

Some additional details: that Tesla.Csxcad is written and published by me, so it could have any number of NuGet-related problems (because I'm a total NuGet noob). And it really have this FSharp.Data dependency.

Contributor

ForNeVeR commented Jan 5, 2016

Some additional details: that Tesla.Csxcad is written and published by me, so it could have any number of NuGet-related problems (because I'm a total NuGet noob). And it really have this FSharp.Data dependency.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jan 5, 2016

Member

image

;-) you have a packages.references in the content folder of you package. This file gets extracted and overwrites the existing file.

Member

forki commented Jan 5, 2016

image

;-) you have a packages.references in the content folder of you package. This file gets extracted and overwrites the existing file.

@ForNeVeR

This comment has been minimized.

Show comment
Hide comment
@ForNeVeR

ForNeVeR Jan 5, 2016

Contributor

Ah. Thanks for pointing that! Definitely my fault and not a Paket issue.

Contributor

ForNeVeR commented Jan 5, 2016

Ah. Thanks for pointing that! Definitely my fault and not a Paket issue.

@ForNeVeR ForNeVeR closed this Jan 5, 2016

@forki forki reopened this Jan 5, 2016

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jan 5, 2016

Member

I think I will add a error message here to make that more bovious.

Member

forki commented Jan 5, 2016

I think I will add a error message here to make that more bovious.

@ForNeVeR

This comment has been minimized.

Show comment
Hide comment
@ForNeVeR

ForNeVeR Jan 5, 2016

Contributor

Well, that would be really nice.

Contributor

ForNeVeR commented Jan 5, 2016

Well, that would be really nice.

@forki forki closed this in e7f4636 Jan 5, 2016

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jan 5, 2016

Member

we now skip this file and show a warning.

image

/cc @theimowski please verify that I didn't break your scenario

Member

forki commented Jan 5, 2016

we now skip this file and show a warning.

image

/cc @theimowski please verify that I didn't break your scenario

@ForNeVeR

This comment has been minimized.

Show comment
Hide comment
@ForNeVeR

ForNeVeR Jan 5, 2016

Contributor

Thank you!

Contributor

ForNeVeR commented Jan 5, 2016

Thank you!

@theimowski

This comment has been minimized.

Show comment
Hide comment
@theimowski

theimowski Jan 5, 2016

Member

No it shouldn't - I zip only paket.dependencies and paket.lock. Think I used 2.40.4 today and it was fine

Member

theimowski commented Jan 5, 2016

No it shouldn't - I zip only paket.dependencies and paket.lock. Think I used 2.40.4 today and it was fine

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jan 5, 2016

Member

thanks buddy

Member

forki commented Jan 5, 2016

thanks buddy

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