RE: [PATCH] git-svn: convert CRLF to LF in commit message to SVN

From: Bennett, Brian <brian.bennett@transamerica.com>
Date: Dec 15, 2017, 14:11 UTC
Message-ID: <BL2PR05MB2179566C1EAE01C2ED34E7CC850B0@BL2PR05MB2179.namprd05.prod.outlook.com>
In-Reply-To: <20171214234943.GF3693@zaya.teonanacatl.net>

Thank you all for your guidance,

I have completed my test this morning with the patch and the 'git svn dcommit' is now SUCCESSFUl!

Thank you again for all of your help and I'll await to see when the patch is posted.

Brian Bennett | Supv System Admin & Support, TA TECH Change Mgmt/Production Support o: 319-355-7602 | c: 319-533-1094 e: brian.bennett@transamerica.com | w: www.transamerica.com

Transamerica 6400 C St. SW, Cedar Rapids, IA 52404 MS-2410 Facebook | LinkedIn

-----Original Message-----
From: Todd Zullinger [mailto:todd.zullinger@gmail.com] On Behalf Of Todd Zullinger
Sent: Thursday, December 14, 2017 5:50 PM
To: Bennett, Brian <Brian.Bennett@Transamerica.com>
Cc: Eric Wong <e@80x24.org>; Junio C Hamano <gitster@pobox.com>; git@vger.kernel.org
Subject: Re: [PATCH] git-svn: convert CRLF to LF in commit message to SVN

Hi Brian,

Bennett, Brian wrote:

> Thank you for your fast response,
> 
> I haven't done a build of this type before (so I could test the patch 
> first) so I'm trying to do that and get this far:

...

> I don't want to drag out testing the patch, so if either of you are 
> able to quickly guide me on what I am doing incorrectly I am willing 
> to get the build done so I can test it. If not, could one of you build 
> with the patch and somehow get that to me so I could test?

I don't know about building git for windows, but since the git-svn command is a perl script, it might be easier to just patch that file.  I think you can find the path where git-svn is installed using: git --exec-path

For this one-liner, I'd just manually apply it.

(If you want to use 'git apply' or the patch command, you'll have to edit the patch to adjust the name of the file, as it's git-svn.perl in the git tree.  The .perl suffix is dropped in the installed version.)

Hopefully that makes it easier for you to test Eric's patch.

-- Todd ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I am in shape. Round is a shape.