install.sh: interaktive Abfragen und verbesserte Ersteinrichtung #4

2026-04-20T12:47:40+02:00

martin commented

2026-04-20 12:47:40 +02:00

Zusammenfassung

.env-Verschiebung automatisieren: Wird eine .env im Projektverzeichnis gefunden, fragt das Skript jetzt nach und führt cp + chmod 600 + rm bei Bestätigung direkt aus
Foto-Verzeichnis interaktiv abfragen: LOCAL_PHOTO_DIR wird nicht mehr stillschweigend als Default gesetzt, sondern per read abgefragt — mit Hinweis falls das Verzeichnis bereits existiert
Ausführlicher .env-Hinweis: Nach Anlegen der Default-Konfiguration werden die Pflichtfelder einzeln erklärt und der nächste Schritt (install.sh erneut ausführen) genannt
bats-Check entfernt: War nur für Entwickler relevant, nicht für Endnutzer
Testfixes: Umlaut-Mismatch "laeuft" → "läuft" in security.bats behoben; teardown() in setup.bash ergänzt, um Lock-Leakage zwischen Tests zu verhindern (31/31 Tests grün)

Akzeptanzkriterien

.env im Projektverzeichnis → Skript fragt und verschiebt bei J/Enter automatisch — verifiziert durch Code-Review
LOCAL_PHOTO_DIR wird interaktiv abgefragt, Vorschlag zeigt aktuellen Wert — verifiziert durch Code-Review
Vorhandenes Verzeichnis wird im Prompt kenntlich gemacht — verifiziert durch Code-Review
Neue .env enthält ausführlichen Hinweis mit Pflichtfeldern und nächsten Schritten — verifiziert durch Code-Review
bats-Prüfung entfernt — verifiziert durch Code-Review
Alle 31 Tests grün — verifiziert durch bats tests/

## Zusammenfassung - **`.env`-Verschiebung automatisieren:** Wird eine `.env` im Projektverzeichnis gefunden, fragt das Skript jetzt nach und führt `cp` + `chmod 600` + `rm` bei Bestätigung direkt aus - **Foto-Verzeichnis interaktiv abfragen:** `LOCAL_PHOTO_DIR` wird nicht mehr stillschweigend als Default gesetzt, sondern per `read` abgefragt — mit Hinweis falls das Verzeichnis bereits existiert - **Ausführlicher `.env`-Hinweis:** Nach Anlegen der Default-Konfiguration werden die Pflichtfelder einzeln erklärt und der nächste Schritt (`install.sh` erneut ausführen) genannt - **`bats`-Check entfernt:** War nur für Entwickler relevant, nicht für Endnutzer - **Testfixes:** Umlaut-Mismatch `"laeuft"` → `"läuft"` in `security.bats` behoben; `teardown()` in `setup.bash` ergänzt, um Lock-Leakage zwischen Tests zu verhindern (31/31 Tests grün) ## Akzeptanzkriterien - [x] `.env` im Projektverzeichnis → Skript fragt und verschiebt bei `J`/Enter automatisch — verifiziert durch Code-Review - [x] `LOCAL_PHOTO_DIR` wird interaktiv abgefragt, Vorschlag zeigt aktuellen Wert — verifiziert durch Code-Review - [x] Vorhandenes Verzeichnis wird im Prompt kenntlich gemacht — verifiziert durch Code-Review - [x] Neue `.env` enthält ausführlichen Hinweis mit Pflichtfeldern und nächsten Schritten — verifiziert durch Code-Review - [x] `bats`-Prüfung entfernt — verifiziert durch Code-Review - [x] Alle 31 Tests grün — verifiziert durch `bats tests/`

martin added 1 commit 2026-04-20 12:47:41 +02:00

install.sh: interaktive Abfragen bei Installation verbessert 0c5774f695

- .env-Verschiebung aus Projektverzeichnis wird angeboten und bei Bestätigung automatisch ausgeführt
- Lokales Foto-Verzeichnis wird interaktiv abgefragt (mit Hinweis falls es bereits existiert)
- Ausführlicher Hinweis nach Anlegen der Default-.env mit Pflichtfeldern und nächsten Schritten
- bats-Hinweis entfernt (nur für Entwickler relevant)
- Tests: Umlaut-Mismatch in security.bats behoben, teardown() für Lock-Isolation ergänzt

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

This branch is already included in the target branch. There is nothing to merge.

This branch is out-of-date with the base branch

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feat/install-interactive-prompts:feat/install-interactive-prompts

git checkout feat/install-interactive-prompts

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: martin/darktable-sync#4