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

handle application data folder is read only #1661

Merged

Conversation

yakkle
Copy link
Contributor

@yakkle yakkle commented Feb 25, 2020

when using docker, if user_data_dir() isn't writable directory,
default_data_dir() use system temp directory + virtualenv.
for example, tempdir is /tmp, it use /tmp/virtualenv

Thanks for contributing a pull request, see checklist all is good!

  • wrote descriptive pull request text
  • added/updated test(s)
  • updated/extended the documentation
  • added news fragment in docs/changelog folder

@gaborbernat
Copy link
Contributor

I think we need a better solution for this in a sense that we should clean-up the temporay folder when this is not the app-data.

@gaborbernat
Copy link
Contributor

@yakkle thanks for starting the work on this, I'll take up continuing this PR, this requires major changes to make it right.

@gaborbernat gaborbernat force-pushed the feature/fix-not-writable-user-data-dir branch from dc55b29 to f151f37 Compare February 25, 2020 13:21
 - when using docker, if `user_data_dir()` isn't writable directory,
   `default_data_dir()` use `system temp directory` + `virtualenv`.
   for example, tempdir is `/tmp`, it use `/tmp/virtualenv`
@gaborbernat gaborbernat force-pushed the feature/fix-not-writable-user-data-dir branch 3 times, most recently from 107c479 to b2994d4 Compare February 25, 2020 18:29
@gaborbernat gaborbernat changed the title fixed FileNotFoundError when directory isn't writable (#1640) handle application data folder is read only Feb 25, 2020
@gaborbernat gaborbernat force-pushed the feature/fix-not-writable-user-data-dir branch 2 times, most recently from 0c0f3ad to 68f030e Compare February 25, 2020 20:54
@gaborbernat gaborbernat force-pushed the feature/fix-not-writable-user-data-dir branch from 68f030e to e8eee79 Compare February 25, 2020 23:45
@gaborbernat gaborbernat merged commit c3453b6 into pypa:master Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants