On Mon, Aug 29, 2016 at 10:55 AM, Poornima Gurusiddaiah <pgurusid@xxxxxxxxxx> wrote:
Hi,
Regarding the enforcement of the dependencies while merging, i see that
the dependent patches on any patch is mentioned in the "Related Changes"
column [1]. It still doesn't enforce, in the cherry-pick way of
submitting changes, by default it ignores the lineage [2]. But there are
ways to enforce this. Will let the gluster infra maintainers to comment
on the same.
[1] https://gerrit-review.googlesource.com/ Documentation/user-review-ui. html#related-changes
[2] https://gerrit-review.googlesource.com/ Documentation/project- configuration.html#project_ options
Regards,
Poornima
Thank you Poornima for pointing this out. You're right, it's worth changing our submission type to either "Rebase If Necessary" or "Merge If Necessary" to enforce this. I'm not sure what the implications are, so I'll report back after I've setup a test on staging so we can experiment and see what works.
--
nigelb
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel