Merge branch 'maint'
authorJunio C Hamano <gitster@pobox.com>
Sun, 26 Oct 2008 23:21:08 +0000 (16:21 -0700)
committerJunio C Hamano <gitster@pobox.com>
Sun, 26 Oct 2008 23:21:08 +0000 (16:21 -0700)
* maint:
add -p: warn if only binary changes present
git-archive: work in bare repos
git-svn: change dashed git-config to git config

1  2 
archive.c
git-svn.perl
diff --combined archive.c
index 849eed553e9d6455c021664135a01b322eef9a31,45d242b884c2796b67329aaabfea62cd1e7bbecb..9ac455d889b72deba8c949da1d9efe2be3a50244
+++ b/archive.c
@@@ -15,7 -15,7 +15,7 @@@ static char const * const archive_usage
  
  #define USES_ZLIB_COMPRESSION 1
  
 -const struct archiver {
 +static const struct archiver {
        const char *name;
        write_archive_fn_t write_archive;
        unsigned int flags;
@@@ -29,10 -29,11 +29,10 @@@ static void format_subst(const struct c
                           struct strbuf *buf)
  {
        char *to_free = NULL;
 -      struct strbuf fmt;
 +      struct strbuf fmt = STRBUF_INIT;
  
        if (src == buf->buf)
                to_free = strbuf_detach(buf, NULL);
 -      strbuf_init(&fmt, 0);
        for (;;) {
                const char *b, *c;
  
@@@ -64,9 -65,10 +64,9 @@@ static void *sha1_file_to_archive(cons
  
        buffer = read_sha1_file(sha1, type, sizep);
        if (buffer && S_ISREG(mode)) {
 -              struct strbuf buf;
 +              struct strbuf buf = STRBUF_INIT;
                size_t size = 0;
  
 -              strbuf_init(&buf, 0);
                strbuf_attach(&buf, buffer, *sizep, *sizep + 1);
                convert_to_working_tree(path, buf.buf, buf.len, &buf);
                if (commit)
@@@ -336,5 -338,7 +336,7 @@@ int write_archive(int argc, const char 
        parse_treeish_arg(argv, &args, prefix);
        parse_pathspec_arg(argv + 1, &args);
  
+       git_config(git_default_config, NULL);
        return ar->write_archive(&args);
  }
diff --combined git-svn.perl
index ef6d773df1143b32f0bc421b59421f7e0eab60bd,2e68c68d49043bf7c99b88987a61fbabf2ca4560..f90ddac908f213fe8d7d58eea1aba45a04ce781e
@@@ -66,7 -66,7 +66,7 @@@ my ($_stdin, $_help, $_edit
        $_version, $_fetch_all, $_no_rebase,
        $_merge, $_strategy, $_dry_run, $_local,
        $_prefix, $_no_checkout, $_url, $_verbose,
 -      $_git_format, $_commit_url);
 +      $_git_format, $_commit_url, $_tag);
  $Git::SVN::_follow_parent = 1;
  my %remote_opts = ( 'username=s' => \$Git::SVN::Prompt::_username,
                      'config-dir=s' => \$Git::SVN::Ra::config_dir,
@@@ -131,15 -131,6 +131,15 @@@ my %cmd = 
                          'revision|r=i' => \$_revision,
                          'no-rebase' => \$_no_rebase,
                        %cmt_opts, %fc_opts } ],
 +      branch => [ \&cmd_branch,
 +                  'Create a branch in the SVN repository',
 +                  { 'message|m=s' => \$_message,
 +                    'dry-run|n' => \$_dry_run,
 +                    'tag|t' => \$_tag } ],
 +      tag => [ sub { $_tag = 1; cmd_branch(@_) },
 +               'Create a tag in the SVN repository',
 +               { 'message|m=s' => \$_message,
 +                 'dry-run|n' => \$_dry_run } ],
        'set-tree' => [ \&cmd_set_tree,
                        "Set an SVN repository to a git tree-ish",
                        { 'stdin|' => \$_stdin, %cmt_opts, %fc_opts, } ],
@@@ -430,15 -421,15 +430,15 @@@ sub cmd_dcommit 
        $head ||= 'HEAD';
        my @refs;
        my ($url, $rev, $uuid, $gs) = working_head_info($head, \@refs);
 +      unless ($gs) {
 +              die "Unable to determine upstream SVN information from ",
 +                  "$head history.\nPerhaps the repository is empty.";
 +      }
        $url = defined $_commit_url ? $_commit_url : $gs->full_url;
        my $last_rev = $_revision if defined $_revision;
        if ($url) {
                print "Committing to $url ...\n";
        }
 -      unless ($gs) {
 -              die "Unable to determine upstream SVN information from ",
 -                  "$head history.\nPerhaps the repository is empty.";
 -      }
        my ($linear_refs, $parents) = linearize_history($gs, \@refs);
        if ($_no_rebase && scalar(@$linear_refs) > 1) {
                warn "Attempting to commit more than one change while ",
        unlink $gs->{index};
  }
  
 +sub cmd_branch {
 +      my ($branch_name, $head) = @_;
 +
 +      unless (defined $branch_name && length $branch_name) {
 +              die(($_tag ? "tag" : "branch") . " name required\n");
 +      }
 +      $head ||= 'HEAD';
 +
 +      my ($src, $rev, undef, $gs) = working_head_info($head);
 +
 +      my $remote = Git::SVN::read_all_remotes()->{svn};
 +      my $glob = $remote->{ $_tag ? 'tags' : 'branches' };
 +      my ($lft, $rgt) = @{ $glob->{path} }{qw/left right/};
 +      my $dst = join '/', $remote->{url}, $lft, $branch_name, ($rgt || ());
 +
 +      my $ctx = SVN::Client->new(
 +              auth    => Git::SVN::Ra::_auth_providers(),
 +              log_msg => sub {
 +                      ${ $_[0] } = defined $_message
 +                              ? $_message
 +                              : 'Create ' . ($_tag ? 'tag ' : 'branch ' )
 +                              . $branch_name;
 +              },
 +      );
 +
 +      eval {
 +              $ctx->ls($dst, 'HEAD', 0);
 +      } and die "branch ${branch_name} already exists\n";
 +
 +      print "Copying ${src} at r${rev} to ${dst}...\n";
 +      $ctx->copy($src, $rev, $dst)
 +              unless $_dry_run;
 +
 +      $gs->fetch_all;
 +}
 +
  sub cmd_find_rev {
        my $revision_or_hash = shift or die "SVN or git revision required ",
                                            "as a command-line argument\n";
@@@ -848,28 -803,8 +848,28 @@@ sub cmd_commit_diff 
        }
  }
  
 +sub escape_uri_only {
 +      my ($uri) = @_;
 +      my @tmp;
 +      foreach (split m{/}, $uri) {
 +              s/([^\w.%+-]|%(?![a-fA-F0-9]{2}))/sprintf("%%%02X",ord($1))/eg;
 +              push @tmp, $_;
 +      }
 +      join('/', @tmp);
 +}
 +
 +sub escape_url {
 +      my ($url) = @_;
 +      if ($url =~ m#^([^:]+)://([^/]*)(.*)$#) {
 +              my ($scheme, $domain, $uri) = ($1, $2, escape_uri_only($3));
 +              $url = "$scheme://$domain$uri";
 +      }
 +      $url;
 +}
 +
  sub cmd_info {
        my $path = canonicalize_path(defined($_[0]) ? $_[0] : ".");
 +      my $fullpath = canonicalize_path($cmd_dir_prefix . $path);
        if (exists $_[1]) {
                die "Too many arguments specified\n";
        }
        my ($file_type, $diff_status) = find_file_type_and_diff_status($path);
  
        if (!$file_type && !$diff_status) {
 -              print STDERR "$path:  (Not a versioned resource)\n\n";
 -              return;
 +              print STDERR "svn: '$path' is not under version control\n";
 +              exit 1;
        }
  
        my ($url, $rev, $uuid, $gs) = working_head_info('HEAD');
        # canonicalize_path() will return "" to make libsvn 1.5.x happy,
        $path = "." if $path eq "";
  
 -      my $full_url = $url . ($path eq "." ? "" : "/$path");
 +      my $full_url = $url . ($fullpath eq "" ? "" : "/$fullpath");
  
        if ($_url) {
 -              print $full_url, "\n";
 +              print escape_url($full_url), "\n";
                return;
        }
  
        my $result = "Path: $path\n";
        $result .= "Name: " . basename($path) . "\n" if $file_type ne "dir";
 -      $result .= "URL: " . $full_url . "\n";
 +      $result .= "URL: " . escape_url($full_url) . "\n";
  
        eval {
                my $repos_root = $gs->repos_root;
                Git::SVN::remove_username($repos_root);
 -              $result .= "Repository Root: $repos_root\n";
 +              $result .= "Repository Root: " . escape_url($repos_root) . "\n";
        };
        if ($@) {
                $result .= "Repository Root: (offline)\n";
        }
  
        my ($lc_author, $lc_rev, $lc_date_utc);
 -      my @args = Git::SVN::Log::git_svn_log_cmd($rev, $rev, "--", $path);
 +      my @args = Git::SVN::Log::git_svn_log_cmd($rev, $rev, "--", $fullpath);
        my $log = command_output_pipe(@args);
        my $esc_color = qr/(?:\033\[(?:(?:\d+;)*\d*)?m)*/;
        while (<$log>) {
@@@ -1191,7 -1126,7 +1191,7 @@@ sub read_repo_config 
                my $v = $opts->{$o};
                my ($key) = ($o =~ /^([a-zA-Z\-]+)/);
                $key =~ s/-//g;
-               my $arg = 'git-config';
+               my $arg = 'git config';
                $arg .= ' --int' if ($o =~ /[:=]i$/);
                $arg .= ' --bool' if ($o !~ /[:=][sfi]$/);
                if (ref $v eq 'ARRAY') {
@@@ -2671,9 -2606,9 +2671,9 @@@ sub rebuild_from_rev_db 
  sub rebuild {
        my ($self) = @_;
        my $map_path = $self->map_path;
 -      return if (-e $map_path && ! -z $map_path);
 +      my $partial = (-e $map_path && ! -z $map_path);
        return unless ::verify_ref($self->refname.'^0');
 -      if ($self->use_svm_props || $self->no_metadata) {
 +      if (!$partial && ($self->use_svm_props || $self->no_metadata)) {
                my $rev_db = $self->rev_db_path;
                $self->rebuild_from_rev_db($rev_db);
                if ($self->use_svm_props) {
                $self->unlink_rev_db_symlink;
                return;
        }
 -      print "Rebuilding $map_path ...\n";
 +      print "Rebuilding $map_path ...\n" if (!$partial);
 +      my ($base_rev, $head) = ($partial ? $self->rev_map_max_norebuild(1) :
 +              (undef, undef));
        my ($log, $ctx) =
            command_output_pipe(qw/rev-list --pretty=raw --no-color --reverse/,
 -                              $self->refname, '--');
 +                              ($head ? "$head.." : "") . $self->refname,
 +                              '--');
        my $metadata_url = $self->metadata_url;
        remove_username($metadata_url);
        my $svn_uuid = $self->ra_uuid;
                    ($metadata_url && $url && ($url ne $metadata_url))) {
                        next;
                }
 +              if ($partial && $head) {
 +                      print "Partial-rebuilding $map_path ...\n";
 +                      print "Currently at $base_rev = $head\n";
 +                      $head = undef;
 +              }
  
                $self->rev_map_set($rev, $c);
                print "r$rev = $c\n";
        }
        command_close_pipe($log, $ctx);
 -      print "Done rebuilding $map_path\n";
 +      print "Done rebuilding $map_path\n" if (!$partial || !$head);
        my $rev_db_path = $self->rev_db_path;
        if (-f $self->rev_db_path) {
                unlink $self->rev_db_path or croak "unlink: $!";
@@@ -2862,12 -2789,6 +2862,12 @@@ sub rev_map_set 
  sub rev_map_max {
        my ($self, $want_commit) = @_;
        $self->rebuild;
 +      my ($r, $c) = $self->rev_map_max_norebuild($want_commit);
 +      $want_commit ? ($r, $c) : $r;
 +}
 +
 +sub rev_map_max_norebuild {
 +      my ($self, $want_commit) = @_;
        my $map_path = $self->map_path;
        stat $map_path or return $want_commit ? (0, undef) : 0;
        sysopen(my $fh, $map_path, O_RDONLY) or croak "open: $!";
@@@ -3459,12 -3380,11 +3459,12 @@@ sub generate_diff 
        while (<$diff_fh>) {
                chomp $_; # this gets rid of the trailing "\0"
                if ($state eq 'meta' && /^:(\d{6})\s(\d{6})\s
 -                                      $::sha1\s($::sha1)\s
 +                                      ($::sha1)\s($::sha1)\s
                                        ([MTCRAD])\d*$/xo) {
                        push @mods, {   mode_a => $1, mode_b => $2,
 -                                      sha1_b => $3, chg => $4 };
 -                      if ($4 =~ /^(?:C|R)$/) {
 +                                      sha1_a => $3, sha1_b => $4,
 +                                      chg => $5 };
 +                      if ($5 =~ /^(?:C|R)$/) {
                                $state = 'file_a';
                        } else {
                                $state = 'file_b';
@@@ -3716,7 -3636,6 +3716,7 @@@ sub R 
        my $fbat = $self->add_file($self->repo_path($m->{file_b}), $pbat,
                                $self->url_path($m->{file_a}), $self->{r});
        print "\tR\t$m->{file_a} => $m->{file_b}\n" unless $::_q;
 +      $self->apply_autoprops($file, $fbat);
        $self->chg_file($fbat, $m);
        $self->close_file($fbat,undef,$self->{pool});
  
@@@ -3743,52 -3662,33 +3743,52 @@@ sub change_file_prop 
        $self->SUPER::change_file_prop($fbat, $pname, $pval, $self->{pool});
  }
  
 -sub chg_file {
 -      my ($self, $fbat, $m) = @_;
 -      if ($m->{mode_b} =~ /755$/ && $m->{mode_a} !~ /755$/) {
 -              $self->change_file_prop($fbat,'svn:executable','*');
 -      } elsif ($m->{mode_b} !~ /755$/ && $m->{mode_a} =~ /755$/) {
 -              $self->change_file_prop($fbat,'svn:executable',undef);
 -      }
 -      my $fh = Git::temp_acquire('git_blob');
 -      if ($m->{mode_b} =~ /^120/) {
 +sub _chg_file_get_blob ($$$$) {
 +      my ($self, $fbat, $m, $which) = @_;
 +      my $fh = Git::temp_acquire("git_blob_$which");
 +      if ($m->{"mode_$which"} =~ /^120/) {
                print $fh 'link ' or croak $!;
                $self->change_file_prop($fbat,'svn:special','*');
 -      } elsif ($m->{mode_a} =~ /^120/ && $m->{mode_b} !~ /^120/) {
 +      } elsif ($m->{mode_a} =~ /^120/ && $m->{"mode_$which"} !~ /^120/) {
                $self->change_file_prop($fbat,'svn:special',undef);
        }
 -      my $size = $::_repository->cat_blob($m->{sha1_b}, $fh);
 -      croak "Failed to read object $m->{sha1_b}" if ($size < 0);
 +      my $blob = $m->{"sha1_$which"};
 +      return ($fh,) if ($blob =~ /^0{40}$/);
 +      my $size = $::_repository->cat_blob($blob, $fh);
 +      croak "Failed to read object $blob" if ($size < 0);
        $fh->flush == 0 or croak $!;
        seek $fh, 0, 0 or croak $!;
  
        my $exp = ::md5sum($fh);
        seek $fh, 0, 0 or croak $!;
 +      return ($fh, $exp);
 +}
  
 +sub chg_file {
 +      my ($self, $fbat, $m) = @_;
 +      if ($m->{mode_b} =~ /755$/ && $m->{mode_a} !~ /755$/) {
 +              $self->change_file_prop($fbat,'svn:executable','*');
 +      } elsif ($m->{mode_b} !~ /755$/ && $m->{mode_a} =~ /755$/) {
 +              $self->change_file_prop($fbat,'svn:executable',undef);
 +      }
 +      my ($fh_a, $exp_a) = _chg_file_get_blob $self, $fbat, $m, 'a';
 +      my ($fh_b, $exp_b) = _chg_file_get_blob $self, $fbat, $m, 'b';
        my $pool = SVN::Pool->new;
 -      my $atd = $self->apply_textdelta($fbat, undef, $pool);
 -      my $got = SVN::TxDelta::send_stream($fh, @$atd, $pool);
 -      die "Checksum mismatch\nexpected: $exp\ngot: $got\n" if ($got ne $exp);
 -      Git::temp_release($fh, 1);
 +      my $atd = $self->apply_textdelta($fbat, $exp_a, $pool);
 +      if (-s $fh_a) {
 +              my $txstream = SVN::TxDelta::new ($fh_a, $fh_b, $pool);
 +              my $res = SVN::TxDelta::send_txstream($txstream, @$atd, $pool);
 +              if (defined $res) {
 +                      die "Unexpected result from send_txstream: $res\n",
 +                          "(SVN::Core::VERSION: $SVN::Core::VERSION)\n";
 +              }
 +      } else {
 +              my $got = SVN::TxDelta::send_stream($fh_b, @$atd, $pool);
 +              die "Checksum mismatch\nexpected: $exp_b\ngot: $got\n"
 +                  if ($got ne $exp_b);
 +      }
 +      Git::temp_release($fh_b, 1);
 +      Git::temp_release($fh_a, 1);
        $pool->clear;
  }