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

get_default_cache writes to a roaming profile directory in recent Windows by default #763

Closed
jonmoore opened this issue Aug 26, 2016 · 1 comment

Comments

@jonmoore
Copy link

@jonmoore jonmoore commented Aug 26, 2016

In get_default_cache the first choice of directory to write to on Windows is given by %APPDATA%. In Windows Vista this is part of the roaming profile for the user, not the local one, i.e. it writes to C:\Users\username\AppData\Roaming, not C:\Users\username\AppData\Local - see https://msdn.microsoft.com/en-us/library/dd378457(v=vs.85).aspx.

This increases the roaming profile size and causes Windows login delays for users.

It may be best to add %LOCALAPPDATA% as the first possible directory to write to.

@jaraco
Copy link
Member

@jaraco jaraco commented Aug 27, 2016

Yes, that's right. It's not obvious to me that LOCALAPPDATA would be preferable, though my instinct is the same. As far as I can tell, the reference you passed doesn't give any guidance on which profile directory to use. Following appdirs as a guide, it does seem to prefer 'local' for cache. Perhaps pkg_resources should simply use appdirs.

@jaraco jaraco closed this in beec7ff Aug 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants