Re: [PATCH] Remove usernames from all commit messages, not just when using svmprops

[Eric Wong <normalperson@yhbt.net>]

Adam Roben <aroben@apple.com> wrote:
> Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
> Signed-off-by: Adam Roben <aroben@apple.com>

Acked-by: Eric Wong <normalperson@yhbt.net>

> ---
>  git-svn.perl |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/git-svn.perl b/git-svn.perl
> index efc4c88..077d6b3 100755
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -1866,11 +1866,14 @@ sub make_log_entry {
>  	} elsif ($self->use_svnsync_props) {
>  		my $full_url = $self->svnsync->{url};
>  		$full_url .= "/$self->{path}" if length $self->{path};
> +		remove_username($full_url);
>  		my $uuid = $self->svnsync->{uuid};
>  		$log_entry{metadata} = "$full_url\@$rev $uuid";
>  		$email ||= "$author\@$uuid"
>  	} else {
> -		$log_entry{metadata} = $self->metadata_url. "\@$rev " .
> +		my $url = $self->metadata_url;
> +		remove_username($url);
> +		$log_entry{metadata} = "$url\@$rev " .
>  		                       $self->ra->get_uuid;
>  		$email ||= "$author\@" . $self->ra->get_uuid;
>  	}
> -- 
> 1.5.2.rc0.14.g520d-dirty

-- 
Eric Wong