Re: [PATCH v2 1/4] gdb/aarch64: record/replay support for LRCPC3

[Thiago Jung Bauermann <thiago.bauermann@linaro.org>]

Ezra Sitorus <ezra.sitorus@arm.com> writes:

>> Hi Ezra! I have one tiny comment with regards to the test (that can
>> truthfully probably be ignored), but you shouldn't need to send a v3 just to
>> fix that. Feel free to add my review tag
>> 
>> Reviewed-By: Guinevere Larsen <guinevere@redhat.com
>
> Hi! Thanks all for the reviews - I'll address those comments. One question I
> have for GDB workflow: do I explicitly need an Approved-By before I can push?

Ideally a maintainer will explicitly provide an Approved-By to signal
that the patch can be pushed. A single tag is enough even if there are
other maintainers/reviewers involved in the discussion, unless the tag
is explicitly limited to the tag giver's area of the project.

Sometimes instead of a tag an approval is given in a hopefully
unambiguous sentence in English to the same effect.

The intended meaning of the tags is described in the gdb/MAINTAINERS
file.

> Or in this case am I waiting on a final approval by Luis or Thiago?

For this series, patches 2, 3 and 4 can be committed.

For patch 1, a v3 is needed.

-- 
Thiago
(he/him)