lbryio/lbry-sdk

Ensure that default download directory is writable on daemon startup

开放

#3,270 创建于 2021年4月20日

在 GitHub 查看贡献者指南
 (10 条评论) (0 个反应) (0 位负责人)Python (500 个派生)batch import
area: downloadsgood first issuehelp wantedtype: improvement

仓库指标

星标
 (7,220 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

Creating a new issue based on lbryio/lbry-sdk#877

  • Update ensure_directory_exists() to check that the directory is writable by the current process.
  • If it's not, return a useful error
  • Add tests to cover the case when
    • download directory does not exist
    • download dir exists but is not writable
    • dir exists and is writable
    • the path refers to a file not a directory

bonus: also check if disk is full when sdk starts. this check should be outside ensure_directory_exists(). it probably goes somewhere near here

贡献者指南

研究方向
更新 `lbry/extras/cli.py` 中的 `ensure directory exists()` 函数，以验证下载目录是否可写，并在不可写时返回有用的错误信息。添加单元测试，覆盖以下情况：目录不存在、目录存在但不可写、目录存在且可写、以及路径是一个文件。可选地，在第 296 行附近添加磁盘空间检查。请参阅相关议题 lbryio/lbry sdk#877 以及仓库中现有的测试模式。
技术栈
python
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
PythonGit
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
75