git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5]
@ 2007-04-24 18:01 Luiz Fernando N. Capitulino
  2007-04-24 20:52 ` Luiz Fernando N. Capitulino
  0 siblings, 1 reply; 6+ messages in thread
From: Luiz Fernando N. Capitulino @ 2007-04-24 18:01 UTC (permalink / raw)
  To: junkio; +Cc: git

 Hi,

 Some random things I've fixed while reading the code, they're
unrelated changes, but I think it's better to send in one shot.

Diff stat:

 Documentation/core-tutorial.txt |    9 ++++-----
 builtin-commit-tree.c           |    3 +--
 cache.h                         |    2 +-
 entry.c                         |   14 +++++++-------
 read-cache.c                    |   16 ++++++++--------
 5 files changed, 21 insertions(+), 23 deletions(-)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/5]
  2007-04-24 18:01 [PATCH 0/5] Luiz Fernando N. Capitulino
@ 2007-04-24 20:52 ` Luiz Fernando N. Capitulino
  2007-04-24 20:56   ` Junio C Hamano
  0 siblings, 1 reply; 6+ messages in thread
From: Luiz Fernando N. Capitulino @ 2007-04-24 20:52 UTC (permalink / raw)
  To: junkio; +Cc: git

Em Tue, 24 Apr 2007 15:01:10 -0300
"Luiz Fernando N. Capitulino" <lcapitulino@mandriva.com.br> escreveu:

|  Hi,
| 
|  Some random things I've fixed while reading the code, they're
| unrelated changes, but I think it's better to send in one shot.

 Sorry for that, looks like git-send-email doesn't want to work
for me.

 Will send again shortly.

-- 
Luiz Fernando N. Capitulino

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/5]
  2007-04-24 20:52 ` Luiz Fernando N. Capitulino
@ 2007-04-24 20:56   ` Junio C Hamano
  2007-04-25 14:04     ` Luiz Fernando N. Capitulino
  0 siblings, 1 reply; 6+ messages in thread
From: Junio C Hamano @ 2007-04-24 20:56 UTC (permalink / raw)
  To: Luiz Fernando N. Capitulino; +Cc: junkio, git

I suspect that you need to quote your human readable name, as it
has a full-stop in it ("N.").  Maybe git-send-email should be
taught about it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/5]
  2007-04-24 20:56   ` Junio C Hamano
@ 2007-04-25 14:04     ` Luiz Fernando N. Capitulino
  2007-04-25 19:04       ` Hermes Trismegisto
  0 siblings, 1 reply; 6+ messages in thread
From: Luiz Fernando N. Capitulino @ 2007-04-25 14:04 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Em Tue, 24 Apr 2007 13:56:11 -0700
Junio C Hamano <junkio@cox.net> escreveu:

| I suspect that you need to quote your human readable name, as it
| has a full-stop in it ("N.").  Maybe git-send-email should be
| taught about it.

 It worked when I tested it sending the series to me and CC'ing
my gmail address.

 Will send this series by hand and figure out a better way to
do this...

-- 
Luiz Fernando N. Capitulino

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/5]
  2007-04-25 14:04     ` Luiz Fernando N. Capitulino
@ 2007-04-25 19:04       ` Hermes Trismegisto
  2007-04-25 19:36         ` Luiz Fernando N. Capitulino
  0 siblings, 1 reply; 6+ messages in thread
From: Hermes Trismegisto @ 2007-04-25 19:04 UTC (permalink / raw)
  To: Luiz Fernando N. Capitulino; +Cc: git

"Luiz Fernando N. Capitulino" <lcapitulino@mandriva.com.br>
writes:

> Em Tue, 24 Apr 2007 13:56:11 -0700
> Junio C Hamano <junkio@cox.net> escreveu:
>
> | I suspect that you need to quote your human readable name, as it
> | has a full-stop in it ("N.").  Maybe git-send-email should be
> | taught about it.
>
>  It worked when I tested it sending the series to me and CC'ing
> my gmail address.

Hmmm.  Maybe vger and my ISP have stricter filter based on set
of addresses that appear on To/Cc lines than the way you receive
mails?

In git-send-email, I notice that $author_name address on From:
line is quoted in dq pairs (in sub send_message) but none of the
addresses in @cc taken from the Signed-off-by lines are not
inspected nor quoted.  I do not know what is being done to To:
recipients offhand and I am too lazy to check.

>  Will send this series by hand and figure out a better way to
> do this...

Thanks.  They look reasonable, at least from my cursory look.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/5]
  2007-04-25 19:04       ` Hermes Trismegisto
@ 2007-04-25 19:36         ` Luiz Fernando N. Capitulino
  0 siblings, 0 replies; 6+ messages in thread
From: Luiz Fernando N. Capitulino @ 2007-04-25 19:36 UTC (permalink / raw)
  To: Hermes Trismegisto; +Cc: git

Em Wed, 25 Apr 2007 12:04:32 -0700
Hermes Trismegisto <junkio@cox.net> escreveu:

| "Luiz Fernando N. Capitulino" <lcapitulino@mandriva.com.br>
| writes:
| 
| > Em Tue, 24 Apr 2007 13:56:11 -0700
| > Junio C Hamano <junkio@cox.net> escreveu:
| >
| > | I suspect that you need to quote your human readable name, as it
| > | has a full-stop in it ("N.").  Maybe git-send-email should be
| > | taught about it.
| >
| >  It worked when I tested it sending the series to me and CC'ing
| > my gmail address.
| 
| Hmmm.  Maybe vger and my ISP have stricter filter based on set
| of addresses that appear on To/Cc lines than the way you receive
| mails?

 Maybe, but I've alreays used quilt send to submit patches and it
works quite well (not sure whether I've submitted more than five
patches to LKML though).

| In git-send-email, I notice that $author_name address on From:
| line is quoted in dq pairs (in sub send_message) but none of the
| addresses in @cc taken from the Signed-off-by lines are not
| inspected nor quoted.  I do not know what is being done to To:
| recipients offhand and I am too lazy to check.

 Heh, I can't check either.

| >  Will send this series by hand and figure out a better way to
| > do this...
| 
| Thanks.  They look reasonable, at least from my cursory look.

 You're welcome.

-- 
Luiz Fernando N. Capitulino

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2007-04-25 19:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-04-24 18:01 [PATCH 0/5] Luiz Fernando N. Capitulino
2007-04-24 20:52 ` Luiz Fernando N. Capitulino
2007-04-24 20:56   ` Junio C Hamano
2007-04-25 14:04     ` Luiz Fernando N. Capitulino
2007-04-25 19:04       ` Hermes Trismegisto
2007-04-25 19:36         ` Luiz Fernando N. Capitulino

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).