Doku und Vorlage auf build-conf.txt umstellen (Karte 961) - #80
Merged
Conversation
Das README beschrieb einen Weg, der nicht funktioniert: es liess Consumer-Projekte eine
plaintext-build.cfg anlegen, waehrend load_build_conf() ausschliesslich build-conf.txt sucht.
Wer der Anleitung folgte, bekam "ERROR: build-conf.txt not found".
Gemessen in plaintext-scripts/*.sh:
build-conf.txt 27 Vorkommen
plaintext-build.cfg 0 (ausser einem Kommentar)
PLAINTEXT_BUILD_CONFIG 0 <- die Doku fuehrte es als Prioritaet 2
(Kontrolle: load_build_conf 4, PLAINTEXT_CONFIG_DIR 17)
Aenderungen:
* plaintext-build.cfg.template -> build-conf.txt.template (git mv, Inhalt unveraendert)
* README Abschnitt 2: legt jetzt build-conf.txt an, mit Hinweis dass der Dateiname nicht frei
waehlbar ist
* README "Configuration": die vierstufige Prioritaetenliste war zur Haelfte erfunden. Ersetzt
durch die beiden Orte, die load_build_conf() tatsaechlich absucht. Dazu der Hinweis, dass die
Werte per export gesetzt werden und gleichnamige Umgebungsvariablen damit UEBERSCHRIEBEN
werden -- die Doku behauptete das Gegenteil ("Individual environment variables, highest wins")
* tui-modules-logic.sh:28: Kommentar "from build-conf.txt or plaintext-build.cfg"
richtiggestellt; der Code darunter las immer nur build-conf.txt
Die sechs plaintext-build.cfg in plaintext-config entfernt PR 86 dort.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MNuJVEMABbAJUHt3gxKHrc
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Das README beschrieb einen Weg, der nicht funktioniert
Abschnitt 2 liess Consumer-Projekte eine
plaintext-build.cfganlegen, währendload_build_conf()ausschließlichbuild-conf.txtsucht. Wer der Anleitung folgte, bekamERROR: build-conf.txt not found.Gemessen in
plaintext-scripts/*.sh:Was sich ändert
1. Vorlage umbenannt, Inhalt unverändert:
plaintext-build.cfg.template→build-conf.txt.template(pergit mv).2. Abschnitt 2 legt jetzt
build-conf.txtan, mit dem Hinweis, dass der Dateiname nicht frei wählbar ist.3. Die Prioritätenliste war zur Hälfte erfunden. Sie versprach vier Ebenen, davon existieren zwei nicht:
Dazu ein Hinweis, der praktisch wichtig ist:
load_build_conf()setzt die Werte perexportund überschreibt damit gleichnamige Umgebungsvariablen. Eine Variable vorher zu setzen wirkt also nicht als Übersteuerung — die Doku behauptete mit „Individual environment variables (highest wins)" genau das Gegenteil.4.
tui-modules-logic.sh:28: der Kommentarread WEBAPP_MODULE from build-conf.txt or plaintext-build.cfgist richtiggestellt. Der Code darunter las immer nurbuild-conf.txt— dieser Kommentar war der einzige Treffer im ganzen Skript-Repo und hat bei der Untersuchung zu Karte 961 zunächst eine falsche Fährte gelegt.Zusammenhang
Die sechs
plaintext-build.cfginplaintext-configentfernt Plaintext-Gmbh/plaintext-config#86. Beide PRs zusammen stellen sicher, dass es nur noch eine Build-Konfiguration gibt — eine Korrektur in der falschen Datei hat bei Karte 955 einen Deploy-Zyklus gekostet.Was ich NICHT geprüft habe
Ob außerhalb dieser Organisation jemand die Vorlage unter ihrem alten Namen einbindet. Innerhalb wird sie nur vom README referenziert (2 Stellen, beide angepasst).