All of lore.kernel.org
 help / color / mirror / Atom feed
From: Randy Dunlap <randy.dunlap@oracle.com>
To: Amerigo Wang <xiyou.wangcong@gmail.com>
Cc: Jiri Slaby <jirislaby@gmail.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	samr <sam@ravnborg.org>
Subject: Re: kbuild docs regression
Date: Thu, 18 Jun 2009 08:54:16 -0700	[thread overview]
Message-ID: <4A3A6328.90408@oracle.com> (raw)
In-Reply-To: <20090618094604.GC6133@cr0.nay.redhat.com>

Amerigo Wang wrote:
> On Tue, Jun 16, 2009 at 03:14:14PM +0200, Jiri Slaby wrote:
>> On 06/15/2009 06:31 PM, Randy Dunlap wrote:
>>> make htmldocs
>>> without using "O=dir" now (as of 2.6.30-git8) fails with
>>>
>>>   DOCPROC Documentation/DocBook/deviceiobook.xml
>>>   DOCPROC Documentation/DocBook/device-drivers.xml
>>> exec /scripts/kernel-doc: No such file or directory
>>> exec /scripts/kernel-doc: No such file or directory
>>> make[1]: *** [Documentation/DocBook/deviceiobook.xml] Error 1
>>>
>>> (+ more)
>>>
>>> Using V=1 shows that SRCTREE is set but KBUILD_SRC is not set:
>> Hmm, thanks, actually it's set, but it's empty :(. Is it expected, or is
>> it a build-sys bug?
> 
> No, it is a bug of docproc. Patch below fixes it.
> 
> Signed-off-by: WANG Cong <xiyou.wangcong@gmail.com>

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>


> ---
> diff --git a/scripts/basic/docproc.c b/scripts/basic/docproc.c
> index 4c9523e..99ca7a6 100644
> --- a/scripts/basic/docproc.c
> +++ b/scripts/basic/docproc.c
> @@ -385,7 +385,7 @@ int main(int argc, char *argv[])
>  	if (!srctree)
>  		srctree = getcwd(NULL, 0);
>  	kernsrctree = getenv("KBUILD_SRC");
> -	if (!kernsrctree)
> +	if (!kernsrctree || !*kernsrctree)
>  		kernsrctree = srctree;
>  	if (argc != 3) {
>  		usage();


-- 
~Randy
LPC 2009, Sept. 23-25, Portland, Oregon
http://linuxplumbersconf.org/2009/

      reply	other threads:[~2009-06-18 15:57 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-15 16:31 kbuild docs regression Randy Dunlap
2009-06-16 13:14 ` Jiri Slaby
2009-06-18  9:46   ` Amerigo Wang
2009-06-18 15:54     ` Randy Dunlap [this message]

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=4A3A6328.90408@oracle.com \
    --to=randy.dunlap@oracle.com \
    --cc=jirislaby@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.org \
    --cc=xiyou.wangcong@gmail.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.