search

Pfuenzle / anime-loads

17 stars 7 forks source link

Config ist im Hauptverzeichnis #6

Closed Mega-Bits closed 3 years ago

Mega-Bits commented 3 years ago

Hey,

habe eben mal das Update gezogen und beim neu erstellen der Daten ist mir aufgefallen das anibot.py seine Config im Hauptverzeichnis anlegt.

als bsp.

/opt/anime-loads/anibot.py
/config/ani.json

Habe ungern die Config außerhalb der Anwendung ;)

Pfuenzle commented 3 years ago

Das war ein workaround für docker, damit in dem gemounteten Ordner nur die config und nicht die .py liegt. Hab auf meinem lokalen Source gestern noch einen Check eingebaut, dass er den Pfad nur nimmt, falls die App in docker läuft und es ansonsten neben der .py speichert. Hab das allerdings noch nicht gepusht, kommt heute Mittag :)

Mega-Bits commented 3 years ago

Habe das jetzt einfach manuell geändert ;)

Pfuenzle commented 3 years ago

Hab für die Configdatei jetzt einen Parameter eingebaut(https://github.com/Pfuenzle/anime-loads/commit/6668d1bafdf0de52bba299c451e7d93856eef120), damit beide Use Cases funktionieren:

--configfile (z.B. ./config.txt, /home/user/season_2020.json)

Wenn der Parameter gegeben ist, wird die Config datei als die gegebene gesetzt, falls nicht, fällt das Script standardmäßig auf ./config/ani.json zurück. Dadurch kann der Container mit /config gemountet werden und du kannst mit--configfile ./ani.json die Configdatei im gleichen Ordner wie das Script haben