[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/13] x86, machine, numa queue (2017-01-23)
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 00/13] x86, machine, numa queue (2017-01-23) |
Date: |
Tue, 24 Jan 2017 17:20:05 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On 24 January 2017 at 13:24, Peter Maydell <address@hidden> wrote:
>> I get a failure in the docs on OSX:
>> GEN qemu-doc.txt
>> CC qapi/qapi-visit-core.o
>> /Users/pm215/src/qemu-for-merges/qemu-doc.texi:7: warning:
>> unrecognized encoding name `UTF-8'.
>> GEN trace/generated-tracers.c
>> /Users/pm215/src/qemu-for-merges/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.
>>
>> This is the same error that was happening on the Travis OSX
>> builds but not on my personal OSX machine, and leaves me
>> suspicious that the qemu-doc.txt stuff is broken but also
>> not always being rebuilt correctly (since it's a bit
>> hit-and-miss whether particular things hit this.)
>
> The cause of this problem I think is that we added a new
> output file qemu-doc.txt, but we forgot to add it to this
> dependency rule in Makefile:
>
> qemu-doc.html qemu-doc.info qemu-doc.pdf: \
> qemu-img.texi qemu-nbd.texi qemu-options.texi qemu-option-trace.texi \
> qemu-monitor.texi qemu-img-cmds.texi qemu-ga.texi \
> qemu-monitor-info.texi
>
> So if you do a make with -j8 then make may decide to parallelize
> the building of the qemu-doc.txt with the creation of qemu-options.texi,
> and then makeinfo can read a half-completed qemu-options.texi.
Good catch! Now, why didn't I see that myself? %-}
> qemu-doc.txt also seems to have been not added to the rm -f
> commands for the distclean target.
>
> Side question: why do we have a rule to create the qemu-doc.pdf
> but not actually install it with install-doc ?
I don't know. Has been that way since the pdf target was added in
commit 20cc999.
- [Qemu-devel] [PULL 06/13] pc: cleanup: move smbios_set_cpuid() into pc_build_smbios(), (continued)
- [Qemu-devel] [PULL 06/13] pc: cleanup: move smbios_set_cpuid() into pc_build_smbios(), Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 07/13] pc: don't return cpu pointer from pc_new_cpu() as it's not needed anymore, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 09/13] MAINTAINERS: Add an entry for hw/core/null-machine.c, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 08/13] machine: Make possible_cpu_arch_ids() return const pointer, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 10/13] qemu-options: Rename variables on the -numa "cpus" option, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 11/13] hw/core/null-machine: Add the possibility to instantiate a CPU and RAM, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 12/13] kvm: Simplify invtsc check, Eduardo Habkost, 2017/01/23
- [Qemu-devel] [PULL 13/13] kvm: Allow invtsc migration if tsc-khz is set explicitly, Eduardo Habkost, 2017/01/23
- Re: [Qemu-devel] [PULL 00/13] x86, machine, numa queue (2017-01-23), Peter Maydell, 2017/01/24
- Re: [Qemu-devel] [PULL 00/13] x86, machine, numa queue (2017-01-23), Peter Maydell, 2017/01/24