lbryio/lbry-sdk

Ensure that default download directory is writable on daemon startup

Open

#3.270 geöffnet am 20. Apr. 2021

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (500 Forks)batch import
area: downloadsgood first issuehelp wantedtype: improvement

Repository-Metriken

Stars
 (7.220 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Aktualisiere `ensure directory exists()` in `lbry/extras/cli.py`, um zu überprüfen, ob das Download Verzeichnis beschreibbar ist, und um eine nützliche Fehlermeldung zurückzugeben, falls nicht. Füge Unit Tests hinzu, die folgende Fälle abdecken: Verzeichnis existiert nicht, existiert aber ist nicht beschreibbar, existiert und ist beschreibbar, und Pfad ist eine Datei. Optional füge eine Plattenplatzprüfung in der Nähe von Zeile 296 hinzu. Siehe verwandtes Issue lbryio/lbry sdk#877 und vorhandene Testmuster im Repository.
Tech Stack
python
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75