On Mon, Jul 08, 2019 at 01:54:04PM -0600, Jonathan Corbet wrote: > On Sat, 6 Jul 2019 14:38:13 -0700 > Luke Nowakowski-Krijger <lnowakow@xxxxxxxxxxxx> wrote: > > At this point in the patch series, the above-mentioned RST files don't > exist. So if somebody tries to build the docs here, the build will fail. > I suspect that it's pretty rare for people to use bisection with docs > builds, but it's still proper practice to ensure that things work at every > step in your series. So the above entries should be added in the patches > that convert the files. > Yes, agreed. It should be at the end of the series. > Also, vcpu-requests.txt is never touched in this patch series, which > suggests that you didn't build the docs even at the end of it. > In the staging tree I was doing work out of I saw a vcpu-requests.rst file so thought I could add it "while I'm here". You're right. I should only add the files I am converting. > Thanks, > > jon Thanks again, - Luke