Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002133EresseaGeneralöffentlich2017-03-02 08:15
ReporterLorsch Bearbeitung durchEnno  
PrioritätnormalSchweregradkleinerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungnicht reproduzierbar 
Behoben in Version3.6 
Zusammenfassung0002133: Falsches Zeitalter im CR
Beschreibung

Runde 943 steht in meinem CR:

1;Zeitalter

Im NR scheint alles richtig zu sein.

TagsKeine Tags zugeordnet.
Parteibu
SpielE2
Report943

Notizen / Dateien

Enno

Enno

2015-09-04 13:11

Administrator   ~0006104

Ich glaube, das ist kein neuer Fehler, sondern immer schon so gewesen. Ich habe den Fehler gefunden, und sehe ihn auch in alten Versionen des Codes (man will ja wissen, wann sich der eingeschleppt hat).

Problem ist, dass in conf/e2/config.xml ein Eintrag fehlt:

Bei den anderen Spielen ist der vorhanden, steht auf 3, und jeweils direkt unter study.speedup, bei E2 fehlt er, und auch historisch ist der nie dort gewesen, scheint mir. Zum Beispiel in Version 3.4.1: https://github.com/eressea/server/blob/hotfix-3.4.1/conf/e2/config.xml

Ich werde das reparieren. aber es wundert mich schon, dass das noch nie jemand gemerkt hat.

Enno

Enno

2015-09-04 14:26

Administrator   ~0006105

Neue Ausgabe im CR (bei der Gelegenheit auch mal Build zu einem String gemacht, mit Anführungszeichen drum): 2;Zeitalter "3.6.2";Build

Enno

Enno

2015-09-04 14:41

Administrator   ~0006106

Wird gefixt in Version 3.6.2 (voraussichtlich 12.09.2015).

Lorsch

Lorsch

2015-09-09 19:23

Reporter   ~0006125

Danke für's fixen. Nur zur Info: das ist nicht schon immer so gewesen. Z.B. in Runde 942 (und vorher):

942;Runde 2;Zeitalter 3.5.6;Build

…und dann eben erst ab Runde 943:

943;Runde 1;Zeitalter 3.6.1;Build

Mir ist es sofort aufgefallen, als Magellan als Datum "Juli 79" angezeigt hat.

Enno

Enno

2015-09-10 15:37

Administrator   ~0006126

Danke, vielleicht schaue ich noch mal rein, was da los war.

Enno

Enno

2015-09-10 17:07

Administrator   ~0006127

In revision 0de5d698de0316c6a301053df32a7f55f11c2cf4 hat sich der default geändert, von 2 auf 1. Im gleichen commit sollte sich auch conf/e2/config.xml geändert haben, mit world.era=2, um das zu kompensieren.

Die Frage ist wohl, warum das jetzt in Runde 943 nicht mehr in der XML Konfiguration war? Relevantes diff:

 <param name="init_spells" value="0"/>
Enno

Enno

2015-09-10 17:09

Administrator   ~0006128

Im master sind die Änderungen aus dem commit alle nicht drin! Auch nicht die seed.population Einstellungen. Da scheint beim merge etwas verloren gegangen zu sein. Muss ich mir wohl mal den git Graphen ansehen.

Enno

Enno

2015-09-10 17:15

Administrator   ~0006129

Das ganze ging wohl im branch feature/bug-xinclude-catalog vor sich. Der wurde in develop gemerged, 08. Juni 2015. Danach ist es kaputt? Das klingt nach einer Aufgabe für git bisect!

Enno

Enno

2015-09-10 17:36

Administrator   ~0006130

git bisect sagt: 85230f7d115a3b88652f45f9b717ff7b508eabc5 is the first bad commit Dann wollen wir mal schauen, warum...

Enno

Enno

2015-09-10 18:54

Administrator   ~0006131

Da scheine ich Unfug mit git rebase gemacht zu haben, was git bisect verwirrt, und möglicherweise den Fehler verursacht hat. Das mache ich einfach nie wieder, dann sollte das nicht mehr passieren.

Ich werde mal die beiden fehlenden config.xml Einträge für seed.population nachtragen, auch wenn die für das laufende Spiel keine Auswirkung haben.

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2015-09-03 21:00 Lorsch Neuer Eintrag
2015-09-03 22:04 Enno Bearbeitung durch => Enno
2015-09-03 22:04 Enno Status neu => zugewiesen
2015-09-04 13:11 Enno Notiz hinzugefügt: 0006104
2015-09-04 14:26 Enno Notiz hinzugefügt: 0006105
2015-09-04 14:41 Enno Notiz hinzugefügt: 0006106
2015-09-04 14:41 Enno Status zugewiesen => erledigt
2015-09-04 14:41 Enno Behoben in Version => 3.6
2015-09-04 14:41 Enno Lösung offen => erledigt
2015-09-09 19:23 Lorsch Notiz hinzugefügt: 0006125
2015-09-09 19:23 Lorsch Status erledigt => Rückmeldung
2015-09-09 19:23 Lorsch Lösung erledigt => wiedereröffnet
2015-09-10 15:37 Enno Notiz hinzugefügt: 0006126
2015-09-10 17:07 Enno Notiz hinzugefügt: 0006127
2015-09-10 17:09 Enno Notiz hinzugefügt: 0006128
2015-09-10 17:15 Enno Notiz hinzugefügt: 0006129
2015-09-10 17:36 Enno Notiz hinzugefügt: 0006130
2015-09-10 18:54 Enno Notiz hinzugefügt: 0006131
2015-09-10 18:54 Enno Status Rückmeldung => erledigt
2015-09-10 18:54 Enno Lösung wiedereröffnet => nicht reproduzierbar
2017-03-02 08:15 Enno Status erledigt => geschlossen