Skip to content

Commit

Permalink
Paket: use Packages storage (fsprojects#615)
Browse files Browse the repository at this point in the history
I've been trying to get everything [building with .NET 6](fsprojects#606) and the related tooling, and have been getting CI build failures in the macOS builds, with errors that look like those described at fsprojects/Paket#3767

It looks like changing Paket to use 'packages' storage instead of 'symlink' avoids that issue, by putting the restored nuget packages directly into the local packages folder instead of symlinking them. (I don't know much about this Paket feature - the docs suggest its for reducing disk usage by avoid duplicates, which might not really be an issue for the builds?)

On a related note, we also have this issue - fsprojects#578 - where you have to either run the build as admin on windows or set some extra OS options otherwise the package restore will fail. I think this change would avoid that as well.

Note: Changing the paket storage to 'none' helped with package restore issue, but caused the 'docs' build to break for me because the Fornax tool couldn't find the libraries, whereas it works when set to 'packages'
  • Loading branch information
Numpsy authored Nov 22, 2023
1 parent 8ef5b24 commit b0f12c5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion paket.dependencies
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ group Docs
source https://api.nuget.org/v3/index.json

framework: netstandard2.0
storage: symlink
storage: packages

nuget FSharp.Formatting ~> 4.0
nuget Markdig
Expand Down
2 changes: 1 addition & 1 deletion paket.lock
Original file line number Diff line number Diff line change
Expand Up @@ -913,7 +913,7 @@ NUGET
System.Runtime.CompilerServices.Unsafe (>= 4.5.3)

GROUP Docs
STORAGE: SYMLINK
STORAGE: PACKAGES
RESTRICTION: == netstandard2.0
NUGET
remote: https://api.nuget.org/v3/index.json
Expand Down

0 comments on commit b0f12c5

Please sign in to comment.