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

2017-01-24 Thread Paolo Bonzini
On 24/01/2017 11:09, Markus Armbruster wrote: > * commit f8bab10 build-sys: add txt documentation rules This could be the culprit... Paolo

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

2017-01-24 Thread Markus Armbruster
Peter Maydell writes: > On 24 January 2017 at 09:53, Markus Armbruster wrote: >> Peter Maydell writes: >> >>> On 17 January 2017 at 16:43, Peter Maydell wrote: In any case, applied this pullreq to master. >>> >>> Although it works fine on my OSX box (with warnings) it seems >>> to have br

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

2017-01-24 Thread Markus Armbruster
Markus Armbruster writes: > Peter Maydell writes: > >> On 17 January 2017 at 16:43, Peter Maydell 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/qe

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

2017-01-24 Thread Peter Maydell
On 24 January 2017 at 09:53, Markus Armbruster wrote: > Peter Maydell writes: > >> On 17 January 2017 at 16:43, Peter Maydell 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: >> ht

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

2017-01-24 Thread Markus Armbruster
Peter Maydell writes: > On 17 January 2017 at 16:43, Peter Maydell 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-

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

2017-01-23 Thread Markus Armbruster
"Daniel P. Berrange" writes: > On Fri, Jan 20, 2017 at 04:46:05PM +0100, Markus Armbruster wrote: >> Peter Maydell writes: >> >> > On 17 January 2017 at 16:43, Peter Maydell >> > wrote: >> >> In any case, applied this pullreq to master. >> > >> > Although it works fine on my OSX box (with war

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

2017-01-23 Thread Daniel P. Berrange
On Fri, Jan 20, 2017 at 04:46:05PM +0100, Markus Armbruster wrote: > Peter Maydell writes: > > > On 17 January 2017 at 16:43, Peter Maydell 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

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

2017-01-23 Thread Alex Bennée
Peter Maydell writes: > On 20 January 2017 at 15:46, Markus Armbruster wrote: >> Peter Maydell writes: >> >>> On 17 January 2017 at 16:43, Peter Maydell wrote: In any case, applied this pullreq to master. >>> >>> Although it works fine on my OSX box (with warnings) it seems >>> to have b

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

2017-01-20 Thread Peter Maydell
On 20 January 2017 at 15:46, Markus Armbruster wrote: > Peter Maydell writes: > >> On 17 January 2017 at 16:43, Peter Maydell 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: >> ht

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

2017-01-20 Thread Markus Armbruster
Peter Maydell writes: > On 17 January 2017 at 16:43, Peter Maydell 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-

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

2017-01-20 Thread Peter Maydell
On 17 January 2017 at 16:43, Peter Maydell 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.t

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

2017-01-17 Thread Eric Blake
On 01/17/2017 06:08 AM, Markus Armbruster wrote: > > Three possible solutions, in increasing order of complexity: > > 1. Live with the warning from old versions. If a new version comes > around that does something with @subtitle, it'll just work. > > 2. Suppress the warning with @iftex-hammer.

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

2017-01-17 Thread Peter Maydell
On 17 January 2017 at 14:23, Markus Armbruster wrote: > Peter Maydell writes: >> As an aside, how useful is the PDF output anyway? In 2017 >> there seems to me quite a good argument for just creating >> HTML... > > We do generate HTML. We also generate PDF. It's better for printing > (if you're

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

2017-01-17 Thread Markus Armbruster
Peter Maydell writes: > On 17 January 2017 at 13:24, Markus Armbruster wrote: >> Peter Maydell writes: >>> (1) if it doesn't actually cause a change in the output, we >>> should either just delete the use of VERSION entirely, or move >>> it to somewhere outside of @subtitle which does actually

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

2017-01-17 Thread Marc-André Lureau
Hi - Original Message - > On 17 January 2017 at 13:24, Markus Armbruster wrote: > > Peter Maydell writes: > >> (1) if it doesn't actually cause a change in the output, we > >> should either just delete the use of VERSION entirely, or move > >> it to somewhere outside of @subtitle which d

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

2017-01-17 Thread Peter Maydell
On 17 January 2017 at 13:24, Markus Armbruster wrote: > Peter Maydell writes: >> (1) if it doesn't actually cause a change in the output, we >> should either just delete the use of VERSION entirely, or move >> it to somewhere outside of @subtitle which does actually >> appear somewhere. There's n

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

2017-01-17 Thread Markus Armbruster
Peter Maydell writes: > On 17 January 2017 at 12:08, Markus Armbruster wrote: >> Right now, that'll make no difference whatsoever, because the programs >> that choke on -D generate no output for the commands using the variable >> defined with -D. All they do is gripe. >> >> Three possible solut

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

2017-01-17 Thread Peter Maydell
On 17 January 2017 at 12:08, Markus Armbruster wrote: > Right now, that'll make no difference whatsoever, because the programs > that choke on -D generate no output for the commands using the variable > defined with -D. All they do is gripe. > > Three possible solutions, in increasing order of co

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

2017-01-17 Thread Markus Armbruster
Peter Maydell writes: > On 17 January 2017 at 08:31, Markus Armbruster wrote: >> Peter Maydell writes: >>> 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; >>> Fedo

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

2017-01-17 Thread Peter Maydell
On 17 January 2017 at 08:31, Markus Armbruster wrote: > Peter Maydell writes: >> 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)

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

2017-01-17 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Tue, Jan 17, 2017 at 12:34 PM Markus Armbruster > wrote: > >> Peter Maydell writes: >> >> > On 16 January 2017 at 09:33, Markus Armbruster >> wrote: >> >> This is Marc-André's "[PATCH v8 00/21] qapi doc generation (whole >> >> version, squashed)" with a fe

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

2017-01-17 Thread Marc-André Lureau
Hi On Tue, Jan 17, 2017 at 12:34 PM Markus Armbruster wrote: > Peter Maydell writes: > > > On 16 January 2017 at 09:33, Markus Armbruster > 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

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

2017-01-17 Thread Markus Armbruster
Peter Maydell writes: > On 16 January 2017 at 09:33, Markus Armbruster 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. >> >>

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

2017-01-16 Thread Peter Maydell
On 16 January 2017 at 09:33, Markus Armbruster 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

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

2017-01-16 Thread Markus Armbruster
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 unsquash