[Release] Ajantis BG1 Expansion Modifikation: Diskussionsthread

[Jastey]

Die Backslashes waren mir noch nicht mal aufgefallen!

-Das hat es gefixt, danke!

 

[Dachrisma]

Pfadangaben im Code alle Backslashes in Slashes umwandeln. Backslashes funktionieren nur in Windows. In Linux oder macOS werden sie zum "escapen" von Sonderzeichen genutzt.

Da hätte ich eine Frage zu. Heißt das, man muss für jedes System (sofern man es einbinden möchte) eine eigene TP schreiben oder gibt es bei WEIDU auch die Funktion ein System abzufragen? (Vermute ich mal nicht)

 

[Argent]

Nein, Slashes funktionieren (meistens) auch unter Windows, daher würde ich in WeiDU-Skripten immer Slashes für Pfadangaben verwenden. (Ich glaube nur UNC-Pfadangaben setzen Backslashes voraus.)

 

[Jastey]

Die Historie des AjantisBG1-GitHub Repos ist jetzt übrigens ein Jammerspiel. Linux liest jede Datei, deren Groß- und Kleinschreibung sich geändert hat, als neue Datei ein. Soweit, so gut.

Ich kann mit GitHub aber nicht umgehen. Ich finde es nicht intuitiv, die Möglicheiten machen keinen Sinn, ich kann zwar Commits zurücknehmen (angeblich), aber wenn ich zwei hintereinander reversen möchte, dann kommen plötzlich Fehlermeldungen, dass das nicht geht, ich kann nicht mergen, wenn Konflikte auftreten, weil ich nicht sehe, wie und wo, ich HASSE es.
So, das musste mal raus.

 

[Jastey]

In Ajantis BG1s GitHub gibt es immer noch einen Konflikt. Falls jemand mir zumindest sagen kann, woran sich GitHub hier stört, ich sehe es nicht.
https://github.com/Gitjas/Ajantis_B...r...Gibberlings3:Ajantis_BG1_Expansion:master

 

[Ychap]

Auf dem master-Branch von Gibberlings3 ist dieser Commit Fix encoding in Translations/German/GAME.TRA enthalten.

Auf deinem master-Branch benennst du diese Datei aber auch um. Dadurch entsteht ein Mergekonflikt.

 

[Jastey]

Aah. Vielen Dank.

 

[Jastey]

@Ychap ich habe das jetzt händisch gelöst. (Mein Trivial-Fixversuch klappte natürlich nicht. Nicht lustig machen.) Du kannst den PR also schließen. Entschuldige die Umstände.
https://github.com/Gitjas/Ajantis_BG1_Expansion/commit/cf968a156c3e8137c1f6ffa326a6e974216d5ec3

 

[Ychap]

Keine Ursache, ich habe im Leben auch schon ein einige Nerven wegen Mergekonflikts verloren.

Ein Problem ist, dass ich @24 in GAME.TRA anders angepasst und am Ende der Datei eine neue Zeile hinzugefügt hatte.
Ein weiteres Problem ist, dass die Dateien auf deinem master-Branch in einem Commit gelöscht und in einem Anderem dann unter neuen Namen erstellt werden. Dadurch checkt git nicht mehr, dass die Dateien eigentlich nur umbennannt wurden.

 

[Jastey]

Ein Problem ist, dass ich @24 in GAME.TRA anders angepasst und am Ende der Datei eine neue Zeile hinzugefügt hatte.

Ah ok, das muss ich mir dann nochmal ansehen.

 

[Jastey]

@Ychap https://github.com/Gitjas/Ajantis_BG1_Expansion/commit/26cb09380ef96bb88c9b8e5e27cbee29806ca5ae

Was meinst Du mit am Ende der Datei eine neue Zeile?

 

[Ychap]

Dass ich am Ende der Datei eine leere Zeile hinzugefügt habe.

Das ist auch nur daran zu erkennen, weil dort kein "⛔" ist:
https://github.com/Gibberlings3/Aja...0fa4560eea0f10f04225f848ea27e06fae95778ce1R93

 

[Jastey]

Ah! Ist das für etwas nützlich? Ich habe kein Problem damit, noch eine Zeile einzufügen.

 

[Ychap]

Eigentlich nicht. Das macht mein Editor einfach immer automatisch, wenn ich etwas abspeichere.

 

[Jastey]

OK, dann sollten Deine Änderungen jetzt drin sein.

 

[Jastey]

Ich habe den Konflikt gelöst bekommen. Ich weiß nicht mehr genau, wieso, aber er hat mir beim Pushen von meinem Rechner aus angeboten, die Datei mit einem Editor zu bearbeiten, und das hat geklappt. Das ist schonmal schön zu wissen.
Gibt nichts zu sehen hier, bitte weitergehen!

 

[Jastey]

Die Mod updatet zur v22. Die Textformatierung in der EE sollte nun korrekt sein.

Changelog:
-change backslashes to slashes in path names in tp2 (fix text encoding errors on EE for Linux).
-Set correct index of TLK file in EET.
-typo corrections for English and Simplified Chinese.
-German: fix wrong text encoding.