2012-03-27 3 views
6

Ich habe einen Änderungssatz zur Überprüfung eingereicht. Leider habe ich vergessen, meine Sandbox zuerst zu aktualisieren, was bedeutet, dass ich einige Änderungen in diesem Set nicht berücksichtigt habe.Wie kann ich eine Änderung in RTC (Eclipse) redigierbar machen?

So verlor ich die Option zum Hinzufügen von Änderungen zu meiner Änderungsmenge.

Ich möchte diese Änderungsgruppe nicht verwerfen, da sie wichtige Änderungen enthält. Ich möchte auch nicht 2 Änderungssätze liefern müssen, weil sie atomare Logik enthalten (Logik, die nicht geteilt werden kann).

Ich habe das Gefühl, dass die "Reverse" -Option meine Änderung zurück in einen bearbeitbaren Zustand versetzt, aber ich habe wirklich keine Ahnung, was ich hier tun soll.

Um es zusammenzufassen: Ich muss meine Änderung wieder bearbeitbar machen, damit ich sie mit einer anderen zusammenführen kann.

Wer weiß, wie ich das tun würde?

Thx, Jungs herrschen!

+0

Ich stieß auf ein ähnliches Problem. Ich habe eine Änderung eingestellt, um abzuschließen (nicht sicher, warum ich das getan habe) und dann habe ich versucht, es zu liefern. Aber dann blockierte der Teamberater die Lieferung, weil in einer Datei ein erforderlicher Kommentarkopf fehlte. An diesem Punkt war ich festgefahren, ich konnte die Datei nicht ändern, weil der Änderungssatz vollständig war und ich den Änderungssatz nicht liefern konnte. Ich habe es durch Überschreiben der Kommentarregel behoben und dann den Kommentar in einem anderen Änderungssatz hinzugefügt. Dies scheint eine ziemlich schlechte Falle zu sein, wie sich in einer Sackgasse in einem Videospiel verfangen und das Spiel zurücksetzen zu müssen. –

Antwort

7

Ich glaube nicht, dass Sie für Ihren Änderungssatz in einen änderbaren Status zurückkehren können, wenn dieser Änderungssatz "abgeschlossen" wurde, bevor er zur Überprüfung eingereicht wurde.

In diesem Fall wird ein „Reverse“ (dh eine neue changeset der vorherigen changeset Cancelling tun), durch eine neue changeset gefolgt, in dem Sie Ihre Arbeit wiederholen und erneut zur Überprüfung einreichen könnte die einzige Lösung sein.

Nach diesem Beispiel von code review in RTC sollten Änderungssätze jedoch während der Überprüfung änderbar sein (für den ursprünglichen Programmierer, um neue Revisionen seiner Dateien basierend auf dem Feedback der Rezensenten einzureichen).

+1

Ich fange an, dich zu lieben: D –

4

Sie sollten den neuen Änderungssatz erstellen.

Ich sage dies aus zwei Gründen:

1) Das ästhetische Argument für nur eine Änderung schnell pro Arbeitseinheit festgelegt, das bricht in der Praxis nach unten - es ist einfach eine Änderung zu vergessen, und Sie können machen müssen Änderungen aufgrund von Fehlern oder Kommentaren.

2) Wenn mehrere Änderungssätze vorhanden sind, sind Ihre Änderungen einfacher zu verstehen. Jeder Änderungssatz kann einen logischen Satz von Änderungen enthalten, so dass ein einzelnes Arbeitselement drei Änderungssätze enthalten kann: "Refactor-Code", "Aktualisieren Sie die Urheberrechte" und "Änderungen von der Überprüfung". Wenn jemand die Dateien in Zukunft kommentiert, werden sie etwas feinkörniger als das ursprüngliche Arbeitselement erhalten.

In Bezug auf das "atomische Logik" -Argument: Es ist wahrscheinlich kein Problem, es sei denn, Ihr Team ist in der Angewohnheit, einzelne Änderungssätze zu liefern/verwerfen. Im RTC-Projekt teilen wir regelmäßig logisch getrennte Änderungen über mehrere Änderungssets und mehrere Komponenten auf.

Wenn Sie Bedenken haben, dass Sie Änderungssets liefern können, die logisch von Änderungen in anderen Komponenten abhängen (wie ich es gelegentlich tue), empfehle ich Ihnen, auf bug 150421 zu klingeln. Bug 153907 beschreibt ein ähnliches Problem, erfordert jedoch eine viel komplexere Lösung (wodurch es weniger wahrscheinlich ist, dass sie ohne Kundendruck implementiert wird).

+2

Stimme voll und ganz zu den mehreren Änderungssätzen Teil. +1 – VonC

+0

In diesem Fall werde ich einen Kommentar abgeben! :) – Erigami

+0

+1 vom Autor auch. Nicht, dass +1 von dir oder irgendetwas benötigt wird: P –

1

Ich stieß auf das gleiche Problem und entschied, einen Patch zu erstellen, verwarf meine Änderungen und dann einen neuen Änderungssatz zu erstellen.

Verwandte Themen