All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luke Nowakowski-Krijger <lnowakow@eng.ucsd.edu>
To: Jonathan Corbet <corbet@lwn.net>
Cc: linux-kernel-mentees@lists.linuxfoundation.org,
	pbonzini@redhat.com, rkrcmar@redhat.com, kvm@vger.kernel.org,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] Documentation: virtual: Add toctree hooks
Date: Mon, 8 Jul 2019 13:28:08 -0700	[thread overview]
Message-ID: <20190708202808.GA8859@luke-XPS-13> (raw)
In-Reply-To: <20190708135404.3eeed68f@lwn.net>

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@eng.ucsd.edu> 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

WARNING: multiple messages have this Message-ID (diff)
From: lnowakow at eng.ucsd.edu (Luke Nowakowski-Krijger)
Subject: [Linux-kernel-mentees] [PATCH 1/3] Documentation: virtual: Add toctree hooks
Date: Mon, 8 Jul 2019 13:28:08 -0700	[thread overview]
Message-ID: <20190708202808.GA8859@luke-XPS-13> (raw)
In-Reply-To: <20190708135404.3eeed68f@lwn.net>

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 at eng.ucsd.edu> 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

WARNING: multiple messages have this Message-ID (diff)
From: lnowakow@eng.ucsd.edu (Luke Nowakowski-Krijger)
Subject: [Linux-kernel-mentees] [PATCH 1/3] Documentation: virtual: Add toctree hooks
Date: Mon, 8 Jul 2019 13:28:08 -0700	[thread overview]
Message-ID: <20190708202808.GA8859@luke-XPS-13> (raw)
Message-ID: <20190708202808.HAA30BSdXb3pDto5xncqWENTVs1q_jx_yNYHrBL_adU@z> (raw)
In-Reply-To: <20190708135404.3eeed68f@lwn.net>

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 at eng.ucsd.edu> 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

  reply	other threads:[~2019-07-08 20:28 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-06 21:38 [PATCH 0/3] [PATCH 0/3] Documentation: virtual: convert files from .txt to Luke Nowakowski-Krijger
2019-07-06 21:38 ` [Linux-kernel-mentees] " Luke Nowakowski-Krijger
2019-07-06 21:38 ` lnowakow
2019-07-06 21:38 ` [PATCH 1/3] Documentation: virtual: Add toctree hooks Luke Nowakowski-Krijger
2019-07-06 21:38   ` [Linux-kernel-mentees] " Luke Nowakowski-Krijger
2019-07-06 21:38   ` lnowakow
2019-07-08 19:54   ` Jonathan Corbet
2019-07-08 19:54     ` [Linux-kernel-mentees] " Jonathan Corbet
2019-07-08 19:54     ` corbet
2019-07-08 20:28     ` Luke Nowakowski-Krijger [this message]
2019-07-08 20:28       ` Luke Nowakowski-Krijger
2019-07-08 20:28       ` lnowakow
2019-07-06 21:38 ` [PATCH 2/3] Documentation: kvm: Convert cpuid.txt to .rst Luke Nowakowski-Krijger
2019-07-06 21:38   ` [Linux-kernel-mentees] " Luke Nowakowski-Krijger
2019-07-06 21:38   ` lnowakow
2019-07-08 20:00   ` Jonathan Corbet
2019-07-08 20:00     ` [Linux-kernel-mentees] " Jonathan Corbet
2019-07-08 20:00     ` corbet
2019-07-08 20:15     ` Luke Nowakowski-Krijger
2019-07-08 20:15       ` [Linux-kernel-mentees] " Luke Nowakowski-Krijger
2019-07-08 20:15       ` lnowakow
2019-07-08 20:20       ` Jonathan Corbet
2019-07-08 20:20         ` [Linux-kernel-mentees] " Jonathan Corbet
2019-07-08 20:20         ` corbet
2019-07-26 22:19         ` Paolo Bonzini
2019-07-26 22:19           ` [Linux-kernel-mentees] " Paolo Bonzini
2019-07-26 22:19           ` pbonzini
2019-07-06 21:38 ` [PATCH 3/3] Documentation: virtual: Convert paravirt_ops.txt " Luke Nowakowski-Krijger
2019-07-06 21:38   ` [Linux-kernel-mentees] " Luke Nowakowski-Krijger
2019-07-06 21:38   ` lnowakow

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190708202808.GA8859@luke-XPS-13 \
    --to=lnowakow@eng.ucsd.edu \
    --cc=corbet@lwn.net \
    --cc=kvm@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel-mentees@lists.linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=rkrcmar@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.