No, it's only necessary if you're changing a submodule, or have changes that are in some other way dependent on each other. Most changes don't need a topic.
On Thu, 9 Jul 2020 at 17:50, Varun Wadekar via Hafnium < hafnium@lists.trustedfirmware.org> wrote:
Hello,
I think this is a good idea to keep multiple moving parts in sync. OTOH, does this also mean that all changes *must* be part of a topic branch? That would be an overkill IMO.
-Varun
-----Original Message----- From: Hafnium hafnium-bounces@lists.trustedfirmware.org On Behalf Of Olivier Deprez via Hafnium Sent: Thursday, July 9, 2020 6:16 AM To: Benjamin Copeland ben.copeland@linaro.org; Joanna Farley < Joanna.Farley@arm.com>; bill.fletcher@linaro.org Cc: hafnium@lists.trustedfirmware.org Subject: Re: [Hafnium] change.submitWholeTopic option for Gerrit
External email: Use caution opening links or attachments
Ben,
Kindly please hold on, I guess I should forward this comm. to other MLs for which the corresponding tforg projects will inherit the newly enabled feature.
Thanks & Regards, Olivier.
From: Benjamin Copeland ben.copeland@linaro.org Sent: 09 July 2020 15:12 To: Joanna Farley; bill.fletcher@linaro.org Cc: Olivier Deprez; Andrew Walbran; hafnium@lists.trustedfirmware.org Subject: Re: [Hafnium] change.submitWholeTopic option for Gerrit
If we are all in agreement I will get it enabled.
Regards
Ben
On Thu, 9 Jul 2020 at 13:51, Joanna Farley <Joanna.Farley@arm.commailto: Joanna.Farley@arm.com> wrote: Agree __
On 09/07/2020, 13:48, "Hafnium on behalf of Olivier Deprez via Hafnium" < hafnium-bounces@lists.trustedfirmware.orgmailto: hafnium-bounces@lists.trustedfirmware.org on behalf of hafnium@lists.trustedfirmware.orgmailto:hafnium@lists.trustedfirmware.org> wrote:
Hi Andrew, We seem to reach a consensus within ARM that it is something we can
enable. As a slight usage detail, we'd expect developers to use their initials for gerrit topics such a xy/topic-name. That's already recommended for TF-A, although not always enforced. This would avoid unintended/accidental merges in different repos because of gerrit topic name clashes.
Does it make sense? Other folks in the ML, please shout if you disagree (you can also
express that you agree 🙂).
Regards, Olivier. ________________________________ From: Andrew Walbran Sent: Wednesday, July 01, 2020 11:28 To: hafnium@lists.trustedfirmware.org<mailto:
hafnium@lists.trustedfirmware.org>; Benjamin Copeland; Olivier Deprez Subject: Re: change.submitWholeTopic option for Gerrit
Any thoughts on this? On Fri, 26 Jun 2020 at 21:21, Andrew Walbran <qwandor@google.com
mailto:qwandor@google.com<mailto:qwandor@google.commailto: qwandor@google.com>> wrote: Hello, How would people feel about enabling the change.submitWholeTopic option ( https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#chan...) for Gerrit?
So far, we have relied on this to ensure that submodule changes get
submitted along with the corresponding change to the main repository. Our usual workflow has been that whenever a change is made to one of the submodule repositories, both that change and the corresponding change to the main repository are tagged with the same topic. That way it is only possible to submit either once they have all been reviewed +2 and the main change has passed presubmit. This avoids submodules getting out of sync or changes to them being missed or not properly tested.
However, it looks like this is a per-host configuration option rather
than per-repository, so it would also affect the other Trusted Firmware projects using the same Gerrit host. Are there any other uses of topics there which would conflict with this config change?
(If there are other people who might have an opinion on this please
add them to the thread.)
Hafnium mailing list Hafnium@lists.trustedfirmware.org<mailto:
Hafnium@lists.trustedfirmware.org> https://lists.trustedfirmware.org/mailman/listinfo/hafnium
-- Hafnium mailing list Hafnium@lists.trustedfirmware.org https://lists.trustedfirmware.org/mailman/listinfo/hafnium -- Hafnium mailing list Hafnium@lists.trustedfirmware.org https://lists.trustedfirmware.org/mailman/listinfo/hafnium