Wireshark-dev: Re: [Wireshark-dev] Wireshark code review

From: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
Date: Wed, 25 Mar 2015 10:03:49 +0100


On Wed, Mar 25, 2015 at 9:15 AM, Dario Lombardo <dario.lombardo.ml@xxxxxxxxx> wrote:
Since you now have 2 changes submitted, you should abandon one of them (do it from the web interface), then follow Alexis' suggestion about sqashing and amending, then push the final revision into the survived change (do it using the correct change-id, read it from the web interface again).

I don't know what happens if one push a change-x from a branch b1, then push the same change-x from another branch b2. Maybe the latter overwrites the first in gerrit? Anyone tried it?
From local branch ? or remote branch ? (master / master-1.12...)

On Wed, Mar 25, 2015 at 7:10 AM, Anil <anilkumar911@xxxxxxxxx> wrote:
Hi All --

I have done a mistake with submitting code for review.

My initial review is https://code.wireshark.org/review/#/c/7751

I fixed the issues reported there and have checked in the new changes. The problem is that the new changes are in a different branch than the original one. So it has created a new review https://code.wireshark.org/review/#/c/7802/

I need some help about what can be done now ?

--Anil

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe


___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe