Re: CVS import [SOLVED]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin wrote:
Hi,

On Mon, 16 Feb 2009, Johannes Schindelin wrote:

On Mon, 16 Feb 2009, Ferry Huberts (Pelagic) wrote:

I solved it:

it has to do with the
core.autocrlf=input
core.safecrlf=true

settings I had in my global config.
Thanks!

Maybe the manual page should warn against having these defined?
Maybe it should be solved differently? As cvsimport needs to operate with autocrlf=false, it seems, it could set that variable when it creates a repository, and check the variable otherwise (erroring out if it is set inappropriately)?

IOW something like this:

-- snip --
 git-cvsimport.perl |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/git-cvsimport.perl b/git-cvsimport.perl
index e439202..a27cc94 100755
--- a/git-cvsimport.perl
+++ b/git-cvsimport.perl
@@ -562,12 +562,16 @@ my %index; # holds filenames of one index per branch
 unless (-d $git_dir) {
 	system("git-init");
 	die "Cannot init the GIT db at $git_tree: $?\n" if $?;
+	system("git-config core.autocrlf false");
+	die "Cannot set core.autocrlf false" if $?;
 	system("git-read-tree");
 	die "Cannot init an empty tree: $?\n" if $?;
$last_branch = $opt_o;
 	$orig_branch = "";
 } else {
+	die "Cannot operate with core.autocrlf other than 'false'"
+		if (`git-config --bool core.autocrlf` =~ /true|input/);
 	open(F, "git-symbolic-ref HEAD |") or
 		die "Cannot run git-symbolic-ref: $!\n";
 	chomp ($last_branch = <F>);
-- snap --

If you could add a test to t9600 and a few words to the man page, that would be awesome.

Ciao,
Dscho

P.S.: I think the same strategy should be applied to git-svn...
I'm willing to give it a stab but I'm not versed on Perl at all. C and Java I can do without breaking a sweat though. Isn't it a better idea to have the original authors do this? They understand the code. Also, doing this would constitute my first patch to git. I'm unfamiliar with its codebase and the requirements of the people that contribute to it. Willing to learn though :-) My patches would then probably need some review and would take a bit longer to develop. If that's acceptable then I'm willing to try.

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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux