Ryan Anderson <[EMAIL PROTECTED]> writes:

> If I hope that nobody has done something like:
>       GIT_AUTHOR="Ryan <> Anderson"
>       GIT_AUTHOR_EMAIL="[EMAIL PROTECTED]"
>
> I get more confusing results.

The function git_author_info() would remove <> from the above
GIT_AUTHOR_* environment values by calling ident.c:copy(), so I
think you would get more-or-less what you _should_ expect without
getting confused.

    $ GIT_AUTHOR_NAME="Ryan <> Anderson" \
      GIT_AUTHOR_EMAIL="[EMAIL PROTECTED]" git-var -l
    GIT_COMMITTER_IDENT=Junio C Hamano <[EMAIL PROTECTED]> 1122855849 -0700
    GIT_AUTHOR_IDENT=Ryan  Anderson <[EMAIL PROTECTED]> 1122855849 -0700

>> Is this the culprit that produced this mechanical-looking line?
>> 
>>     To: [EMAIL PROTECTED],git@vger.kernel.org
>
> No, that line was exactly what I put into the readline entry.

I was mostly talking about Email::Valid seeming to be stripping
out the display-name[*] part and keeping only addr-spec[*] part,
like this:

    $ cat j.perl
    #!/usr/bin/perl
    use Email::Valid;
    for ('Junio C Hamano <[EMAIL PROTECTED]>',
         'Ryan Anderson <[EMAIL PROTECTED]>') {
        print $_, " => ", lc(Email::Valid->address($_)), "\n";
    }
    $ perl j.perl
    Junio C Hamano <[EMAIL PROTECTED]> => [EMAIL PROTECTED]
    Ryan Anderson <[EMAIL PROTECTED]> => [EMAIL PROTECTED]

Also, I wonder if running lc() to downcase the local-part[*] is
safe/allowed/correct; domain[*] part is case insensitive and
should be OK to downcase, though.

> ..., because of the way I pull in all the relevant emails
> from various places.  So I really needed a way to cull the duplicates.
> ...  I could do soemthing like this, instead, I suppose:

I understand your needs, and you can make it a "sub
filter_dups", which I think would make things a lot more
pleasant to read.


[Footnote]

Terms marked [*] are from RFC2822, section 3.4.

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to