All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rene Herman <rene.herman@gmail.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: ALSA devel <alsa-devel@alsa-project.org>,
	Jaroslav Kysela <perex@suse.cz>
Subject: Re: [PATCH] remove email addresses from MODULE_AUTHOR tag
Date: Fri, 11 May 2007 23:12:58 +0200	[thread overview]
Message-ID: <4644DC5A.7060905@gmail.com> (raw)
In-Reply-To: <s5hd517ich2.wl%tiwai@suse.de>

On 05/11/2007 05:52 PM, Takashi Iwai wrote:

>> Following up the recent MODULE_MAINTAINER discussion on LKML:
>>
>> http://lkml.org/lkml/2007/4/4/170
>>
>> that concluded with MODULE_MAINTAINER not being a good idea, here's a patch 
>> that just deletes the email addresses from the MODULE_AUTHOR tags for ALSA.
>>
>> The email address is the problem I was trying to fix; with multiple current 
>> and non-current authors and maintainers who might not even be authors the 
>> address(es) available from the tag confuse the issue of whom to contact. 
>> It's moreover also information that easily outdated.
>>
>> A bit more than half of the tags in the kernel don't include an email 
>> address already (the core contributors typically don't include any) and I'll 
>> submit patches removing more.
> 
> Did we get consensus about this removal action?
> I've seen only one follow up by Krzysztof.
> (I personlly don't mind to remove mail addresses, though.)

Yep, Krysztof was the only one to specifically ACK but seeing as how I asked 
for objections:

http://lkml.org/lkml/2007/4/27/488

I took not seeing any as more ACKs. Alan Cox is objecting a bit again though 
preferring the originally proposed MODULE_MAINTAINER:

http://lkml.org/lkml/2007/5/10/628

I seem unlikely to convince him but I believe it's upto specific subsystems 
whether or not they'll take patches removing the email adresses from the 
tag, ie, in the case of ALSA upto you and Jaroslav.

I do believe it actually solves a problem. In the end, all that I really 
care about are the ones that I specifically want to take on (which in the 
case of ALSA is eventually most or all of isa/, but I'm not sufficiently 
upto speed yet, and certainly not on the _real_ driver code as opposed to 
the bus glue) but in a global sense, I'd like to just see them all go.

Rene.

  reply	other threads:[~2007-05-11 21:15 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-11  1:08 [PATCH] remove email addresses from MODULE_AUTHOR tag Rene Herman
2007-05-11 15:52 ` Takashi Iwai
2007-05-11 21:12   ` Rene Herman [this message]
2007-05-12 17:01     ` Rene Herman
2007-05-19  1:38       ` Rene Herman
2007-05-19 14:35         ` Takashi Iwai
2007-05-22  9:54           ` Matthias Koenig

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=4644DC5A.7060905@gmail.com \
    --to=rene.herman@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=perex@suse.cz \
    --cc=tiwai@suse.de \
    /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.