inbox: add ->version method

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

diff --git a/script/public-inbox-convert b/script/public-inbox-convert
index 9d2d2512a403f99f2d994429d43f166796575e9e..56a810eb054ffc1ea41585eab1052dabb0b7bc12 100755 (executable)
--- a/script/public-inbox-convert
+++ b/script/public-inbox-convert
@@ -4,11 +4,9 @@
 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:
@@ -20,7 +18,6 @@ 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;
@@ -30,25 +27,25 @@ $old_dir = abs_path($old_dir);
 my $old;
 if ($config) {
        $config->each_inbox(sub {
-               $old = $_[0] if abs_path($_[0]->{mainrepo}) eq $old_dir;
+               $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 };
-$new->{mainrepo} = abs_path($new_dir);
+$new->{inboxdir} = abs_path($new_dir);
 $new->{version} = 2;
 $new = PublicInbox::InboxWritable->new($new);
 my $v2w;
@@ -62,9 +59,9 @@ sub link_or_copy ($$) {
 }
 
 $old->with_umask(sub {
-       my $old_cfg = "$old->{mainrepo}/config";
+       my $old_cfg = "$old->{inboxdir}/config";
        local $ENV{GIT_CONFIG} = $old_cfg;
-       my $new_cfg = "$new->{mainrepo}/all.git/config";
+       my $new_cfg = "$new->{inboxdir}/all.git/config";
        $v2w = PublicInbox::V2Writable->new($new, 1);
        $v2w->init_inbox($jobs);
        unlink $new_cfg;
@@ -79,9 +76,9 @@ $old->with_umask(sub {
                        $src->mm_alt->{dbh}->sqlite_backup_to_file($dst);
                }
        }
-       my $desc = "$old->{mainrepo}/description";
-       link_or_copy($desc, "$new->{mainrepo}/description") if -e $desc;
-       my $clone = "$old->{mainrepo}/cloneurl";
+       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