lbryio/lbry-sdk

Ensure that default download directory is writable on daemon startup

Open

#3,270 建立於 2021年4月20日

在 GitHub 查看
 (10 留言) (0 反應) (0 負責人)Python (500 fork)batch import
area: downloadsgood first issuehelp wantedtype: improvement

倉庫指標

Star
 (7,220 star)
PR 合併指標
 (30 天內沒有已合併 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