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

Allow override of NuGetCacheFolder location through environment variable #2035

Merged
merged 2 commits into from Nov 23, 2016

Conversation

Projects
None yet
2 participants
@chappoo

chappoo commented Nov 22, 2016

The NuGet CLI (2.x) provides an environment variable override for specifying the location of the NuGet cache. This is key when running multiple processes that pull NuGet packages to avoid file locks in the cache (i.e. in a multi- build agent per server environment). This change enables support for the override in Paket, via an environment variable (e.g. Windows PS):

$env:NuGetCachePath = "C:\Custom\Cache\Location"

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 23, 2016

Member

can you please add this to the docs as well?

Member

forki commented Nov 23, 2016

can you please add this to the docs as well?

Steve Chapman
@chappoo

This comment has been minimized.

Show comment
Hide comment
@chappoo

chappoo Nov 23, 2016

Done - also added a bit of extra blurb about the machine cache, as it seems it's used regardless of any additional caches. If you'd prefer it elsewhere, happy to do so?

chappoo commented Nov 23, 2016

Done - also added a bit of extra blurb about the machine cache, as it seems it's used regardless of any additional caches. If you'd prefer it elsewhere, happy to do so?

@chappoo

This comment has been minimized.

Show comment
Hide comment
@chappoo

chappoo Nov 23, 2016

Also, looks like there's problem with the CI checks.. unrelated?

chappoo commented Nov 23, 2016

Also, looks like there's problem with the CI checks.. unrelated?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 23, 2016

Member

awesome. thx

Member

forki commented Nov 23, 2016

awesome. thx

@forki forki merged commit fcc61dc into fsprojects:master Nov 23, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment