Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-14 Thread Peter Maydell
On Thu, 14 Feb 2019 at 18:46, Paolo Bonzini wrote: > > On 14/02/19 16:24, Peter Maydell wrote: > > It does check, but I forgot to make the makefiles pay attention > > to the check. > > > > That's very irritating that Fedora is using a weird filename for > > the tool -- what is their justification

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-14 Thread Paolo Bonzini
On 14/02/19 16:24, Peter Maydell wrote: > It does check, but I forgot to make the makefiles pay attention > to the check. > > That's very irritating that Fedora is using a weird filename for > the tool -- what is their justification for doing that? > I suppose we'll have to make configure cope :-(

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-14 Thread Peter Maydell
On Thu, 14 Feb 2019 at 14:56, Marc-André Lureau wrote: > > Hi > > On Thu, Feb 14, 2019 at 12:30 PM Peter Maydell > wrote: > > > > Ping! Thanks to Alex for doing low-level review of this patchset. > > Overall, looks good to me too. > > Minor nit: configure doesn't check presence of sphinx-build,

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-14 Thread Marc-André Lureau
Hi On Thu, Feb 14, 2019 at 12:30 PM Peter Maydell wrote: > > Ping! Thanks to Alex for doing low-level review of this patchset. Overall, looks good to me too. Minor nit: configure doesn't check presence of sphinx-build, and on my fc29, it's sphinx-build-3 :) CI files will probably need to be up

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-14 Thread Peter Maydell
Ping! Thanks to Alex for doing low-level review of this patchset. I'm particularly interested in high-level review: * is this the right way to be going? * if we committed this and then did a release with the docs as they are like this, would that be ok? * what's the most important next step a

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-03 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series failed the docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGI

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
] Enable build and install of our rST docs === TEST SCRIPT BEGIN === #!/bin/bash git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread Peter Maydell
On Fri, 1 Feb 2019 at 15:06, wrote: > > Patchew URL: > https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ > > > > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Subject: [Qemu-d

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
] Enable build and install of our rST docs === TEST SCRIPT BEGIN === #!/bin/bash git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
] Enable build and install of our rST docs === TEST SCRIPT BEGIN === #!/bin/bash git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
] Enable build and install of our rST docs === TEST SCRIPT BEGIN === #!/bin/bash git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
] Enable build and install of our rST docs === TEST SCRIPT BEGIN === #!/bin/bash git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Message-id

Re: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190201145035.22739-1-peter.mayd...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs Type: series Message-id

[Qemu-devel] [PATCH 00/11] Enable build and install of our rST docs

2019-02-01 Thread Peter Maydell
This patchset enables building and installing the various rST docs we have started to accumulate in our docs/ directory. It does this using Sphinx (which is the docs tooling that the Linux kernel uses). The series is not trying to take us in one giant leap to a brave new Sphinx-powered world -- it