Skip to content
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

[WIP] Cache downloaded files by default #1734

Closed
wants to merge 2 commits into from

Conversation

dstufft
Copy link
Member

@dstufft dstufft commented Apr 20, 2014

  • Enables a speed up by caching downloaded files by default
  • Will use the appropiate cross platform location to store cache files
  • Adds a new option, --no-download-cache to disable the download cache

This still requires tests.

Fixes #1732.

* Enables a speed up by caching downloaded files by default
* Will use the appropiate cross platform location to store cache
  files
* Adds a new option, `--no-download-cache` to disable the download
  cache
@dstufft dstufft changed the title [WIP] Cache by default [WIP] Cache downloaded files by default Apr 20, 2014
@dstufft dstufft added this to the Improve our User Experience milestone Apr 20, 2014
@dstufft
Copy link
Member Author

dstufft commented Apr 20, 2014

This is blocked on waiting on a decision on #1733 if the appdirs locations are appropriate or not.

@Ivoz
Copy link
Contributor

Ivoz commented Apr 20, 2014

Would be nice to refactor the cache code for issues #1634 / #1673 before landing this

@dstufft
Copy link
Member Author

dstufft commented May 3, 2014

Closing this.

@dstufft dstufft closed this May 3, 2014
@dstufft dstufft deleted the cache-by-default branch May 3, 2014 01:36
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants