Wireshark mailing list archives
Re: Adding a new dissector - beginners guide
From: Thomas Wiens <th.wiens () gmx de>
Date: Fri, 22 Aug 2014 16:15:42 +0200
On 22 August 2014 16:05, wrote Graham Bloice:
As I noted on the review, I think you must have removed the Change-ID: line from the commit message that Gerrit uses to track a new patch set for an existing change. You should have used `git commit --amend` to commit and use the existing commit message. See the "Amending a change" section in http://wiki.wireshark.org/Development/SubmittingPatches To recover this, we can either consider the latest change as the "primary" change and abandon the older one (which would effectively throw away Bill's fine comments) or abandon the new change and resubmit your changes as a new patch set to the older change.
If it's possible to abandon the new change. What should I do? I think, I'll have to go back to the old change-id-version, and then apply my changes again with "git commit --amend". How do I get back to the old version? ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: Adding a new dissector - beginners guide, (continued)
- Re: Adding a new dissector - beginners guide Thomas Wiens (Aug 21)
- Re: Adding a new dissector - beginners guide Martin Kaiser (Aug 21)
- Re: Adding a new dissector - beginners guide Jeff Morriss (Aug 21)
- Re: Adding a new dissector - beginners guide Christopher Maynard (Aug 21)
- Re: Adding a new dissector - beginners guide Graham Bloice (Aug 21)
- Re: Adding a new dissector - beginners guide Thomas Wiens (Aug 22)
- Re: Adding a new dissector - beginners guide Graham Bloice (Aug 22)
- Re: Adding a new dissector - beginners guide Bill Meier (Aug 22)
- Re: Adding a new dissector - beginners guide Thomas Wiens (Aug 22)
- Re: Adding a new dissector - beginners guide Graham Bloice (Aug 22)
- Re: Adding a new dissector - beginners guide Thomas Wiens (Aug 22)
- Re: Adding a new dissector - beginners guide Bill Meier (Aug 22)
- Re: Adding a new dissector - beginners guide Thomas Wiens (Aug 23)
- Re: Adding a new dissector - beginners guide mmann78 (Aug 23)
- Re: Adding a new dissector - beginners guide Jeff Morriss (Aug 22)