Re: Indicating dependencies between Gerrit changes

[Peter Wu <peter () lekensteyn nl>]

On Sat, Mar 18, 2017 at 07:40:00PM -0700, Guy Harris wrote:
> > On Mar 18, 2017, at 7:26 PM, Michael Mann (Code Review) <code-review-do-not-reply () wireshark org> wrote:
> >
> > Michael Mann has posted comments on this change.
> >
> > Change subject: TLS13: add certificate_authorities (47) dissection for draft -19
> > ......................................................................
> >
> >
> > Patch Set 3:
> >
> > > The build is failing on all platforms; I'm not sure why the Petri
> > > dish build didn't fail.
> >
> > Because there's a long chain of TLS patches, and I thought I had the correct order.  Committed the patch with 
> > tls_dissect_certificate_authorities.
>
> It looks as if there's a way in Gerrit to say "this change depends on this other change":
>
>       https://www.mediawiki.org/wiki/Gerrit/Advanced_usage#Create_a_dependency

That was already done in this case, but since there were no merge
conflicts and Gerrit is not configured to follow the patch order, it was
possible that the patch got cherry-picked without merging the base
patches first.

If that is not configurable for each patch, separately, maybe I should
post a comment next time?
-- 
Kind regards,
Peter Wu
https://lekensteyn.nl
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe