All of lore.kernel.org
 help / color / mirror / Atom feed
From: Petr Baudis <pasky@suse.cz>
To: Eric Raymond <esr@thyrsus.com>
Cc: git@vger.kernel.org
Subject: Re: CIA hook for contrib/
Date: Wed, 21 Apr 2010 23:04:27 +0200	[thread overview]
Message-ID: <20100421210427.GF3563@machine.or.cz> (raw)
In-Reply-To: <20100421155149.GA11223@thyrsus.com>

Thanks for doing all the changes!

On Wed, Apr 21, 2010 at 11:51:49AM -0400, Eric Raymond wrote:
> Petr Baudis <pasky@suse.cz>:
> > I would personally prefer to have this configurable; I consider trying
> > to protect your e-mail address against harvesters is a lost fight anyway
> > and you'd be much better off just getting a good spam filter, rather
> > than making yours and others' life harder by trying to fight in vain.
> > But it's not too important for me since I can just disable this easily.
> 
> But email name collisions within projects are vanishingly rare, so I
> don't see a lot of benefit in publishing the FQDN.

That's a good point, I guess I was just ranting in general. ;-)

> > Your life would be much easier in both the shell and python script if
> > you used something like:
> > 
> > 	git log -1 '--pretty=format:%an <%ae>%n%at%n%s'
> > 
> > You would also get <ts> in the correct format, incl. timezone correction.
> 
> Good point.  I inherited that nasty code; perhaps this facility did not
> exist when it was written.

It most probably wasn't, the original perl script in particular was
written at the very dawn of time from git history perspective.

-- 
				Petr "Pasky" Baudis
http://pasky.or.cz/ | "Ars longa, vita brevis." -- Hippocrates

  reply	other threads:[~2010-04-21 21:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-27 10:26 CIA hook for contrib/ Eric Raymond
2010-03-29  6:14 ` Junio C Hamano
2010-03-29  9:04   ` Eric Raymond
2010-04-21 10:10 ` Petr Baudis
2010-04-21 15:51   ` Eric Raymond
2010-04-21 21:04     ` Petr Baudis [this message]
2010-04-21 21:55       ` Eric Raymond
2010-04-21 22:00         ` Eric Raymond

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=20100421210427.GF3563@machine.or.cz \
    --to=pasky@suse.cz \
    --cc=esr@thyrsus.com \
    --cc=git@vger.kernel.org \
    /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.