inbox: add ->version method

[public-inbox.git] / script / public-inbox-convert

diff --git a/script/public-inbox-convert b/script/public-inbox-convert
index 9aa27814d4538a28c75c513d3b90a7e825714767..56a810eb054ffc1ea41585eab1052dabb0b7bc12 100755 (executable)
--- a/script/public-inbox-convert
+++ b/script/public-inbox-convert
@@ -1,16 +1,15 @@
 #!/usr/bin/perl -w
-# Copyright (C) 2018 all contributors <meta@public-inbox.org>
+# Copyright (C) 2018-2019 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <http://www.gnu.org/licenses/agpl-3.0.txt>
 use strict;
 use warnings;
 use Getopt::Long qw(:config gnu_getopt no_ignore_case auto_abbrev);
-use PublicInbox::MIME;
 use PublicInbox::InboxWritable;
 use PublicInbox::Config;
 use PublicInbox::V2Writable;
-use PublicInbox::Import;
 use PublicInbox::Spawn qw(spawn);
 use Cwd 'abs_path';
+use File::Copy 'cp'; # preserves permissions:
 my $usage = "Usage: public-inbox-convert OLD NEW\n";
 my $jobs;
 my $index = 1;
@@ -19,47 +18,71 @@ my %opts = (
        '--index!' => \$index,
 );
 GetOptions(%opts) or die "bad command-line args\n$usage";
-GetOptions(%opts) or die "bad command-line args\n$usage";
 my $old_dir = shift or die $usage;
 my $new_dir = shift or die $usage;
 die "$new_dir exists\n" if -d $new_dir;
 die "$old_dir not a directory\n" unless -d $old_dir;
-my $config = PublicInbox::Config->new;
+my $config = eval { PublicInbox::Config->new };
 $old_dir = abs_path($old_dir);
 my $old;
-$config->each_inbox(sub {
-       $old = $_[0] if abs_path($_[0]->{mainrepo}) eq $old_dir;
-});
+if ($config) {
+       $config->each_inbox(sub {
+               $old = $_[0] if abs_path($_[0]->{inboxdir}) eq $old_dir;
+       });
+}
 unless ($old) {
        warn "W: $old_dir not configured in " .
                PublicInbox::Config::default_file() . "\n";
        $old = {
-               mainrepo => $old_dir,
+               inboxdir => $old_dir,
                name => 'ignored',
                address => [ 'old@example.com' ],
        };
        $old = PublicInbox::Inbox->new($old);
 }
 $old = PublicInbox::InboxWritable->new($old);
-if (($old->{version} || 1) >= 2) {
+if ($old->version >= 2) {
        die "Only conversion from v1 inboxes is supported\n";
 }
 my $new = { %$old };
-delete $new->{altid}; # TODO: support altid for v2
-$new->{mainrepo} = abs_path($new_dir);
+$new->{inboxdir} = abs_path($new_dir);
 $new->{version} = 2;
 $new = PublicInbox::InboxWritable->new($new);
 my $v2w;
 $old->umask_prepare;
+
+sub link_or_copy ($$) {
+       my ($src, $dst) = @_;
+       link($src, $dst) and return;
+       $!{EXDEV} or warn "link $src, $dst failed: $!, trying cp\n";
+       cp($src, $dst) or die "cp $src, $dst failed: $!\n";
+}
+
 $old->with_umask(sub {
-       local $ENV{GIT_CONFIG} = "$old->{mainrepo}/config";
+       my $old_cfg = "$old->{inboxdir}/config";
+       local $ENV{GIT_CONFIG} = $old_cfg;
+       my $new_cfg = "$new->{inboxdir}/all.git/config";
        $v2w = PublicInbox::V2Writable->new($new, 1);
        $v2w->init_inbox($jobs);
-       chomp(my $sr = $old->git->qx('config', 'core.sharedRepository'));
-       if ($sr ne '') {
-               PublicInbox::Import::run_die(['git', 'config',
-                       "--file=$new->{mainrepo}/all.git/config",
-                       'core.sharedRepository', $sr]);
+       unlink $new_cfg;
+       link_or_copy($old_cfg, $new_cfg);
+       if (my $alt = $new->{altid}) {
+               require PublicInbox::AltId;
+               foreach my $i (0..$#$alt) {
+                       my $src = PublicInbox::AltId->new($old, $alt->[$i], 0);
+                       $src->mm_alt or next;
+                       my $dst = PublicInbox::AltId->new($new, $alt->[$i], 1);
+                       $dst = $dst->{filename};
+                       $src->mm_alt->{dbh}->sqlite_backup_to_file($dst);
+               }
+       }
+       my $desc = "$old->{inboxdir}/description";
+       link_or_copy($desc, "$new->{inboxdir}/description") if -e $desc;
+       my $clone = "$old->{inboxdir}/cloneurl";
+       if (-e $clone) {
+               warn <<"";
+$clone may not be valid after migrating to v2, not copying
+
        }
 });
 my $state = '';
@@ -77,7 +100,7 @@ while (<$rd>) {
                $state = 'blob';
        } elsif (/^commit /) {
                $state = 'commit';
-       } elsif (/^data (\d+)/) {
+       } elsif (/^data ([0-9]+)/) {
                my $len = $1;
                $w->print($_) or $im->wfail;
                while ($len) {
@@ -88,7 +111,7 @@ while (<$rd>) {
                }
                next;
        } elsif ($state eq 'commit') {
-               if (m{^M 100644 :(\d+) (${h}{2}/${h}{38})}o) {
+               if (m{^M 100644 :([0-9]+) (${h}{2}/${h}{38})}o) {
                        my ($mark, $path) = ($1, $2);
                        $D{$path} = $mark;
                        if ($last && $last ne 'm') {
@@ -108,7 +131,7 @@ while (<$rd>) {
                        $last = 'd';
                        next;
                }
-               if (m{^from (:\d+)}) {
+               if (m{^from (:[0-9]+)}) {
                        $prev = $from;
                        $from = $1;
                        # no next