Adding content: none to paket.dependencies && paket install does not remove installed content

[agross]

I had this in my depfile:

source https://www.nuget.org/api/v2

nuget JasmineTest 2.0.0 

I ran paket install and had the content added. I then added content: none to the depfile, executed git clean -xdf and ran pakat install again. The new content: none option was not picked up and the content was restored.

[matthid]

I can confirm that this is happening, however I was never able to repro it...

[forki]

/cc @theimowski do we support this?

[agross]

I think it might be related to detecting whether the depfile has changed between paket installs. Just a guess, though.

Alex

Alexander Groß
Tiny phone, tiny mail

On Mon, Mar 9, 2015 at 2:58 PM, Steffen Forkmann notifications@github.com
wrote:

/cc @theimowski do we support this?

Reply to this email directly or view it on GitHub:
#687 (comment)

[theimowski]

Will investigate, hopefully soon

[theimowski]

@agross I can't reproduce - you sure content: none was spelled correctly?

[agross]

Here's a reproduction: https://github.com/agross/content-repro

Uncomment the commented line in https://github.com/agross/content-repro/blob/master/paket.dependencies and remove the original nuget. Then run paket install and git status after that.

Here's my output:

$ paket install
Paket version 0.35.0.0
found: content-repro\paket.dependencies
Resolving packages:
 - JasmineTest 2.2.0
Locked version resolutions written to content-repro\paket.lock
1 second - ready.

$ git diff
diff --git a/paket.dependencies b/paket.dependencies
index 5f4c593..0e17d56 100644
--- a/paket.dependencies
+++ b/paket.dependencies
@@ -1,4 +1,4 @@
 source https://www.nuget.org/api/v2

-nuget JasmineTest
-# nuget JasmineTest content: none
+#nuget JasmineTest
+nuget JasmineTest content: none

[theimowski]

I don't think we support this option on package level, but rather on global level. See http://fsprojects.github.io/Paket/dependencies-file.html#No-content-option

[agross]

@forki added it on all levels: #587

[theimowski]

Aww sorry wasn't aware of that...

[agross]

No worries :-)

[ctaggart]

This bug applied to both 0.33.13 and 0.35.0 that I tested. It only fails when no version is specified. These all work:

nuget elmah 1.2.2 content: none
nuget Microsoft.SqlServer.Types 11.0.0 content: none
nuget Microsoft.ApplicationInsights.Web pre content: none

These do not:

nuget elmah content: none
nuget Microsoft.SqlServer.Types content: none
nuget Microsoft.ApplicationInsights.Web content: none

[theimowski]

ok so I investigated a bit more, and the key point here is that you should use paket update rather than paket install.
As per the docs:

install: Ensures that all dependencies in your paket.dependencies file are present in the `packages` directory and referenced correctly in all projects.

while

update: Recomputes the dependency resolution, updates the paket.lock file and propagates any resulting package changes into all project files referencing updated packages.

In other words, if you don't paket update, the lock file won't be affected and content: none won't be applied

[agross]

Hm... I was under the impression @forki changed the model to be more like bundler: Change something in your depfile, run install and be done with it.

[forki]

"more like" is correct. But it doesn't detect all changes, yet
On Mar 20, 2015 3:15 PM, "Alexander Groß" notifications@github.com wrote:

Hm... I was under the impression @forki https://github.com/forki
changed the model to be more like bundler: Change something in your
depfile, run install and be done with it.

—
Reply to this email directly or view it on GitHub
#687 (comment).