[Development] Using Git notes to reflect actual cherry-picks
Tor Arne Vestbø
Tor.arne.Vestbo at qt.io
Sat Feb 12 16:15:36 CET 2022
Very cool! As ossi suggests, this could potentially be handled by Gerrit, similar to this:
If we enable something like this we should namespace the notes, e.g. refs/notes/cherry-picks so that fetching and showing them in git log output is optional.
sOn 12 Feb 2022, at 08:04, Thiago Macieira <thiago.macieira at intel.com<mailto:thiago.macieira at intel.com>> wrote:
(I don't think qtbase is the right place for this)
I'd like to propose we use Git notes to include information into commits about
successful cherry-picks. I have the scripts done, see above. All we need is
that they are run weekly or the results pushed into the repository.
The Pick-To footer is very useful so we don't have to remember to actually
cherry-pick a commit when it's done integrating: the bot will take care of
doing that for us and let us know if it fails. However, it's not reliable:
* a change may be abandoned in Gerrit after the bot started with it (too
difficult to continue, author didn't realise it didn't apply, etc.)
* a change may be cherry-picked without the footer, either in Gerrit or
directly using git cherry-pick -x
For that reason, I'd like to ask that we add the use of notes, to indicate
cherry-picks that have succeeded. Moreover, it will tell the user what commit
exists in that particular branch. For example, after running the script, I see
for a commit of mine:
Pick-to: 6.2.3 6.2 6.3
Reviewed-by: Lars Knoll <lars.knoll at qt.io<mailto:lars.knoll at qt.io>>
That tells us that the cherry-picks to 6.2 and 6.2.3 aren't present (I
abandoned after conflicts happened).
Pick-to: 6.3 6.2 6.2.3 5.15
Reviewed-by: Mårten Nordheim <marten.nordheim at qt.io<mailto:marten.nordheim at qt.io>>
This one looks wrong: 6.2.3 is present but not 6.2, So I searched Gerrit and
indeed, the change simply failed in the CI 3 weeks ago and no one pushed the
The scripts are pretty fast, but I've also made them work incrementally. See
the attached run example.
Thiago Macieira - thiago.macieira (AT) intel.com<http://intel.com>
Software Architect - Intel DPG Cloud Engineering
Development mailing list
Development at qt-project.org<mailto:Development at qt-project.org>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Development