All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Markus Armbruster <armbru@redhat.com>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16
Date: Mon, 16 Jan 2017 15:58:41 +0000	[thread overview]
Message-ID: <CAFEAcA_BBbHeQhWYf_qrL+TwxGT8w34SvUNG_gcNfpNnyaGCzg@mail.gmail.com> (raw)
In-Reply-To: <1484559200-2301-1-git-send-email-armbru@redhat.com>

On 16 January 2017 at 09:33, Markus Armbruster <armbru@redhat.com> wrote:
> This is Marc-André's "[PATCH v8 00/21] qapi doc generation (whole
> version, squashed)" with a few commit messages tweaked, and "[PATCH v8
> 14/21] (SQUASHED) move doc to schema" unsquashed into 161 patches.
>
> We did all the respins with in this squashed form to reduce noise.
> However, since the unsquashed form is better suited for review, and
> probably nicer if we have to revisit this part of the work down the
> road, I'm proposing to merge this unsquashed.
>
> If you want me to post the unsquashed patches, I'm happy to redo this
> pull request.
>
> If you'd rather pull the squashed version, likewise.
>
> I'm afraid this is a bit of a doc conflict magnet.  The sooner we can
> get it in, the easier for Marc-André and me.
>
> v2:
> * Rebased (v1 conflicts with commit e1ff3c6)
> * test-qapi.py tweaked to avoid trailing empty lines in .out
>
> The following changes since commit b6af8ea60282df514f87d32e36afd1c9aeee28c8:
>
>   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-and-machine-pull-request' into staging (2017-01-13 14:38:21 +0000)
>
> are available in the git repository at:
>
>   git://repo.or.cz/qemu/armbru.git tags/pull-qapi-2017-01-16
>
> for you to fetch changes up to 56e8bdd46a8a42d89b0afea9da83ae7679cc0439:
>
>   build-sys: add qapi doc generation targets (2017-01-16 10:11:43 +0100)
>
> ----------------------------------------------------------------
> QAPI patches for 2017-01-16

This seems to trigger some warnings from older makeinfo versions:

On the Fedora 20 ppc64be box and Ubuntu 14.04.5 LTS aarch64 box
(Ubuntu: makeinfo (GNU texinfo) 5.2;
 Fedora: makeinfo (GNU texinfo) 5.1)

/home/pm215/qemu/docs/qemu-ga-ref.texi:41: warning: undefined flag: VERSION
/home/pm215/qemu/docs/qemu-ga-ref.texi:41: warning: undefined flag: VERSION
/home/pm215/qemu/docs/qemu-qmp-ref.texi:41: warning: undefined flag: VERSION
/home/pm215/qemu/docs/qemu-qmp-ref.texi:41: warning: undefined flag: VERSION

On OSX (makeinfo (GNU texinfo) 4.8):
./qemu-options.texi:60: warning: unlikely character [ in @var.
./qemu-options.texi:60: warning: unlikely character ] in @var.
./qemu-options.texi:61: warning: unlikely character [ in @var.
./qemu-options.texi:61: warning: unlikely character ] in @var.

(OSX also has the long standing warning
/Users/pm215/src/qemu-for-merges/qemu-doc.texi:7: warning:
unrecognized encoding name `UTF-8'.
but you can ignore that since it's been that way for years
and is because osx's makeinfo is ancient.)

thanks
-- PMM

  reply	other threads:[~2017-01-16 15:59 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-16  9:33 [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16 Markus Armbruster
2017-01-16 15:58 ` Peter Maydell [this message]
2017-01-17  8:31   ` Markus Armbruster
2017-01-17  9:54     ` Marc-André Lureau
2017-01-17 10:13       ` Markus Armbruster
2017-01-17 11:08     ` Peter Maydell
2017-01-17 12:08       ` Markus Armbruster
2017-01-17 12:24         ` Peter Maydell
2017-01-17 13:24           ` Markus Armbruster
2017-01-17 13:59             ` Peter Maydell
2017-01-17 14:15               ` Marc-André Lureau
2017-01-17 14:23               ` Markus Armbruster
2017-01-17 16:43                 ` Peter Maydell
2017-01-20 14:39                   ` Peter Maydell
2017-01-20 15:46                     ` Markus Armbruster
2017-01-20 15:48                       ` Peter Maydell
2017-01-23 12:48                         ` Alex Bennée
2017-01-23 13:59                       ` Daniel P. Berrange
2017-01-23 14:49                         ` Markus Armbruster
2017-01-24  9:53                     ` Markus Armbruster
2017-01-24 10:03                       ` Peter Maydell
2017-01-24 11:08                         ` Markus Armbruster
2017-01-24 10:09                       ` Markus Armbruster
2017-01-24 10:49                         ` Paolo Bonzini
2017-01-17 17:05         ` Eric Blake

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFEAcA_BBbHeQhWYf_qrL+TwxGT8w34SvUNG_gcNfpNnyaGCzg@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=armbru@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.