mbox series

[0/2] *** This is based on pull request from Alex Bennée ***

Message ID 20210317191115.1445-1-luoyonggang@gmail.com (mailing list archive)
Headers show
Series *** This is based on pull request from Alex Bennée *** | expand

Message

Yonggang Luo March 17, 2021, 7:11 p.m. UTC
*** BLURB HERE ***
The pull request
https://patchew.org/QEMU/20210317072216.16316-1-alex.bennee@linaro.org/

Yonggang Luo (2):
  plugins: Update qemu-plugins.symbols to match qemu-plugins.h
  plugins: Move all typedef and type declaration to the front of the
    qemu-plugin.h

 include/qemu/qemu-plugin.h   | 187 +++++++++++++++++------------------
 plugins/qemu-plugins.symbols |  25 +++--
 2 files changed, 104 insertions(+), 108 deletions(-)

Comments

Markus Armbruster March 18, 2021, 8:50 a.m. UTC | #1
Please use the subject line for explaining the purpose of the series.
"Based on" information should go into the body.  Ideally also in
machine-readable form, like

    Based-on: <20171220185924.32756-1-armbru@redhat.com>

where the thing in <anglular brackets> is the pull request's Message-id.

Note: I used one of my pull requests, not Alex's, for this example.

Yonggang Luo <luoyonggang@gmail.com> writes:

> *** BLURB HERE ***

Recommend to delete this placeholder next time :)

> The pull request
> https://patchew.org/QEMU/20210317072216.16316-1-alex.bennee@linaro.org/
>
> Yonggang Luo (2):
>   plugins: Update qemu-plugins.symbols to match qemu-plugins.h
>   plugins: Move all typedef and type declaration to the front of the
>     qemu-plugin.h
>
>  include/qemu/qemu-plugin.h   | 187 +++++++++++++++++------------------
>  plugins/qemu-plugins.symbols |  25 +++--
>  2 files changed, 104 insertions(+), 108 deletions(-)
Yonggang Luo March 18, 2021, 9:40 a.m. UTC | #2
On Thu, Mar 18, 2021 at 8:50 AM Markus Armbruster <armbru@redhat.com> wrote:
>
> Please use the subject line for explaining the purpose of the series.
> "Based on" information should go into the body.  Ideally also in
> machine-readable form, like
>
>     Based-on: <20171220185924.32756-1-armbru@redhat.com>
>
> where the thing in <anglular brackets> is the pull request's Message-id.
>
> Note: I used one of my pull requests, not Alex's, for this example.
>
> Yonggang Luo <luoyonggang@gmail.com> writes:
>
> > *** BLURB HERE ***
>
> Recommend to delete this placeholder next time :)
Thanks, i'll do it next time
>
> > The pull request
> > https://patchew.org/QEMU/20210317072216.16316-1-alex.bennee@linaro.org/
> >
> > Yonggang Luo (2):
> >   plugins: Update qemu-plugins.symbols to match qemu-plugins.h
> >   plugins: Move all typedef and type declaration to the front of the
> >     qemu-plugin.h
> >
> >  include/qemu/qemu-plugin.h   | 187 +++++++++++++++++------------------
> >  plugins/qemu-plugins.symbols |  25 +++--
> >  2 files changed, 104 insertions(+), 108 deletions(-)
>


--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo