Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16

[Markus Armbruster <armbru@redhat.com>]

Peter Maydell <peter.maydell@linaro.org> writes:

> On 24 January 2017 at 09:53, Markus Armbruster <armbru@redhat.com> wrote:
>> Peter Maydell <peter.maydell@linaro.org> writes:
>>
>>> On 17 January 2017 at 16:43, Peter Maydell <peter.maydell@linaro.org> wrote:
>>>> In any case, applied this pullreq to master.
>>>
>>> Although it works fine on my OSX box (with warnings) it seems
>>> to have broken the Travis OSX builds:
>>> https://travis-ci.org/qemu/qemu/jobs/192750574
>>>
>>>   GEN     qemu-options.texi
>>>   GEN     qemu-monitor.texi
>>>   GEN     qemu-img-cmds.texi
>>>   GEN     qemu-monitor-info.texi
>>>   GEN     qemu-doc.txt
>>> qemu-doc.texi:7: warning: unrecognized encoding name `UTF-8'.
>>>   GEN     qemu-img.1
>>> /Users/travis/build/qemu/qemu//qemu-options.texi:60: warning: unlikely
>>> character [ in @var.
>>> /Users/travis/build/qemu/qemu//qemu-options.texi:60: warning: unlikely
>>> character ] in @var.
>>> /Users/travis/build/qemu/qemu//qemu-options.texi:61: warning: unlikely
>>> character [ in @var.
>>> /Users/travis/build/qemu/qemu//qemu-options.texi:61: warning: unlikely
>>> character ] in @var.
>>> /Users/travis/build/qemu/qemu//qemu-options.texi:630: No matching `@end table'.
>>> /Users/travis/build/qemu/qemu//qemu-doc.texi:1419: Cross reference to
>>> nonexistent node `bt-hcis' (perhaps incorrect sectioning?).
>>> makeinfo: Removing output file `qemu-doc.txt' due to errors; use
>>> --force to preserve.
>>> make: *** [qemu-doc.txt] Error 1
>>
>> Are you *sure* this series causes the failure?
>
> It's pretty solidly looking like it in travis: everything
> passes up until this merge is applied, and then everything
> fails...
>
>> I generated qemu-options.texi for every merge commit since v2.8.0, and
>> the only one that changes it is commit 598cf1c "Merge remote-tracking
>> branch 'remotes/bonzini/tags/for-upstream' into staging".
>
> Travis was already failing with this error before 598cf1c.
> First build with this error was on commit 23eb9e6; the build on
> the merge before that was fine.
> https://travis-ci.org/qemu/qemu/jobs/192750574
> is the build log for the first failure.
> https://travis-ci.org/qemu/qemu/jobs/192689212
> is the build log for the previous success.

Yeah, that's conclusive.

> I do note one difference: the successful build does not appear
> to try to build qemu-doc.txt. Perhaps your merge enabled
> an attempt to build some flavour of the docs that has
> always been broken ?

Since .texi doesn't change, the way we process it is the most likely
suspect.  In my other message, I list three commits that affect
processing.  I need help with testing them on OS X.