[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16 |
Date: |
Tue, 24 Jan 2017 12:08:40 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On 24 January 2017 at 09:53, Markus Armbruster <address@hidden> wrote:
>> Peter Maydell <address@hidden> writes:
>>
>>> On 17 January 2017 at 16:43, Peter Maydell <address@hidden> 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
>>> address@hidden 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.
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, (continued)
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Markus Armbruster, 2017/01/17
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Peter Maydell, 2017/01/17
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Peter Maydell, 2017/01/20
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Markus Armbruster, 2017/01/20
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Peter Maydell, 2017/01/20
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Alex Bennée, 2017/01/23
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Daniel P. Berrange, 2017/01/23
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Markus Armbruster, 2017/01/23
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Markus Armbruster, 2017/01/24
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Peter Maydell, 2017/01/24
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16,
Markus Armbruster <=
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Markus Armbruster, 2017/01/24
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Paolo Bonzini, 2017/01/24
- Re: [Qemu-devel] [PULL v2 000/180] QAPI patches for 2017-01-16, Eric Blake, 2017/01/17