...

[floscher]

Hi Matthias, warum machst Du denn meine ganze Arbeit von gestern Abend wieder zunichte???? Ich habe da ganz schön was geschaft, damit das speichern von Favoriten, die Wikipedia-Infos und das Preprocessing besser laufen. Und nun muss ich feststellen, dass Du das alles rückgängig gemacht hast!!!!!

Kannst Du das rückgängig machen? Wenn nicht, danke für die stundenlange Zusatzarbeit, die morgen auf mich zu kommt, wenn ich die Änderungen manuell wieder einpflegen kann.

[MStumpp]

Keine Ahnung von welcher Arbeit du redest. Ich hatte heute keine Konflikte beim committen und an dem Client Sachen habe ich schon gar nichts gemacht...

[floscher]

Siehe Diff...!

Ich habe dir mal ein paar Änderungen gekennzeichnet, die Du rückgängig gemacht hast.

In den 50 (!) Dateien, die du mit diesem Pull-Request bearbeitet hast, habe ich fast ausschließlich Änderungen gesehen, die meine Änderungen an Client und Preprocessor genau wieder auf den Zustand davor zurücksetzen.

[floscher]

https://github.com/MStumpp/pse13ss13-app/pull/170/files

[MStumpp]

Da kann ich nichts dazu sagen. Ich weiß überhaupt nicht was das ist, geschweige habe ich da was dran geändert. Noch hatte ich heute Konflikte.

[floscher]

Das hier ist doch ein von Dir geöffneter Pull-Request, mit dem Du deinen Branch auf den master-Branch gemergt hast und dabei die auf dem master-branch gemachten Änderungen mit deinem Branch überschrieben hast.

Hier ist der entsprechende Commit: https://github.com/MStumpp/pse13ss13-app/commit/d066cba21ef5fec6b3dced4b58fec4476a63e8b1

Und meine entsprechenden Commits: https://github.com/MStumpp/pse13ss13-app/commit/a1309ead5417658aa87f992d8f01cfa8ad937a46 https://github.com/MStumpp/pse13ss13-app/commit/4f5e350f6734e2ee6ca918e8636183a3e74e8d41 https://github.com/MStumpp/pse13ss13-app/commit/e381e83e10809d25400651fd0433c894f127775b

[MStumpp]

Ja gut, wenn ich eine alte Version der betroffenen Dateien habe und an diesen nichts geändert habe, dann gehe ich davon aus, dass Git aktuellere, geänderte Dateien nicht mit meinen alten, nicht geänderten Dateien überschreibt. So einen Fall hat man ja permanent.

Außerdem steht doch bei den Diffs ganz oben: "MStumpp merged 5 commits into matthias-master from master about 3 hours ago". Das heißt doch, commits werden vom Master branch in meinen Matthias-Master branch geschrieben, und nicht umgekehrt.

[floscher]

Scheinbar hat das hier nicht ganz so gut funktioniert. Schaue Dir in Zukunft bitte deine Pull-Requests auf github an, da wird dir nochmal detailliert angezeigt, was in den Master-branch übernommen wird.

Darf ich davon ausgehen, dass du dann auch nicht weißt, wie man das automatisiert rückgängig machen kann? Falls ja, werde ich wohl oder übel morgen händisch ran müssen.

Gute Nacht, ich schlafe erst nochmal drüber.

[MStumpp]

Wie gesagt, https://github.com/MStumpp/pse13ss13-app/commit/d066cba21ef5fec6b3dced4b58fec4476a63e8b1 ist ein Merge vom Master zu Matthias-Master und nicht umgekehrt. Dieser Merge hat demnach nichts am Master branch geändert, sondern ausschließlich am Matthias-Master branch.

[floscher]

Und in https://github.com/MStumpp/pse13ss13-app/commit/68eb39a6d5cfda06a989b5a1131544d302a79cdd wird es dann auf master gemerged.

[floscher]

Aber danke für den Hinweis, dadurch bin ich auf eine Lösung gekommen. Habe jetzt einen Rollback durchgeführt, sodass meine Änderungen wieder drin sind.

Der master-Branch ist jetzt wieder auf dem Stand vom Commit https://github.com/MStumpp/pse13ss13-app/commit/1cb05b7a754170c167d916eb422497995d7f9d75

[MStumpp]

OK, also muss ich nichts mehr rollbacken?

[MStumpp]

Hast du denn den DropboxUtil wieder in die Tests in die shared eingefügt? Die muss durch FileUtil ersetzt werden.

[MStumpp]

Übrigens läuft die Routenberechnung schön durch... Zumindest kommt jetzt immer was zurück und meist sind das dann immer mehrere hundert Knoten. So wie es sein soll. Läuft aber aktuell nur lokal bei mir.

[floscher]

Rollback nein. Höchstens deinen branch (richtig) auf master mergen.

DropboxUtil, nein: https://github.com/MStumpp/pse13ss13-app/tree/master/WalkaRound/shared/datastructures/src/main/java/edu/kit/iti/algo2/pse2013/walkaround/shared

Routenberechnung: Gut, kannst Du schon absehen, wann das über den öffentlichen Server verfügbar sein wird?

[MStumpp]

Ja jetzt nicht mehr, weil du nen rollback gemacht hast. Die Matthias-Maste branch ist jetzt allerdings 9 commits ahead. Wenn ich die jetzt mit dem Master merge, habe wir wahrscheinlich das gleiche Problem.

Edit (von @floscher): Jetzt nicht mehr. Siehe unten.

[MStumpp]

Eigentlich heute, allerdings hatte ich regelmäßig die Situation, dass die Tests im shared package nach eingechecken Änderungen nicht durchliefen. Solange das so ist, d.h. bei Änderungen nicht lokal getestet wird und danach erst eingecheckt wird, kann ich nichts deployen.

[floscher]

Ich habe dir jetzt den matthias-master-branch auf den master gemergt: https://github.com/MStumpp/pse13ss13-app/commit/0239359703299e00e32ae290e3b422e46432c40d Und die shared-Tests laufen bei mir alle durch (mit -Xms2048M -Xss64m).

Jetzt aber endgültig Gute Nacht.

[MStumpp]

Ich habe den matthias-master schon vor ein paar Minuten gelöscht, da ich diesen frisch vom master branch erstellen möchte, um mögliche Konflikte zu vermeiden.

[MStumpp]

Tests laufen jetzt auch wieder durch. Ich setz mich dann gleich morgen früh an den öffentlichen Server...