Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro_alves@portugalmail.pt>
To: Lerele <lerele@champenstudios.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [gdbserver/win32] (3/11) Fix suspend count handling
Date: Mon, 03 Dec 2007 03:53:00 -0000	[thread overview]
Message-ID: <47535716.3010208@portugalmail.pt> (raw)
In-Reply-To: <4751B674.50306@champenstudios.com>

Lerele wrote:
> 
> Just a couple small tiny details:
> 
> Not that it's super important, but in all the series of patches where 
> both our names appear, I think my name should be listed second (if at 
> all) in the changelog, as Pedro did the vast majority of the work for 
> these patches, and merged with the one I sent.
> What's fair, is fair, I think. :)
> 

I don't think there's any written rule about the order
of the names, but, if were to write one, I'd say that the
chronological order in which people touched the patch
would be the most objective.  Saying who's done more than
who is very subjective, and honestly, I'm certainly not
losing my sleep over that.

> Also, is it necessary to include the eMail?

This will need an answer from someone who knows the
rules better than me.

It is costumary to post the raw email address on the body
of the patch submission messages, on the ChangeLog entry.
I'll try to remember not putting yours raw the next time
I submit a patch with your name on it.  Please understand
that you'd be the exception to the rule, so it is
susceptible to being forgotten or overlooked.

Looking over the ChangeLog files of gcc/binutils/gdb [1],
I see a few entries setting precedent of using mangled
addresses.

[1] grep "^[0-9]" ChangeLog*| grep -v "@"

> I'd prefer not to include mine if this is possible, if this can lower 
> spam getting into my mail box.
> 

If you're already being spammed, then I doubt this
can make a difference.

I'd like to move forward, so I've removed your email
address from the pending entries, making them like like
so for now:

2007-12-03  Leo Zayas
	    Pedro Alves  <pedro_alves@portugalmail.pt>

Changing it to some other form if needed can be easilly
done later.

Patch checked in.

-- 
Pedro Alves


  reply	other threads:[~2007-12-03  3:53 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-12  2:07 Pedro Alves
2007-12-01 18:53 ` Daniel Jacobowitz
2007-12-01 19:36   ` Lerele
2007-12-03  3:53     ` Pedro Alves [this message]
2007-12-03  3:53   ` Pedro Alves

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=47535716.3010208@portugalmail.pt \
    --to=pedro_alves@portugalmail.pt \
    --cc=gdb-patches@sourceware.org \
    --cc=lerele@champenstudios.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox