LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Randy Dunlap <randy.dunlap@oracle.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: lkml <linux-kernel@vger.kernel.org>, Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [PATCH] kernel-doc: set verbose mode via environment
Date: Thu, 28 Feb 2008 08:30:09 -0800	[thread overview]
Message-ID: <20080228083009.5955a7eb.randy.dunlap@oracle.com> (raw)
In-Reply-To: <20080227191104.6f166972.akpm@linux-foundation.org>

On Wed, 27 Feb 2008 19:11:04 -0800 Andrew Morton wrote:

> On Sun, 24 Feb 2008 17:53:58 -0800 Randy Dunlap <randy.dunlap@oracle.com> wrote:
> 
> > From: Randy Dunlap <randy.dunlap@oracle.com>
> > 
> > Allow setting environment variable "KERNEL_DOC_VERBOSE=1" to enable
> > verbose mode in scripts/kernel-doc.  Useful for getting more
> > info and warnings from kernel-doc.
> > 
> > Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
> > ---
> >  scripts/kernel-doc |    4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > --- linux-2.6.25-rc3.orig/scripts/kernel-doc
> > +++ linux-2.6.25-rc3/scripts/kernel-doc
> > @@ -247,6 +247,10 @@ my ($function, %function_table,%paramete
> >  my ($type,$declaration_name,$return_type);
> >  my ($newsection,$newcontents,$prototype,$filelist, $brcount, %source_map);
> >  
> > +if (defined($ENV{'KERNEL_DOC_VERBOSE'})) {
> > +	$verbose = "$ENV{'KERNEL_DOC_VERBOSE'}";
> > +}
> > +
> >  # Generated docbook code is inserted in a template at a point where
> >  # docbook v3.1 requires a non-zero sequence of RefEntry's; see:
> >  # http://www.oasis-open.org/docbook/documentation/reference/html/refentry.html
> 
> Shouldn't this be, err, documented somewhere?

I did think about that.  I was still wondering where to add it.

> I'd have expected to obtain this functionality by running `make V=1 foodocs'?

Sam, any thoughts on Makefile passing "-v" to scripts/kernel-doc ?

---
~Randy

  reply	other threads:[~2008-02-28 16:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-25  1:53 Randy Dunlap
2008-02-28  3:11 ` Andrew Morton
2008-02-28 16:30   ` Randy Dunlap [this message]
2008-02-28 18:32     ` Sam Ravnborg
2008-02-29  0:09       ` [PATCH v2] " Randy Dunlap
2008-03-02 21:15         ` Christian Kujau

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=20080228083009.5955a7eb.randy.dunlap@oracle.com \
    --to=randy.dunlap@oracle.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.org \
    --subject='Re: [PATCH] kernel-doc: set verbose mode via environment' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).