mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Robert P. J. Day" <rpjday@mindspring.com>
To: Rusty Russell <rusty@rustcorp.com.au>
Cc: Rene Herman <rene.herman@gmail.com>,
	Alan Cox <alan@lxorguk.ukuu.org.uk>, Adrian Bunk <bunk@stusta.de>,
	Marcel Holtmann <marcel@holtmann.org>,
	Christoph Hellwig <hch@infradead.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: MODULE_MAINTAINER
Date: Mon, 23 Apr 2007 07:52:09 -0400 (EDT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0704230748270.25463@localhost.localdomain> (raw)
In-Reply-To: <1177327476.17026.75.camel@localhost.localdomain>

On Mon, 23 Apr 2007, Rusty Russell wrote:

> On Mon, 2007-04-23 at 11:33 +0200, Rene Herman wrote:
> > On 04/04/2007 06:38 PM, Rene Herman wrote:
> >
> > Rusty?
>
> Valid points have been made on both sides.  I suggest:
>
> #define MODULE_MAINTAINER(_maintainer) \
> 	MODULE_AUTHOR("(Maintained by) "_maintainer)

why bring MODULE_AUTHOR into it?  just define it in terms of
MODULE_INFO:

#define MODULE_MAINTAINER(_m) MODULE_INFO(_m, "(Maintained by)"  \
	maintainer)

technically, the maintainer is not the same as the author so why
confuse the issue with an extra unnecessary macro expansion?

rday
-- 
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================

  reply	other threads:[~2007-04-23 11:53 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-04 11:26 MODULE_MAINTAINER Rene Herman
2007-04-04 11:29 ` MODULE_MAINTAINER Rene Herman
2007-04-04 12:33 ` MODULE_MAINTAINER Christoph Hellwig
2007-04-04 13:02   ` MODULE_MAINTAINER Rene Herman
2007-04-04 14:57     ` MODULE_MAINTAINER Adrian Bunk
2007-04-04 16:33       ` MODULE_MAINTAINER Stefan Richter
2007-04-04 16:38         ` MODULE_MAINTAINER Adrian Bunk
2007-04-04 16:45           ` MODULE_MAINTAINER Stefan Richter
2007-04-04 14:48   ` MODULE_MAINTAINER Marcel Holtmann
2007-04-04 15:02     ` MODULE_MAINTAINER Adrian Bunk
2007-04-04 15:50       ` MODULE_MAINTAINER Rene Herman
2007-04-04 16:00         ` MODULE_MAINTAINER Alan Cox
2007-04-04 16:06           ` MODULE_MAINTAINER Marcel Holtmann
2007-04-04 16:38           ` MODULE_MAINTAINER Rene Herman
2007-04-04 17:00             ` MODULE_MAINTAINER Takashi Iwai
2007-04-04 17:48               ` MODULE_MAINTAINER Adrian Bunk
2007-04-04 18:01                 ` MODULE_MAINTAINER Rene Herman
2007-04-04 19:12                   ` MODULE_MAINTAINER Adrian Bunk
2007-04-05  0:08                     ` MODULE_MAINTAINER Stefan Richter
2007-04-23  9:33             ` MODULE_MAINTAINER Rene Herman
2007-04-23 11:24               ` MODULE_MAINTAINER Rusty Russell
2007-04-23 11:52                 ` Robert P. J. Day [this message]
2007-04-23 12:00                   ` MODULE_MAINTAINER Robert P. J. Day
2007-04-23 12:32                   ` MODULE_MAINTAINER Rene Herman
2007-04-26  1:18                     ` MODULE_MAINTAINER Andrew Morton
2007-04-26 10:03                       ` MODULE_MAINTAINER Rusty Russell
2007-04-26 10:41                       ` MODULE_MAINTAINER Rene Herman
2007-04-26 13:54                         ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 14:55                           ` MODULE_MAINTAINER Rene Herman
2007-04-26 16:00                             ` MODULE_MAINTAINER Alan Cox
2007-04-26 16:45                               ` MODULE_MAINTAINER Rene Herman
2007-04-26 15:41                           ` MODULE_MAINTAINER Randy Dunlap
2007-04-26 15:52                             ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 16:44                               ` MODULE_MAINTAINER Randy Dunlap
2007-04-26 17:12                                 ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 19:37                               ` MODULE_MAINTAINER Krzysztof Halasa
2007-04-26 19:43                                 ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 20:02                                   ` MODULE_MAINTAINER Krzysztof Halasa
2007-04-26 20:24                                     ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 21:51                                       ` MODULE_MAINTAINER Krzysztof Halasa
2007-04-26 22:01                                         ` MODULE_MAINTAINER Adrian Bunk
2007-04-26 22:07                                           ` MODULE_MAINTAINER Krzysztof Halasa
2007-04-26 22:28                                       ` MODULE_MAINTAINER Rene Herman
2007-04-26 20:11                                   ` MODULE_MAINTAINER Rene Herman
2007-04-26 22:24                                     ` MODULE_MAINTAINER Gene Heskett
2007-04-27  9:06                                       ` MODULE_MAINTAINER Stefan Richter
2007-04-26 22:03                               ` MODULE_MAINTAINER Rene Herman
2007-04-27 21:06                                 ` MODULE_MAINTAINER Rene Herman
2007-04-28 21:03                                   ` MODULE_MAINTAINER Krzysztof Halasa
2007-04-23 23:46                   ` MODULE_MAINTAINER Rusty Russell

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=Pine.LNX.4.64.0704230748270.25463@localhost.localdomain \
    --to=rpjday@mindspring.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bunk@stusta.de \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcel@holtmann.org \
    --cc=rene.herman@gmail.com \
    --cc=rusty@rustcorp.com.au \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®