Document that File::Find doesn't follow symlinks
[p5sagit/p5-mst-13.2.git] / lib / CPAN.pm
index 2a5ef29..8271076 100644 (file)
@@ -1,11 +1,12 @@
 package CPAN;
-use vars qw{$META $Signal $Cwd $End $Suppress_readline};
+use vars qw{$Try_autoload 
+           $META $Signal $Cwd $End $Suppress_readline %Dontload};
 
-$VERSION = '1.19';
+$VERSION = '1.27';
 
-# $Id: CPAN.pm,v 1.121 1997/02/03 09:08:23 k Exp $
+# $Id: CPAN.pm,v 1.160 1997/07/28 12:21:56 k Exp $
 
-# my $version = substr q$Revision: 1.121 $, 10; # only used during development
+# my $version = substr q$Revision: 1.160 $, 10; # only used during development
 
 use Carp ();
 use Config ();
@@ -22,8 +23,6 @@ use Safe ();
 use Text::ParseWords ();
 use Text::Wrap;
 
-$Cwd = Cwd::cwd();
-
 END { $End++; &cleanup; }
 
 %CPAN::DEBUG = qw(
@@ -53,17 +52,264 @@ use strict qw(vars);
                                           # MakeMaker, gives us
                                           # catfile and catdir
 
-$META ||= new CPAN;                 # In case we reeval ourselves we
-                                    # need a ||
-
-CPAN::Config->load unless defined $CPAN::No_Config_is_ok;
-
-@EXPORT = qw( 
+@EXPORT = qw(
             autobundle bundle expand force get
             install make readme recompile shell test clean
            );
 
+#-> sub CPAN::AUTOLOAD ;
+sub AUTOLOAD {
+    my($l) = $AUTOLOAD;
+    $l =~ s/.*:://;
+    my(%EXPORT);
+    @EXPORT{@EXPORT} = '';
+    if (exists $EXPORT{$l}){
+       CPAN::Shell->$l(@_);
+    } else {
+       my $ok = CPAN::Shell->try_dot_al($AUTOLOAD);
+       if ($ok) {
+           goto &$AUTOLOAD;
+       } else {
+           warn "not OK: $@";
+       }
+       warn "CPAN doesn't know how to autoload $AUTOLOAD :-(
+Nothing Done.
+";
+       sleep 1;
+       CPAN::Shell->h;
+    }
+}
+
+#-> sub CPAN::shell ;
+sub shell {
+    $Suppress_readline ||= ! -t STDIN;
+
+    my $prompt = "cpan> ";
+    local($^W) = 1;
+    unless ($Suppress_readline) {
+       require Term::ReadLine;
+#      import Term::ReadLine;
+       $term = Term::ReadLine->new('CPAN Monitor');
+       $readline::rl_completion_function =
+           $readline::rl_completion_function = 'CPAN::Complete::cpl';
+    }
+
+    no strict;
+    $META->checklock();
+    my $getcwd;
+    $getcwd = $CPAN::Config->{'getcwd'} || 'cwd';
+    my $cwd = CPAN->$getcwd();
+    my $rl_avail = $Suppress_readline ? "suppressed" :
+       ($term->ReadLine ne "Term::ReadLine::Stub") ? "enabled" :
+           "available (try ``install Bundle::CPAN'')";
+
+    print qq{
+cpan shell -- CPAN exploration and modules installation (v$CPAN::VERSION)
+Readline support $rl_avail
+
+} unless $CPAN::Config->{'inhibit_startup_message'} ;
+    while () {
+       if ($Suppress_readline) {
+           print $prompt;
+           last unless defined ($_ = <> );
+           chomp;
+       } else {
+           last unless defined ($_ = $term->readline($prompt));
+       }
+       s/^\s+//;
+       next if /^$/;
+       $_ = 'h' if $_ eq '?';
+       if (/^\!/) {
+           s/^\!//;
+           my($eval) = $_;
+           package CPAN::Eval;
+           use vars qw($import_done);
+           CPAN->import(':DEFAULT') unless $import_done++;
+           CPAN->debug("eval[$eval]") if $CPAN::DEBUG;
+           eval($eval);
+           warn $@ if $@;
+       } elsif (/^q(?:uit)?$/i) {
+           last;
+       } elsif (/./) {
+           my(@line);
+           if ($] < 5.00322) { # parsewords had a bug until recently
+               @line = split;
+           } else {
+               eval { @line = Text::ParseWords::shellwords($_) };
+               warn($@), next if $@;
+           }
+           $CPAN::META->debug("line[".join("|",@line)."]") if $CPAN::DEBUG;
+           my $command = shift @line;
+           eval { CPAN::Shell->$command(@line) };
+           warn $@ if $@;
+       }
+    } continue {
+       &cleanup, die "Goodbye\n" if $Signal;
+       chdir $cwd;
+       print "\n";
+    }
+}
+
+package CPAN::CacheMgr;
+use vars qw($Du);
+@CPAN::CacheMgr::ISA = qw(CPAN::InfoObj);
+use File::Find;
+
+package CPAN::Config;
+import ExtUtils::MakeMaker 'neatvalue';
+use vars qw(%can $dot_cpan);
+
+%can = (
+  'commit' => "Commit changes to disk",
+  'defaults' => "Reload defaults from disk",
+  'init'   => "Interactive setting of all options",
+);
+
+package CPAN::FTP;
+use vars qw($Ua);
+@CPAN::FTP::ISA = qw(CPAN::Debug);
+
+package CPAN::Complete;
+@CPAN::Complete::ISA = qw(CPAN::Debug);
+
+package CPAN::Index;
+use vars qw($last_time $date_of_03);
+@CPAN::Index::ISA = qw(CPAN::Debug);
+$last_time ||= 0;
+$date_of_03 ||= 0;
+
+package CPAN::InfoObj;
+@CPAN::InfoObj::ISA = qw(CPAN::Debug);
+
+package CPAN::Author;
+@CPAN::Author::ISA = qw(CPAN::InfoObj);
+
+package CPAN::Distribution;
+@CPAN::Distribution::ISA = qw(CPAN::InfoObj);
+
+package CPAN::Bundle;
+@CPAN::Bundle::ISA = qw(CPAN::Module);
+
+package CPAN::Module;
+@CPAN::Module::ISA = qw(CPAN::InfoObj);
+
+package CPAN::Shell;
+use vars qw($AUTOLOAD $redef @ISA);
+@CPAN::Shell::ISA = qw(CPAN::Debug);
+
+#-> sub CPAN::Shell::AUTOLOAD ;
+sub AUTOLOAD {
+    my($autoload) = $AUTOLOAD;
+    $autoload =~ s/.*:://;
+    if ($autoload =~ /^w/) {
+       if ($CPAN::META->has_inst('CPAN::WAIT')) {
+           CPAN::WAIT->wh;
+       } else {
+           print STDERR qq{
+Commands starting with "w" require CPAN::WAIT to be installed.
+Please consider installing CPAN::WAIT to use the fulltext index.
+For this you just need to type 
+    install CPAN::WAIT
+}
+       }
+    } else {
+       my $ok = CPAN::Shell->try_dot_al($AUTOLOAD);
+       if ($ok) {
+           goto &$AUTOLOAD;
+       } else {
+           warn "not OK: $@";
+       }
+       warn "CPAN::Shell doesn't know how to autoload $autoload :-(
+Nothing Done.
+";
+       sleep 1;
+       CPAN::Shell->h;
+    }
+}
+
+#-> CPAN::Shell::try_dot_al
+sub try_dot_al {
+    my($class,$autoload) = @_;
+    return unless $CPAN::Try_autoload;
+    # I don't see how to re-use that from the AutoLoader...
+    my($name,$ok);
+    # Braces used to preserve $1 et al.
+    {
+       my ($pkg,$func) = $autoload =~ /(.*)::([^:]+)$/;
+       $pkg =~ s|::|/|g;
+       if (defined($name=$INC{"$pkg.pm"}))
+           {
+               $name =~ s|^(.*)$pkg\.pm$|$1auto/$pkg/$func.al|;
+               $name = undef unless (-r $name); 
+           }
+       unless (defined $name)
+           {
+               $name = "auto/$autoload.al";
+               $name =~ s|::|/|g;
+           }
+    }
+    my $save = $@;
+    eval {local $SIG{__DIE__};require $name};
+    if ($@) {
+       if (substr($autoload,-9) eq '::DESTROY') {
+           *$autoload = sub {};
+           $ok = 1;
+       } else {
+           if ($name =~ s/(\w{12,})\.al$/substr($1,0,11).".al"/e){
+               eval {local $SIG{__DIE__};require $name};
+           }
+           if ($@){
+               $@ =~ s/ at .*\n//;
+               Carp::croak $@;
+           } else {
+               $ok = 1;
+           }
+       }
+    } else {
+       $ok = 1;
+    }
+    $@ = $save;
+    my $lm = Carp::longmess();
+#    warn "ok[$ok] autoload[$autoload] longmess[$lm]"; # debug
+    return $ok;
+}
+
+# This should be left to a runtime evaluation
+eval {require CPAN::WAIT;};
+unless ($@) {
+    unshift @ISA, "CPAN::WAIT";
+}
+
+#### autoloader is experimental
+#### to try it we have to set $Try_autoload and uncomment
+#### the use statement and uncomment the __END__ below
+#### You also need AutoSplit 1.01 available. MakeMaker will
+#### then build CPAN with all the AutoLoad stuff.
+# use AutoLoader;
+# $Try_autoload = 1;
+
+if ($CPAN::Try_autoload) {
+    for my $p (qw(
+              CPAN::Author CPAN::Bundle CPAN::CacheMgr CPAN::Complete
+              CPAN::Config CPAN::Debug CPAN::Distribution CPAN::FTP
+              CPAN::FTP::netrc CPAN::Index CPAN::InfoObj CPAN::Module
+                )) {
+       *{"$p\::AUTOLOAD"} = \&AutoLoader::AUTOLOAD;
+    }
+}
+
+
+package CPAN;
 
+$META ||= CPAN->new;                 # In case we reeval ourselves we
+                                    # need a ||
+
+# Do this after you have set up the whole inheritance
+CPAN::Config->load unless defined $CPAN::No_Config_is_ok;
+
+1;
+
+# __END__ # uncomment this and AutoSplit version 1.01 will split it
 
 #-> sub CPAN::autobundle ;
 sub autobundle;
@@ -77,29 +323,11 @@ sub force;
 sub install;
 #-> sub CPAN::make ;
 sub make;
-#-> sub CPAN::shell ;
-sub shell;
 #-> sub CPAN::clean ;
 sub clean;
 #-> sub CPAN::test ;
 sub test;
 
-#-> sub CPAN::AUTOLOAD ;
-sub AUTOLOAD {
-    my($l) = $AUTOLOAD;
-    $l =~ s/.*:://;
-    my(%EXPORT);
-    @EXPORT{@EXPORT} = '';
-    if (exists $EXPORT{$l}){
-       CPAN::Shell->$l(@_);
-    } else {
-       warn "CPAN doesn't know how to autoload $AUTOLOAD :-(
-Nothing Done.
-";
-       CPAN::Shell->h;
-    }
-}
-
 #-> sub CPAN::all ;
 sub all {
     my($mgr,$class) = @_;
@@ -129,7 +357,7 @@ sub checklock {
                        qq{    kill $other\n}.
                            qq{    rm $lockfile\n};
            } elsif (-w $lockfile) {
-               my($ans)=
+               my($ans) =
                    ExtUtils::MakeMaker::prompt
                        (qq{Other job not responding. Shall I overwrite }.
                         qq{the lockfile? (Y/N)},"y");
@@ -182,8 +410,7 @@ or
        $Signal = 1;
     };
     $SIG{'__DIE__'} = \&cleanup;
-    print STDERR "Signal handler set.\n"
-       unless $CPAN::Config->{'inhibit_startup_message'};
+    $self->debug("Signal handler set.") if $CPAN::DEBUG;
 }
 
 #-> sub CPAN::DESTROY ;
@@ -191,79 +418,78 @@ sub DESTROY {
     &cleanup; # need an eval?
 }
 
+#-> sub CPAN::cwd ;
+sub cwd {Cwd::cwd();}
+
+#-> sub CPAN::getcwd ;
+sub getcwd {Cwd::getcwd();}
+
 #-> sub CPAN::exists ;
 sub exists {
     my($mgr,$class,$id) = @_;
     CPAN::Index->reload;
-    Carp::croak "exists called without class argument" unless $class;
+    ### Carp::croak "exists called without class argument" unless $class;
     $id ||= "";
     exists $META->{$class}{$id};
 }
 
-#-> sub CPAN::hasFTP ;
-sub hasFTP {
-    my($self,$arg) = @_;
-    if (defined $arg) {
-       return $self->{'hasFTP'} = $arg;
-    } elsif (not defined $self->{'hasFTP'}) {
-       eval {require Net::FTP;};
-       $self->{'hasFTP'} = $@ ? 0 : 1;
-    }
-    return $self->{'hasFTP'};
-}
+#-> sub CPAN::has_inst
+sub has_inst {
+    my($self,$mod,$message) = @_;
+    Carp::croak("CPAN->has_inst() called without an argument")
+       unless defined $mod;
+    if (defined $message && $message eq "no") {
+       $Dontload{$mod}||=1;
+       return 0;
+    } elsif (exists $Dontload{$mod}) {
+       return 0;
+    }
+    my $file = $mod;
+    $file =~ s|::|/|g;
+    $file =~ s|/|\\|g if $^O eq 'MSWin32';
+    $file .= ".pm";
+    if (exists $INC{$file} && $INC{$file}) {
+#      warn "$file in %INC"; #debug
+       return 1;
+    } elsif ( my($obj) = CPAN::Shell->expand('Module',$mod) ) {
+       if ($obj->inst_file) {
+           require $file;
+           print "CPAN: $mod successfully required\n";
 
-#-> sub CPAN::hasLWP ;
-sub hasLWP {
-    my($self,$arg) = @_;
-    if (defined $arg) {
-       return $self->{'hasLWP'} = $arg;
-    } elsif (not defined $self->{'hasLWP'}) {
-       eval {require LWP;};
-       $LWP::VERSION ||= 0;
-        $self->{'hasLWP'} = $LWP::VERSION >= 4.98;
-    }
-    return $self->{'hasLWP'};
-}
+           if ($mod eq "CPAN::WAIT") {
+               push @CPAN::Shell::ISA, CPAN::WAIT unless $@;
+           }
+           warn $@ if $@;
+           return $@ ? 0 : 1;
+       } elsif ($mod eq "MD5"){
+           print qq{
+  CPAN: MD5 security checks disabled because MD5 not installed.
+  Please consider installing the MD5 module
 
-#-> sub CPAN::hasMD5 ;
-sub hasMD5 {
-    my($self,$arg) = @_;
-    if (defined $arg) {
-       $self->{'hasMD5'} = $arg;
-    } elsif (not defined $self->{'hasMD5'}) {
-       eval {require MD5;};
-       if ($@) {
-           print "MD5 security checks disabled because MD5 not installed.
-  Please consider installing MD5\n";
-           $self->{'hasMD5'} = 0;
-       } else {
-           $self->{'hasMD5'}++;
+};
+           sleep 2;
        }
-    }
-    return $self->{'hasMD5'};
-}
+    } elsif (eval { require $file }) {
+       # we can still have luck, if the program is fed with a bogus
+       # database or what
+       return 1;
+    } elsif ($mod eq "Net::FTP") {
+       warn qq{
+  Please, install Net::FTP as soon as possible. CPAN.pm installs it for you
+  if you just type
+      install Bundle::libnet
+  Thank you.
 
-#-> sub CPAN::hasWAIT ;
-sub hasWAIT {
-    my($self,$arg) = @_;
-    if (defined $arg) {
-       $self->{'hasWAIT'} = $arg;
-    } elsif (not defined $self->{'hasWAIT'}) {
-       eval {require CPAN::WAIT;};
-       if ($@) {
-           $self->{'hasWAIT'} = 0;
-       } else {
-           $self->{'hasWAIT'} = 1;
-       }
+};
+       sleep 2;
     }
-    return $self->{'hasWAIT'};
+    return 0;
 }
 
 #-> sub CPAN::instance ;
 sub instance {
     my($mgr,$class,$id) = @_;
     CPAN::Index->reload;
-    Carp::croak "instance called without class argument" unless $class;
     $id ||= "";
     $META->{$class}{$id} ||= $class->new(ID => $id );
 }
@@ -285,93 +511,9 @@ sub cleanup {
     return unless -f $META->{'LOCK'};
     unlink $META->{'LOCK'};
     print STDERR "Lockfile removed.\n";
-#    my $mess = Carp::longmess(@_);
-#    die @_;
-}
-
-#-> sub CPAN::shell ;
-sub shell {
-    $Suppress_readline ||= ! -t STDIN;
-
-    my $prompt = "cpan> ";
-    local($^W) = 1;
-    unless ($Suppress_readline) {
-       require Term::ReadLine;
-       import Term::ReadLine;
-       $term = new Term::ReadLine 'CPAN Monitor';
-       $readline::rl_completion_function =
-           $readline::rl_completion_function = 'CPAN::Complete::complete';
-    }
-
-    no strict;
-    $META->checklock();
-    my $cwd = Cwd::cwd();
-    # How should we determine if we have more than stub ReadLine enabled?
-    my $rl_avail = $Suppress_readline ? "suppressed" :
-       defined &Term::ReadLine::Perl::readline ? "enabled" :
-           "available (get Term::ReadKey and Term::ReadLine::Perl)";
-
-    print qq{
-cpan shell -- CPAN exploration and modules installation (v$CPAN::VERSION)
-Readline support $rl_avail
-
-} unless $CPAN::Config->{'inhibit_startup_message'} ;
-    while () {
-       if ($Suppress_readline) {
-           print $prompt;
-           last unless defined ($_ = <>);
-           chomp;
-       } else {
-#           if ($CPAN::DEBUG) {
-#               my($report,$item);
-#               $report = "";
-#               for $item (qw/ReadLine IN OUT MinLine findConsole Features/) {
-#                   $report .= sprintf "%-15s", $item;
-#                   $report .= $term->$item() || "";
-#                   $report .= "\n";
-#               }
-#               CPAN->debug($report);
-#           }
-           last unless defined ($_ = $term->readline($prompt));
-       }
-       s/^\s//;
-       next if /^$/;
-       $_ = 'h' if $_ eq '?';
-       if (/^\!/) {
-           s/^\!//;
-           my($eval) = $_;
-           package CPAN::Eval;
-           use vars qw($import_done);
-           CPAN->import(':DEFAULT') unless $import_done++;
-           CPAN->debug("eval[$eval]") if $CPAN::DEBUG;
-           eval($eval);
-           warn $@ if $@;
-       } elsif (/^q(?:uit)?$/i) {
-           last;
-       } elsif (/./) {
-           my(@line);
-           if ($] < 5.00322) { # parsewords had a bug until recently
-               @line = split;
-           } else {
-               eval { @line = Text::ParseWords::shellwords($_) };
-               warn($@), next if $@;
-           }
-           $CPAN::META->debug("line[".join("|",@line)."]") if $CPAN::DEBUG;
-           my $command = shift @line;
-           eval { CPAN::Shell->$command(@line) };
-           warn $@ if $@;
-       }
-    } continue {
-       &cleanup, die if $Signal;
-       chdir $cwd;
-       print "\n";
-    }
 }
 
 package CPAN::CacheMgr;
-use vars qw($Du);
-@CPAN::CacheMgr::ISA = qw(CPAN::InfoObj);
-use File::Find;
 
 #-> sub CPAN::CacheMgr::as_string ;
 sub as_string {
@@ -399,14 +541,14 @@ sub cachesize {
 # }
 
 #-> sub CPAN::CacheMgr::clean_cache ;
-sub clean_cache {
-    my $self = shift;
-    my $dir;
-    while ($self->{DU} > $self->{'MAX'} and $dir = shift @{$self->{FIFO}}) {
-       $self->force_clean_cache($dir);
-    }
-    $self->debug("leaving clean_cache with $self->{DU}") if $CPAN::DEBUG;
-}
+#=# sub clean_cache {
+#=#    my $self = shift;
+#=#    my $dir;
+#=#    while ($self->{DU} > $self->{'MAX'} and $dir = shift @{$self->{FIFO}}) {
+#=#        $self->force_clean_cache($dir);
+#=#    }
+#=#    $self->debug("leaving clean_cache with $self->{DU}") if $CPAN::DEBUG;
+#=# }
 
 #-> sub CPAN::CacheMgr::dir ;
 sub dir {
@@ -416,8 +558,12 @@ sub dir {
 #-> sub CPAN::CacheMgr::entries ;
 sub entries {
     my($self,$dir) = @_;
+    return unless defined $dir;
+    $self->debug("reading dir[$dir]") if $CPAN::DEBUG;
     $dir ||= $self->{ID};
-    my($cwd) = Cwd::cwd();
+    my $getcwd;
+    $getcwd  = $CPAN::Config->{'getcwd'} || 'cwd';
+    my($cwd) = CPAN->$getcwd();
     chdir $dir or Carp::croak("Can't chdir to $dir: $!");
     my $dh = DirHandle->new(".") or Carp::croak("Couldn't opendir $dir: $!");
     my(@entries);
@@ -432,22 +578,22 @@ sub entries {
        }
     }
     chdir $cwd or Carp::croak("Can't chdir to $cwd: $!");
-    sort {-M $b <=> -M $a} @entries;
+    sort { -M $b <=> -M $a} @entries;
 }
 
 #-> sub CPAN::CacheMgr::disk_usage ;
 sub disk_usage {
     my($self,$dir) = @_;
-    if (! defined $dir or $dir eq "") {
-       $self->debug("Cannot determine disk usage for some reason") if $CPAN::DEBUG;
-       return;
-    }
-    return if defined $self->{SIZE}{$dir};
+#    if (! defined $dir or $dir eq "") {
+#      $self->debug("Cannot determine disk usage for some reason") if $CPAN::DEBUG;
+#      return;
+#    }
+    return if $self->{SIZE}{$dir};
     local($Du) = 0;
     find(
         sub {
             return if -l $_;
-            $Du += -s;
+            $Du += -s _;
         },
         $dir
        );
@@ -456,14 +602,10 @@ sub disk_usage {
     $self->debug("measured $dir is $Du") if $CPAN::DEBUG;
     $self->{DU} += $Du/1024/1024;
     if ($self->{DU} > $self->{'MAX'} ) {
-       my($toremove) = $self->{FIFO}[0];
+       my($toremove) = shift @{$self->{FIFO}};
        printf "...Hold on a sec... cleaning from cache (%.1f>%.1f MB): $toremove\n",
                $self->{DU}, $self->{'MAX'};
-       $self->clean_cache;
-    } else {
-       $self->debug("NOT have to clean the cache: $self->{DU} <= $self->{'MAX'}")
-           if $CPAN::DEBUG;
-       $self->debug($self->as_string) if $CPAN::DEBUG;
+       $self->force_clean_cache($toremove);
     }
     $self->{DU};
 }
@@ -481,6 +623,9 @@ sub force_clean_cache {
 #-> sub CPAN::CacheMgr::new ;
 sub new {
     my $class = shift;
+    my $time = time;
+    my($debug,$t2);
+    $debug = "";
     my $self = {
                ID => $CPAN::Config->{'build_dir'},
                MAX => $CPAN::Config->{'build_cache'},
@@ -493,9 +638,12 @@ sub new {
     my $e;
     for $e ($self->entries) {
        next if $e eq ".." || $e eq ".";
-       $self->debug("Have to check size $e") if $CPAN::DEBUG;
        $self->disk_usage($e);
     }
+    $t2 = time;
+    $debug .= "timing of CacheMgr->new: ".($t2 - $time);
+    $time = $t2;
+    CPAN->debug($debug) if $CPAN::DEBUG;
     $self;
 }
 
@@ -509,10 +657,16 @@ sub debug {
                                                # eg readline
     ($caller) = caller(0);
     $caller =~ s/.*:://;
-#    print "caller[$caller]func[$func]line[$line]rest[@rest]\n";
-#    print "CPAN::DEBUG{caller}[$CPAN::DEBUG{$caller}]CPAN::DEBUG[$CPAN::DEBUG]\n";
+    $arg = "" unless defined $arg;
+    my $rest = join ":", map { defined $_ ? $_ : "UNDEF" } @rest;
+#    print "caller[$caller]\n";
+#    print "func[$func]\n";
+#    print "line[$line]\n";
+#    print "rest[@rest]\n";
+#    print "CPAN::DEBUG{caller}[$CPAN::DEBUG{$caller}]\n";
+#    print "CPAN::DEBUG[$CPAN::DEBUG]\n";
     if ($CPAN::DEBUG{$caller} & $CPAN::DEBUG){
-       if (ref $arg) {
+       if ($arg and ref $arg) {
            eval { require Data::Dumper };
            if ($@) {
                print $arg->as_string;
@@ -520,20 +674,12 @@ sub debug {
                print Data::Dumper::Dumper($arg);
            }
        } else {
-           print "Debug($caller:$func,$line,@rest): $arg\n"
+           print "Debug($caller:$func,$line,[$rest]): $arg\n"
        }
     }
 }
 
 package CPAN::Config;
-import ExtUtils::MakeMaker 'neatvalue';
-use vars qw(%can);
-
-%can = (
-  'commit' => "Commit changes to disk",
-  'defaults' => "Reload defaults from disk",
-  'init'   => "Interactive setting of all options",
-);
 
 #-> sub CPAN::Config::edit ;
 sub edit {
@@ -572,7 +718,8 @@ sub edit {
        } else {
            $CPAN::Config->{$o} = $args[0] if defined $args[0];
            print "    $o    ";
-           print defined $CPAN::Config->{$o} ? $CPAN::Config->{$o} : "UNDEFINED";
+           print defined $CPAN::Config->{$o} ?
+               $CPAN::Config->{$o} : "UNDEFINED";
        }
     }
 }
@@ -600,8 +747,9 @@ Please specify a filename where to save the configuration or try
     my $msg = <<EOF unless $configpm =~ /MyConfig/;
 
 # This is CPAN.pm's systemwide configuration file.  This file provides
-# defaults for users, and the values can be changed in a per-user configuration
-# file. The user-config file is being looked for as ~/.cpan/CPAN/MyConfig.pm.
+# defaults for users, and the values can be changed in a per-user
+# configuration file. The user-config file is being looked for as
+# ~/.cpan/CPAN/MyConfig.pm.
 
 EOF
     $msg ||= "\n";
@@ -621,7 +769,7 @@ EOF
 
     #$mode = 0444 | ( $mode & 0111 ? 0111 : 0 );
     #chmod $mode, $configpm;
-    $self->defaults;
+###why was that so?    $self->defaults;
     print "commit: wrote $configpm\n";
     1;
 }
@@ -646,87 +794,91 @@ sub init {
     1;
 }
 
-my $dot_cpan;
 #-> sub CPAN::Config::load ;
 sub load {
-    my($self) = @_;
+    my($self) = shift;
+    my(@miss);
     eval {require CPAN::Config;};       # We eval, because of some MakeMaker problems
     unshift @INC, $CPAN::META->catdir($ENV{HOME},".cpan") unless $dot_cpan++;
     eval {require CPAN::MyConfig;};     # where you can override system wide settings
-    unless ( $self->load_succeeded ) {
-         require CPAN::FirstTime;
-         my($configpm,$fh);
-         if (defined $INC{"CPAN/Config.pm"} && -w $INC{"CPAN/Config.pm"}) {
-             $configpm = $INC{"CPAN/Config.pm"};
-         } elsif (defined $INC{"CPAN/MyConfig.pm"} && -w $INC{"CPAN/MyConfig.pm"}) {
-             $configpm = $INC{"CPAN/MyConfig.pm"};
-         } else {
-             my($path_to_cpan) = File::Basename::dirname($INC{"CPAN.pm"});
-             my($configpmdir) = MM->catdir($path_to_cpan,"CPAN");
-             my($configpmtest) = MM->catfile($configpmdir,"Config.pm");
-             if (-d $configpmdir or File::Path::mkpath($configpmdir)) {
-                 if (-w $configpmtest) {
-                     $configpm = $configpmtest;
-                 } elsif (-w $configpmdir) {
-#_#_# following code dumped core on me with 5.003_11, a.k.
-                     unlink "$configpmtest.bak" if -f "$configpmtest.bak";
-                     rename $configpmtest, "$configpmtest.bak" if -f $configpmtest;
-                     my $fh = FileHandle->new;
-                     if ($fh->open(">$configpmtest")) {
-                         $fh->print("1;\n");
-                         $configpm = $configpmtest;
-                     } else {
-                         # Should never happen
-                         Carp::confess("Cannot open >$configpmtest");
-                     }
-                 }
-             }
-             unless ($configpm) {
-                 $configpmdir = MM->catdir($ENV{HOME},".cpan","CPAN");
-                 File::Path::mkpath($configpmdir);
-                 $configpmtest = MM->catfile($configpmdir,"MyConfig.pm");
-                 if (-w $configpmtest) {
-                     $configpm = $configpmtest;
-                 } elsif (-w $configpmdir) {
-#_#_# following code dumped core on me with 5.003_11, a.k.
-                     my $fh = FileHandle->new;
-                     if ($fh->open(">$configpmtest")) {
-                         $fh->print("1;\n");
-                         $configpm = $configpmtest;
-                     } else {
-                         # Should never happen
-                         Carp::confess("Cannot open >$configpmtest");
-                     }
-                 } else {
-                     Carp::confess(qq{WARNING: CPAN.pm is unable to }.
-                                   qq{create a configuration file.});
-                 }
-             }
-         }
-         CPAN->debug(qq{Calling CPAN::FirstTime::init("$configpm")})
-             if $CPAN::DEBUG;
-         print qq{
-Configuring CPAN.pm.
+    return unless @miss = $self->not_loaded;
+    require CPAN::FirstTime;
+    my($configpm,$fh,$redo,$theycalled);
+    $redo ||= "";
+    $theycalled++ if @miss==1 && $miss[0] eq 'inhibit_startup_message';
+    if (defined $INC{"CPAN/Config.pm"} && -w $INC{"CPAN/Config.pm"}) {
+       $configpm = $INC{"CPAN/Config.pm"};
+       $redo++;
+    } elsif (defined $INC{"CPAN/MyConfig.pm"} && -w $INC{"CPAN/MyConfig.pm"}) {
+       $configpm = $INC{"CPAN/MyConfig.pm"};
+       $redo++;
+    } else {
+       my($path_to_cpan) = File::Basename::dirname($INC{"CPAN.pm"});
+       my($configpmdir) = MM->catdir($path_to_cpan,"CPAN");
+       my($configpmtest) = MM->catfile($configpmdir,"Config.pm");
+       if (-d $configpmdir or File::Path::mkpath($configpmdir)) {
+           if (-w $configpmtest) {
+               $configpm = $configpmtest;
+           } elsif (-w $configpmdir) {
+               #_#_# following code dumped core on me with 5.003_11, a.k.
+               unlink "$configpmtest.bak" if -f "$configpmtest.bak";
+               rename $configpmtest, "$configpmtest.bak" if -f $configpmtest;
+               my $fh = FileHandle->new;
+               if ($fh->open(">$configpmtest")) {
+                   $fh->print("1;\n");
+                   $configpm = $configpmtest;
+               } else {
+                   # Should never happen
+                   Carp::confess("Cannot open >$configpmtest");
+               }
+           }
+       }
+       unless ($configpm) {
+           $configpmdir = MM->catdir($ENV{HOME},".cpan","CPAN");
+           File::Path::mkpath($configpmdir);
+           $configpmtest = MM->catfile($configpmdir,"MyConfig.pm");
+           if (-w $configpmtest) {
+               $configpm = $configpmtest;
+           } elsif (-w $configpmdir) {
+               #_#_# following code dumped core on me with 5.003_11, a.k.
+               my $fh = FileHandle->new;
+               if ($fh->open(">$configpmtest")) {
+                   $fh->print("1;\n");
+                   $configpm = $configpmtest;
+               } else {
+                   # Should never happen
+                   Carp::confess("Cannot open >$configpmtest");
+               }
+           } else {
+               Carp::confess(qq{WARNING: CPAN.pm is unable to }.
+                             qq{create a configuration file.});
+           }
+       }
+    }
+    local($") = ", ";
+    print qq{
+We have to reconfigure CPAN.pm due to following uninitialized parameters:
+
+@miss
+} if $redo && ! $theycalled;
+    print qq{
 $configpm initialized.
 };
-         CPAN::FirstTime::init($configpm);
-    }
+    sleep 2;
+    CPAN::FirstTime::init($configpm);
 }
 
-#-> sub CPAN::Config::load_succeeded ;
-sub load_succeeded {
-    my($miss) = 0;
+#-> sub CPAN::Config::not_loaded ;
+sub not_loaded {
+    my(@miss);
     for (qw(
            cpan_home keep_source_where build_dir build_cache index_expire
            gzip tar unzip make pager makepl_arg make_arg make_install_arg
            urllist inhibit_startup_message ftp_proxy http_proxy no_proxy
           )) {
-       unless (defined $CPAN::Config->{$_}){
-           $miss++;
-           CPAN->debug("undefined configuration parameter $_") if $CPAN::DEBUG;
-       }
+       push @miss, $_ unless defined $CPAN::Config->{$_};
     }
-    return !$miss;
+    return @miss;
 }
 
 #-> sub CPAN::Config::unload ;
@@ -758,8 +910,8 @@ EOF
     undef; #don't reprint CPAN::Config
 }
 
-#-> sub CPAN::Config::complete ;
-sub complete {
+#-> sub CPAN::Config::cpl ;
+sub cpl {
     my($word,$line,$pos) = @_;
     $word ||= "";
     my(@o_conf) = (keys %CPAN::Config::can, keys %$CPAN::Config);
@@ -767,34 +919,6 @@ sub complete {
 }
 
 package CPAN::Shell;
-use vars qw($AUTOLOAD $redef @ISA);
-@CPAN::Shell::ISA = qw(CPAN::Debug);
-if ($CPAN::META->hasWAIT) {
-    unshift @ISA, "CPAN::WAIT";
-}
-# private function ro re-eval this module (handy during development)
-#-> sub CPAN::Shell::AUTOLOAD ;
-sub AUTOLOAD {
-    my($autoload) = $AUTOLOAD;
-    $autoload =~ s/.*:://;
-    if ($autoload =~ /^w/) {
-       if ($CPAN::META->hasWAIT) {
-           CPAN::WAIT->wh;
-           return;
-       } else {
-           warn qq{
-Commands starting with "w" require CPAN::WAIT to be installed.
-Please consider installing CPAN::WAIT to use the fulltext index.
-Type "install CPAN::WAIT" and restart CPAN.pm.
-}
-       }
-    } else {
-       warn "CPAN::Shell doesn't know how to autoload $autoload :-(
-Nothing Done.
-";
-    }
-    CPAN::Shell->h;
-}
 
 #-> sub CPAN::Shell::h ;
 sub h {
@@ -835,7 +959,7 @@ sub a { print shift->format_result('Author',@_);}
 sub b {
     my($self,@which) = @_;
     CPAN->debug("which[@which]") if $CPAN::DEBUG;
-    my($incdir,$bdir,$dh); 
+    my($incdir,$bdir,$dh);
     foreach $incdir ($CPAN::Config->{'cpan_home'},@INC) {
        $bdir = $CPAN::META->catdir($incdir,"Bundle");
        if ($dh = DirHandle->new($bdir)) { # may fail
@@ -865,7 +989,7 @@ sub i {
     for $type (@type) {
        push @result, $self->expand($type,@args);
     }
-    my $result =  @result==1 ?
+    my $result =  @result == 1 ?
        $result[0]->as_string :
            join "", map {$_->as_glimpse} @result;
     $result ||= "No objects found of any type for argument @args\n";
@@ -917,11 +1041,13 @@ sub o {
                    }
                    $CPAN::DEBUG = $max;
                } else {
+                   my($known) = 0;
                    for (keys %CPAN::DEBUG) {
                        next unless lc($_) eq lc($what);
                        $CPAN::DEBUG |= $CPAN::DEBUG{$_};
+                       $known = 1;
                    }
-                   print "unknown argument [$what]\n";
+                   print "unknown argument [$what]\n" unless $known;
                }
            }
        } else {
@@ -951,7 +1077,10 @@ Known options:
 
 #-> sub CPAN::Shell::reload ;
 sub reload {
-    if ($_[1] =~ /cpan/i) {
+    my($self,$command,@arg) = @_;
+    $command ||= "";
+    $self->debug("self[$self]command[$command]arg[@arg]") if $CPAN::DEBUG;
+    if ($command =~ /cpan/i) {
        CPAN->debug("reloading the whole CPAN.pm") if $CPAN::DEBUG;
        my $fh = FileHandle->new($INC{'CPAN.pm'});
        local($/);
@@ -970,8 +1099,11 @@ sub reload {
        eval <$fh>;
        warn $@ if $@;
        print "\n$redef subroutines redefined\n";
-    } elsif ($_[1] =~ /index/) {
+    } elsif ($command =~ /index/) {
        CPAN::Index->force_reload;
+    } else {
+       print qq{cpan     re-evals the CPAN.pm file\n};
+       print qq{index    re-reads the index files\n};
     }
 }
 
@@ -1088,7 +1220,7 @@ sub _u_r_common {
        }
     }
     if ($what eq "r" && $version_zeroes) {
-       my $s = $version_zeroes>1 ? "s have" : " has";
+       my $s = $version_zeroes > 1 ? "s have" : " has";
        print qq{$version_zeroes installed module$s no version number to compare\n};
     }
     @result;
@@ -1193,7 +1325,7 @@ sub expand {
            push @m, $obj;
        }
     }
-    return @m;
+    return wantarray ? @m : $m[0];
 }
 
 #-> sub CPAN::Shell::format_result ;
@@ -1202,7 +1334,7 @@ sub format_result {
     my($type,@args) = @_;
     @args = '/./' unless @args;
     my(@result) = $self->expand($type,@args);
-    my $result =  @result==1 ?
+    my $result =  @result == 1 ?
        $result[0]->as_string :
            join "", map {$_->as_glimpse} @result;
     $result ||= "No objects of type $type found for argument @args\n";
@@ -1248,7 +1380,13 @@ sub rematein {
            $obj = $CPAN::META->instance('CPAN::Author',$s);
            print "Don't be silly, you can't $meth ", $obj->fullname, " ;-)\n";
        } else {
-           print "Warning: Cannot $meth $s, don't know what it is\n";
+           print qq{Warning: Cannot $meth $s, don\'t know what it is.
+Try the command
+
+    i /$s/
+
+to find objects with similar identifiers.
+};
        }
     }
 }
@@ -1271,8 +1409,6 @@ sub clean   { shift->rematein('clean',@_); }
 sub look   { shift->rematein('look',@_); }
 
 package CPAN::FTP;
-use vars qw($Ua);
-@CPAN::FTP::ISA = qw(CPAN::Debug);
 
 #-> sub CPAN::FTP::ftp_get ;
 sub ftp_get {
@@ -1305,14 +1441,22 @@ sub ftp_get {
 }
 
 #-> sub CPAN::FTP::localize ;
+# sorry for the ugly code here, I'll clean it up as soon as Net::FTP
+# is in the core
 sub localize {
     my($self,$file,$aslocal,$force) = @_;
     $force ||= 0;
     Carp::croak "Usage: ->localize(cpan_file,as_local_file[,$force])"
        unless defined $aslocal;
-    $self->debug("file [$file] aslocal [$aslocal]") if $CPAN::DEBUG;
+    $self->debug("file[$file] aslocal[$aslocal] force[$force]")
+       if $CPAN::DEBUG;
 
     return $aslocal if -f $aslocal && -r _ && ! $force;
+    my($restore) = 0;
+    if (-f $aslocal){
+       rename $aslocal, "$aslocal.bak";
+       $restore++;
+    }
 
     my($aslocal_dir) = File::Basename::dirname($aslocal);
     File::Path::mkpath($aslocal_dir);
@@ -1322,10 +1466,10 @@ sub localize {
     to insufficient permissions.\n} unless -w $aslocal_dir;
 
     # Inheritance is not easier to manage than a few if/else branches
-    if ($CPAN::META->hasLWP) {
+    if ($CPAN::META->has_inst('LWP')) {
        require LWP::UserAgent;
        unless ($Ua) {
-           $Ua = new LWP::UserAgent;
+           $Ua = LWP::UserAgent->new;
            my($var);
            $Ua->proxy('ftp',  $var)
                if $var = $CPAN::Config->{'ftp_proxy'} || $ENV{'ftp_proxy'};
@@ -1346,12 +1490,12 @@ sub localize {
        $self->debug("localizing[$url]") if $CPAN::DEBUG;
        if ($url =~ /^file:/) {
            my $l;
-           if ($CPAN::META->hasLWP) {
+           if ($CPAN::META->has_inst('LWP')) {
                require URI::URL;
-               my $u = new URI::URL $url;
+               my $u =  URI::URL->new($url);
                $l = $u->path;
            } else { # works only on Unix, is poorly constructed, but
-                     # hopefully better than nothing. 
+                     # hopefully better than nothing.
                     # RFC 1738 says fileurl BNF is
                     # fileurl = "file://" [ host | "localhost" ] "/" fpath
                     # Thanks to "Mark D. Baushke" <mdb@cisco.com> for the code
@@ -1361,13 +1505,13 @@ sub localize {
            return $l if -f $l && -r _;
            # Maybe mirror has compressed it?
            if (-f "$l.gz") {
-               $self->debug("found compressed $l.gz");
+               $self->debug("found compressed $l.gz") if $CPAN::DEBUG;
                system("$CPAN::Config->{gzip} -dc $l.gz > $aslocal");
                return $aslocal if -f $aslocal;
            }
        }
 
-       if ($CPAN::META->hasLWP) {
+       if ($CPAN::META->has_inst('LWP')) {
            print "Fetching $url with LWP\n";
            my $res = $Ua->mirror($url, $aslocal);
            if ($res->is_success) {
@@ -1377,7 +1521,7 @@ sub localize {
        if ($url =~ m|^ftp://(.*?)/(.*)/(.*)|) {
            # that's the nice and easy way thanks to Graham
            my($host,$dir,$getfile) = ($1,$2,$3);
-           if ($CPAN::META->hasFTP) {
+           if ($CPAN::META->has_inst('Net::FTP')) {
                $dir =~ s|/+|/|g;
                $self->debug("Going to fetch file [$getfile]
   from dir [$dir]
@@ -1385,13 +1529,6 @@ sub localize {
   as local [$aslocal]") if $CPAN::DEBUG;
                CPAN::FTP->ftp_get($host,$dir,$getfile,$aslocal) && return $aslocal;
                warn "Net::FTP failed for some reason\n";
-           } else {
-               warn qq{
-  Please, install Net::FTP as soon as possible. Just type
-    install Net::FTP
-  Thank you.
-
-}
            }
        }
 
@@ -1403,7 +1540,7 @@ sub localize {
        # does ncftp handle http?
        for $funkyftp ($CPAN::Config->{'lynx'},$CPAN::Config->{'ncftp'}) {
            next unless defined $funkyftp;
-           next unless -x $funkyftp;
+           next if $funkyftp =~ /^\s*$/;
            my($want_compressed);
            print(
                  qq{
@@ -1413,12 +1550,17 @@ Trying with $funkyftp to get
            $want_compressed = $aslocal =~ s/\.gz//;
            my($source_switch) = "";
            $source_switch = "-source" if $funkyftp =~ /\blynx$/;
+           $source_switch = "-c" if $funkyftp =~ /\bncftp$/;
            my($system) = "$funkyftp $source_switch '$url' > $aslocal";
+           $self->debug("system[$system]") if $CPAN::DEBUG;
            my($wstatus);
-           if (($wstatus = system($system)) == 0) {
+           if (($wstatus = system($system)) == 0
+               &&
+               -s $aslocal   # lynx returns 0 on my system even if it fails
+              ) {
                if ($want_compressed) {
                    $system = "$CPAN::Config->{'gzip'} -dt $aslocal";
-                   if (system($system)==0) {
+                   if (system($system) == 0) {
                        rename $aslocal, "$aslocal.gz";
                    } else {
                        $system = "$CPAN::Config->{'gzip'} $aslocal";
@@ -1427,7 +1569,7 @@ Trying with $funkyftp to get
                    return "$aslocal.gz";
                } else {
                    $system = "$CPAN::Config->{'gzip'} -dt $aslocal";
-                   if (system($system)==0) {
+                   if (system($system) == 0) {
                        $system = "$CPAN::Config->{'gzip'} -d $aslocal";
                        system($system);
                    } else {
@@ -1437,9 +1579,11 @@ Trying with $funkyftp to get
                }
            } else {
                my $estatus = $wstatus >> 8;
+               my $size = -s $aslocal;
                print qq{
 System call "$system"
-returned status $estatus (wstat $wstatus)
+returned status $estatus (wstat $wstatus), left
+$aslocal with size $size
 };
            }
        }
@@ -1451,7 +1595,7 @@ returned status $estatus (wstat $wstatus)
                my $timestamp = 0;
                my($dev,$ino,$mode,$nlink,$uid,$gid,$rdev,$size,$atime,$mtime,
                   $ctime,$blksize,$blocks) = stat($aslocal);
-               $timestamp = $mtime ||=0;
+               $timestamp = $mtime ||= 0;
 
                my($netrc) = CPAN::FTP::netrc->new;
                my($verbose) = $CPAN::DEBUG{'FTP'} & $CPAN::DEBUG ? " -v" : "";
@@ -1553,8 +1697,8 @@ Subprocess "|$CPAN::Config->{'ftp'}$verbose -n"
 
        print "Can't access URL $url.\n\n";
        my(@mess,$mess);
-       push @mess, "LWP" unless CPAN->hasLWP;
-       push @mess, "Net::FTP" unless CPAN->hasFTP;
+       push @mess, "LWP" unless CPAN->has_inst('LWP');
+       push @mess, "Net::FTP" unless CPAN->has_inst('Net::FTP');
        my($ext);
        for $ext (qw/lynx ncftp ftp/) {
            $CPAN::Config->{$ext} ||= "";
@@ -1568,9 +1712,80 @@ Subprocess "|$CPAN::Config->{'ftp'}$verbose -n"
        print Text::Wrap::wrap("","",$mess), "\n";
     }
     print "Cannot fetch $file\n";
+    if ($restore) {
+       rename "$aslocal.bak", $aslocal;
+       print "Trying to get away with old file:\n";
+       print $self->ls($aslocal);
+       return $aslocal;
+    }
     return;
 }
 
+# find2perl needs modularization, too, all the following is stolen
+# from there
+sub ls {
+    my($self,$name) = @_;
+    my ($dev,$ino,$mode,$nlink,$uid,$gid,$rdev,$sizemm,
+     $atime,$mtime,$ctime,$blksize,$blocks) = lstat($name);
+
+    my($perms,%user,%group);
+    my $pname = $name;
+
+    if ($blocks) {
+       $blocks = int(($blocks + 1) / 2);
+    }
+    else {
+       $blocks = int(($sizemm + 1023) / 1024);
+    }
+
+    if    (-f _) { $perms = '-'; }
+    elsif (-d _) { $perms = 'd'; }
+    elsif (-c _) { $perms = 'c'; $sizemm = &sizemm; }
+    elsif (-b _) { $perms = 'b'; $sizemm = &sizemm; }
+    elsif (-p _) { $perms = 'p'; }
+    elsif (-S _) { $perms = 's'; }
+    else         { $perms = 'l'; $pname .= ' -> ' . readlink($_); }
+
+    my(@rwx) = ('---','--x','-w-','-wx','r--','r-x','rw-','rwx');
+    my(@moname) = qw(Jan Feb Mar Apr May Jun Jul Aug Sep Oct Nov Dec);
+    my $tmpmode = $mode;
+    my $tmp = $rwx[$tmpmode & 7];
+    $tmpmode >>= 3;
+    $tmp = $rwx[$tmpmode & 7] . $tmp;
+    $tmpmode >>= 3;
+    $tmp = $rwx[$tmpmode & 7] . $tmp;
+    substr($tmp,2,1) =~ tr/-x/Ss/ if -u _;
+    substr($tmp,5,1) =~ tr/-x/Ss/ if -g _;
+    substr($tmp,8,1) =~ tr/-x/Tt/ if -k _;
+    $perms .= $tmp;
+
+    my $user = $user{$uid} || $uid;   # too lazy to implement lookup
+    my $group = $group{$gid} || $gid;
+
+    my($sec,$min,$hour,$mday,$mon,$year) = localtime($mtime);
+    my($timeyear);
+    my($moname) = $moname[$mon];
+    if (-M _ > 365.25 / 2) {
+       $timeyear = $year + 1900;
+    }
+    else {
+       $timeyear = sprintf("%02d:%02d", $hour, $min);
+    }
+
+    sprintf "%5lu %4ld %-10s %2d %-8s %-8s %8s %s %2d %5s %s\n",
+           $ino,
+                $blocks,
+                     $perms,
+                           $nlink,
+                               $user,
+                                    $group,
+                                         $sizemm,
+                                             $moname,
+                                                $mday,
+                                                    $timeyear,
+                                                        $pname;
+}
+
 package CPAN::FTP::netrc;
 
 sub new {
@@ -1596,7 +1811,7 @@ sub new {
                my($t) = shift @tokens;
                if ($t eq "default"){
                    $hasdefault++;
-                   warn "saw a default entry before tokens[@tokens]";
+                   # warn "saw a default entry before tokens[@tokens]";
                    last NETRC;
                }
                last TOKEN if $t eq "macdef";
@@ -1629,10 +1844,9 @@ sub contains {
 }
 
 package CPAN::Complete;
-@CPAN::Complete::ISA = qw(CPAN::Debug);
 
-#-> sub CPAN::Complete::complete ;
-sub complete {
+#-> sub CPAN::Complete::cpl ;
+sub cpl {
     my($word,$line,$pos) = @_;
     $word ||= "";
     $line ||= "";
@@ -1654,76 +1868,73 @@ sub complete {
     } elsif ( $line !~ /^[\!abdhimorutl]/ ) {
        @return = ();
     } elsif ($line =~ /^a\s/) {
-       @return = completex('CPAN::Author',$word);
+       @return = cplx('CPAN::Author',$word);
     } elsif ($line =~ /^b\s/) {
-       @return = completex('CPAN::Bundle',$word);
+       @return = cplx('CPAN::Bundle',$word);
     } elsif ($line =~ /^d\s/) {
-       @return = completex('CPAN::Distribution',$word);
+       @return = cplx('CPAN::Distribution',$word);
     } elsif ($line =~ /^([mru]|make|clean|test|install|readme|look)\s/ ) {
-       @return = (completex('CPAN::Module',$word),completex('CPAN::Bundle',$word));
+       @return = (cplx('CPAN::Module',$word),cplx('CPAN::Bundle',$word));
     } elsif ($line =~ /^i\s/) {
-       @return = complete_any($word);
+       @return = cpl_any($word);
     } elsif ($line =~ /^reload\s/) {
-       @return = complete_reload($word,$line,$pos);
+       @return = cpl_reload($word,$line,$pos);
     } elsif ($line =~ /^o\s/) {
-       @return = complete_option($word,$line,$pos);
+       @return = cpl_option($word,$line,$pos);
     } else {
        @return = ();
     }
     return @return;
 }
 
-#-> sub CPAN::Complete::completex ;
-sub completex {
+#-> sub CPAN::Complete::cplx ;
+sub cplx {
     my($class, $word) = @_;
     grep /^\Q$word\E/, map { $_->id } $CPAN::META->all($class);
 }
 
-#-> sub CPAN::Complete::complete_any ;
-sub complete_any {
+#-> sub CPAN::Complete::cpl_any ;
+sub cpl_any {
     my($word) = shift;
     return (
-           completex('CPAN::Author',$word),
-           completex('CPAN::Bundle',$word),
-           completex('CPAN::Distribution',$word),
-           completex('CPAN::Module',$word),
+           cplx('CPAN::Author',$word),
+           cplx('CPAN::Bundle',$word),
+           cplx('CPAN::Distribution',$word),
+           cplx('CPAN::Module',$word),
           );
 }
 
-#-> sub CPAN::Complete::complete_reload ;
-sub complete_reload {
+#-> sub CPAN::Complete::cpl_reload ;
+sub cpl_reload {
     my($word,$line,$pos) = @_;
     $word ||= "";
     my(@words) = split " ", $line;
     CPAN->debug("word[$word] line[$line] pos[$pos]") if $CPAN::DEBUG;
     my(@ok) = qw(cpan index);
-    return @ok if @words==1;
-    return grep /^\Q$word\E/, @ok if @words==2 && $word;
+    return @ok if @words == 1;
+    return grep /^\Q$word\E/, @ok if @words == 2 && $word;
 }
 
-#-> sub CPAN::Complete::complete_option ;
-sub complete_option {
+#-> sub CPAN::Complete::cpl_option ;
+sub cpl_option {
     my($word,$line,$pos) = @_;
     $word ||= "";
     my(@words) = split " ", $line;
     CPAN->debug("word[$word] line[$line] pos[$pos]") if $CPAN::DEBUG;
     my(@ok) = qw(conf debug);
-    return @ok if @words==1;
-    return grep /^\Q$word\E/, @ok if @words==2 && $word;
+    return @ok if @words == 1;
+    return grep /^\Q$word\E/, @ok if @words == 2 && $word;
     if (0) {
     } elsif ($words[1] eq 'index') {
        return ();
     } elsif ($words[1] eq 'conf') {
-       return CPAN::Config::complete(@_);
+       return CPAN::Config::cpl(@_);
     } elsif ($words[1] eq 'debug') {
        return sort grep /^\Q$word\E/, sort keys %CPAN::DEBUG, 'all';
     }
 }
 
 package CPAN::Index;
-use vars qw($last_time);
-@CPAN::Index::ISA = qw(CPAN::Debug);
-$last_time ||= 0;
 
 #-> sub CPAN::Index::force_reload ;
 sub force_reload {
@@ -1738,36 +1949,57 @@ sub reload {
     my $time = time;
 
     # XXX check if a newer one is available. (We currently read it from time to time)
+    for ($CPAN::Config->{index_expire}) {
+       $_ = 0.001 unless $_ > 0.001;
+    }
     return if $last_time + $CPAN::Config->{index_expire}*86400 > $time;
+    my($debug,$t2);
     $last_time = $time;
 
-    $cl->read_authindex($cl->reload_x(
+    $cl->rd_authindex($cl->reload_x(
                                      "authors/01mailrc.txt.gz",
                                      "01mailrc.gz",
                                      $force));
+    $t2 = time;
+    $debug = "timing reading 01[".($t2 - $time)."]";
+    $time = $t2;
     return if $CPAN::Signal; # this is sometimes lengthy
-    $cl->read_modpacks($cl->reload_x(
+    $cl->rd_modpacks($cl->reload_x(
                                     "modules/02packages.details.txt.gz",
                                     "02packag.gz",
                                     $force));
+    $t2 = time;
+    $debug .= "02[".($t2 - $time)."]";
+    $time = $t2;
     return if $CPAN::Signal; # this is sometimes lengthy
-    $cl->read_modlist($cl->reload_x(
+    $cl->rd_modlist($cl->reload_x(
                                    "modules/03modlist.data.gz",
                                    "03mlist.gz",
                                    $force));
+    $t2 = time;
+    $debug .= "03[".($t2 - $time)."]";
+    $time = $t2;
+    CPAN->debug($debug) if $CPAN::DEBUG;
 }
 
 #-> sub CPAN::Index::reload_x ;
 sub reload_x {
     my($cl,$wanted,$localname,$force) = @_;
     $force ||= 0;
-    my $abs_wanted = CPAN->catfile($CPAN::Config->{'keep_source_where'},$localname);
-    if (-f $abs_wanted &&
+    CPAN::Config->load; # we should guarantee loading wherever we rely
+                        # on Config XXX
+    my $abs_wanted = CPAN->catfile($CPAN::Config->{'keep_source_where'},
+                                  $localname);
+    if (
+       -f $abs_wanted &&
        -M $abs_wanted < $CPAN::Config->{'index_expire'} &&
-       !$force) {
-       my($s) = $CPAN::Config->{'index_expire'} != 1;
+       !$force
+       ) {
+       my $s = $CPAN::Config->{'index_expire'} == 1 ? "" : "s";
+#      use Devel::Symdump;
+#      print Devel::Symdump->isa_tree, "\n";
        $cl->debug(qq{$abs_wanted younger than $CPAN::Config->{'index_expire'} }.
-                  qq{day$s. I\'ll use that.\n});
+                  qq{day$s. I\'ll use that.});
        return $abs_wanted;
     } else {
        $force ||= 1;
@@ -1775,11 +2007,11 @@ sub reload_x {
     return CPAN::FTP->localize($wanted,$abs_wanted,$force);
 }
 
-#-> sub CPAN::Index::read_authindex ;
-sub read_authindex {
+#-> sub CPAN::Index::rd_authindex ;
+sub rd_authindex {
     my($cl,$index_target) = @_;
     my $pipe = "$CPAN::Config->{gzip} --decompress --stdout $index_target";
-    warn "Going to read $index_target\n";
+    print "Going to read $index_target\n";
     my $fh = FileHandle->new("$pipe|");
     while (<$fh>) {
        chomp;
@@ -1795,30 +2027,30 @@ sub read_authindex {
     $? and Carp::croak "FAILED $pipe: exit status [$?]";
 }
 
-#-> sub CPAN::Index::read_modpacks ;
-sub read_modpacks {
+#-> sub CPAN::Index::rd_modpacks ;
+sub rd_modpacks {
     my($cl,$index_target) = @_;
     my $pipe = "$CPAN::Config->{gzip} --decompress --stdout $index_target";
-    warn "Going to read $index_target\n";
+    print "Going to read $index_target\n";
     my $fh = FileHandle->new("$pipe|");
     while (<$fh>) {
-       next if 1../^\s*$/;
+       last if /^\s*$/;
+    }
+    while (<$fh>) {
        chomp;
        my($mod,$version,$dist) = split;
-       $version =~ s/^\+//;
+$dist = '' unless defined $dist;
+###    $version =~ s/^\+//;
 
        # if it as a bundle, instatiate a bundle object
-       my($bundle);
-       if ($mod =~ /^Bundle::(.*)/) {
-           $bundle = $1;
-       }
-
+       my($bundle,$id,$userid);
+       
        if ($mod eq 'CPAN') {
-           local($^W)=0;
+           local($^W)= 0;
            if ($version > $CPAN::VERSION){
                print qq{
-  Hey, you know what? There\'s a new CPAN.pm version (v$version)
-  available! I\'d suggest--provided you have time--you try
+  There\'s a new CPAN.pm version (v$version) available!
+  You might want to try
     install CPAN
     reload cpan
   without quitting the current session. It should be a seemless upgrade
@@ -1828,12 +2060,13 @@ sub read_modpacks {
                print qq{\n};
            }
            last if $CPAN::Signal;
+       } elsif ($mod =~ /^Bundle::(.*)/) {
+           $bundle = $1;
        }
 
-       my($id);
        if ($bundle){
            $id =  $CPAN::META->instance('CPAN::Bundle',$mod);
-           $id->set('CPAN_VERSION' => $version, 'CPAN_FILE' => $dist);
+###        $id->set('CPAN_VERSION' => $version, 'CPAN_FILE' => $dist);
 # This "next" makes us faster but if the job is running long, we ignore
 # rereads which is bad. So we have to be a bit slower again.
 #      } elsif ($CPAN::META->exists('CPAN::Module',$mod)) {
@@ -1841,12 +2074,19 @@ sub read_modpacks {
        } else {
            # instantiate a module object
            $id = $CPAN::META->instance('CPAN::Module',$mod);
-           $id->set('CPAN_VERSION' => $version, 'CPAN_FILE' => $dist);
+###        $id->set('CPAN_VERSION' => $version, 'CPAN_FILE' => $dist)
+###            if $id->cpan_version ne $version || $id->cpan_file ne $dist; # good speed in here
        }
 
-       # determine the author
-       my($userid) = $dist =~ /([^\/]+)/;
-       $id->set('CPAN_USERID' => $userid) if $userid =~ /\w/;
+       if ($id->cpan_file ne $dist){
+           # determine the author
+           ($userid) = $dist =~ /([^\/]+)/;
+           $id->set(
+                    'CPAN_USERID' => $userid,
+                    'CPAN_VERSION' => $version,
+                    'CPAN_FILE' => $dist
+                   );
+       }
 
        # instantiate a distribution object
        unless ($CPAN::META->exists('CPAN::Distribution',$dist)) {
@@ -1854,8 +2094,7 @@ sub read_modpacks {
                                  'CPAN::Distribution' => $dist
                                 )->set(
                                        'CPAN_USERID' => $userid
-                                      )
-                                    if $userid =~ /\w/;
+                                      );
        }
 
        return if $CPAN::Signal;
@@ -1864,19 +2103,23 @@ sub read_modpacks {
     $? and Carp::croak "FAILED $pipe: exit status [$?]";
 }
 
-#-> sub CPAN::Index::read_modlist ;
-sub read_modlist {
+#-> sub CPAN::Index::rd_modlist ;
+sub rd_modlist {
     my($cl,$index_target) = @_;
     my $pipe = "$CPAN::Config->{gzip} --decompress --stdout $index_target";
-    warn "Going to read $index_target\n";
+    print "Going to read $index_target\n";
     my $fh = FileHandle->new("$pipe|");
-    my $eval = "";
+    my $eval;
     while (<$fh>) {
-       next if 1../^\s*$/;
-       next if /use vars/; # will go away in 03...
-       $eval .= $_;
-       return if $CPAN::Signal;
+       if (/^Date:\s+(.*)/){
+           return if $date_of_03 eq $1;
+           ($date_of_03) = $1;
+       }
+       last if /^\s*$/;
     }
+    local($/) = undef;
+    $eval = <$fh>;
+    $fh->close;
     $eval .= q{CPAN::Modulelist->data;};
     local($^W) = 0;
     my($comp) = Safe->new("CPAN::Safe1");
@@ -1891,7 +2134,6 @@ sub read_modlist {
 }
 
 package CPAN::InfoObj;
-@CPAN::InfoObj::ISA = qw(CPAN::Debug);
 
 #-> sub CPAN::InfoObj::new ;
 sub new { my $this = bless {}, shift; %$this = @_; $this }
@@ -1943,7 +2185,6 @@ sub author {
 }
 
 package CPAN::Author;
-@CPAN::Author::ISA = qw(CPAN::InfoObj);
 
 #-> sub CPAN::Author::as_glimpse ;
 sub as_glimpse {
@@ -1968,7 +2209,6 @@ sub fullname { shift->{'FULLNAME'} }
 sub email    { shift->{'EMAIL'} }
 
 package CPAN::Distribution;
-@CPAN::Distribution::ISA = qw(CPAN::InfoObj);
 
 #-> sub CPAN::Distribution::called_for ;
 sub called_for {
@@ -2004,34 +2244,33 @@ sub get {
     my $packagedir;
 
     $self->debug("local_file[$local_file]") if $CPAN::DEBUG;
-    if ($CPAN::META->hasMD5) {
+    if ($CPAN::META->has_inst('MD5')) {
+       $self->debug("MD5 is installed, verifying");
        $self->verifyMD5;
+    } else {
+       $self->debug("MD5 is NOT installed");
+    }
+    $self->debug("Removing tmp") if $CPAN::DEBUG;
+    File::Path::rmtree("tmp");
+    mkdir "tmp", 0755 or Carp::croak "Couldn't mkdir tmp: $!";
+    chdir "tmp";
+    $self->debug("Changed directory to tmp") if $CPAN::DEBUG;
+    if ($local_file =~ /(\.tar\.(gz|Z)|\.tgz)$/i){
+       $self->untar_me($local_file);
+    } elsif ( $local_file =~ /\.zip$/i ) {
+       $self->unzip_me($local_file);
+    } elsif ( $local_file =~ /\.pm\.(gz|Z)$/) {
+       $self->pm2dir_me($local_file);
+    } else {
+       $self->{archived} = "NO";
     }
-    if ($local_file =~ /(\.tar\.(gz|Z)|\.tgz|\.zip)$/i){
-       $self->debug("Removing tmp") if $CPAN::DEBUG;
-       File::Path::rmtree("tmp");
-       mkdir "tmp", 0777 or Carp::croak "Couldn't mkdir tmp: $!";
+    chdir "..";
+    if ($self->{archived} ne 'NO') {
        chdir "tmp";
-       $self->debug("Changed directory to tmp") if $CPAN::DEBUG;
-       if ($local_file =~ /z$/i){
-           $self->{archived} = "tar";
-           if (system("$CPAN::Config->{gzip} --decompress --stdout $local_file | $CPAN::Config->{tar} xvf -")==0) {
-               $self->{unwrapped} = "YES";
-           } else {
-               $self->{unwrapped} = "NO";
-           }
-       } elsif ($local_file =~ /zip$/i) {
-           $self->{archived} = "zip";
-           if (system("$CPAN::Config->{unzip} $local_file")==0) {
-               $self->{unwrapped} = "YES";
-           } else {
-               $self->{unwrapped} = "NO";
-           }
-       }
        # Let's check if the package has its own directory.
-       opendir DIR, "." or Carp::croak("Weird: couldn't opendir .: $!");
-       my @readdir = grep $_ !~ /^\.\.?$/, readdir DIR; ### MAC??
-       closedir DIR;
+       my $dh = DirHandle->new(".") or Carp::croak("Couldn't opendir .: $!");
+       my @readdir = grep $_ !~ /^\.\.?$/, $dh->read; ### MAC??
+       $dh->close;
        my ($distdir,$packagedir);
        if (@readdir == 1 && -d $readdir[0]) {
            $distdir = $readdir[0];
@@ -2052,8 +2291,8 @@ sub get {
            }
        }
        $self->{'build_dir'} = $packagedir;
-
        chdir "..";
+
        $self->debug("Changed directory to .. (self is $self [".$self->as_string."])")
            if $CPAN::DEBUG;
        File::Path::rmtree("tmp");
@@ -2071,22 +2310,59 @@ sub get {
                my $fh = FileHandle->new(">$makefilepl")
                    or Carp::croak("Could not open >$makefilepl");
                my $cf = $self->called_for || "unknown";
-               $fh->print(qq{
-# This Makefile.PL has been autogenerated by the module CPAN.pm
+               $fh->print(
+qq{# This Makefile.PL has been autogenerated by the module CPAN.pm
+# because there was no Makefile.PL supplied.
 # Autogenerated on: }.scalar localtime().qq{
+
                    use ExtUtils::MakeMaker;
                    WriteMakefile(NAME => q[$cf]);
+
 });
                print qq{Package comes without Makefile.PL.\n}.
                    qq{  Writing one on our own (calling it $cf)\n};
            }
        }
-    } else {
-       $self->{archived} = "NO";
     }
     return $self;
 }
 
+sub untar_me {
+    my($self,$local_file) = @_;
+    $self->{archived} = "tar";
+    my $system = "$CPAN::Config->{gzip} --decompress --stdout " .
+       "$local_file | $CPAN::Config->{tar} xvf -";
+    if (system($system)== 0) {
+       $self->{unwrapped} = "YES";
+    } else {
+       $self->{unwrapped} = "NO";
+    }
+}
+
+sub unzip_me {
+    my($self,$local_file) = @_;
+    $self->{archived} = "zip";
+    my $system = "$CPAN::Config->{unzip} $local_file";
+    if (system($system) == 0) {
+       $self->{unwrapped} = "YES";
+    } else {
+       $self->{unwrapped} = "NO";
+    }
+}
+
+sub pm2dir_me {
+    my($self,$local_file) = @_;
+    $self->{archived} = "pm";
+    my $to = File::Basename::basename($local_file);
+    $to =~ s/\.(gz|Z)$//;
+    my $system = "$CPAN::Config->{gzip} --decompress --stdout $local_file > $to";
+    if (system($system) == 0) {
+       $self->{unwrapped} = "YES";
+    } else {
+       $self->{unwrapped} = "NO";
+    }
+}
+
 #-> sub CPAN::Distribution::new ;
 sub new {
     my($class,%att) = @_;
@@ -2114,10 +2390,12 @@ Please define it with "o conf shell <your shell>"
     my $dist = $self->id;
     my $dir  = $self->dir or $self->get;
     $dir = $self->dir;
-    my $pwd  = Cwd::cwd();
+    my $getcwd;
+    $getcwd = $CPAN::Config->{'getcwd'} || 'cwd';
+    my $pwd  = CPAN->$getcwd();
     chdir($dir);
     print qq{Working directory is $dir.\n};
-    system($CPAN::Config->{'shell'})==0 or die "Subprocess shell error";
+    system($CPAN::Config->{'shell'}) == 0 or die "Subprocess shell error";
     chdir($pwd);
 }
 
@@ -2154,111 +2432,105 @@ sub verifyMD5 {
        $self->{MD5_STATUS} eq "OK" and push @e, "MD5 Checksum was ok";
        print join "", map {"  $_\n"} @e and return if @e;
     }
-    my($local_file);
-    my(@local) = split("/",$self->{ID});
-    my($basename) = pop @local;
+    my($lc_want,$lc_file,@local,$basename);
+    @local = split("/",$self->{ID});
+    pop @local;
     push @local, "CHECKSUMS";
-    my($local_wanted) =
-       CPAN->catfile(
-                     $CPAN::Config->{keep_source_where},
-                     "authors",
-                     "id",
-                     @local
-                    );
+    $lc_want =
+       CPAN->catfile($CPAN::Config->{keep_source_where},
+                     "authors", "id", @local);
     local($") = "/";
     if (
-       -f $local_wanted
+       -f $lc_want
        &&
-       $self->MD5_check_file($local_wanted,$basename)
+       $self->MD5_check_file($lc_want)
        ) {
        return $self->{MD5_STATUS} = "OK";
     }
-    $local_file = CPAN::FTP->localize(
-                                     "authors/id/@local",
-                                     $local_wanted,
-                                     'force>:-{');
-    my($checksum_pipe);
-    if ($local_file) {
-       # fine
-    } else {
+    $lc_file = CPAN::FTP->localize("authors/id/@local",
+                                  $lc_want,'force>:-{');
+    unless ($lc_file) {
        $local[-1] .= ".gz";
-       $local_file = CPAN::FTP->localize(
-                                         "authors/id/@local",
-                                         "$local_wanted.gz",
-                                         'force>:-{'
-                                        );
-       my $system = "$CPAN::Config->{gzip} --decompress $local_file";
-       system($system)==0 or die "Could not uncompress $local_file";
-       $local_file =~ s/\.gz$//;
+       $lc_file = CPAN::FTP->localize("authors/id/@local",
+                                      "$lc_want.gz",'force>:-{');
+       my @system = ($CPAN::Config->{gzip}, '--decompress', $lc_file);
+       system(@system) == 0 or die "Could not uncompress $lc_file";
+       $lc_file =~ s/\.gz$//;
     }
-    $self->MD5_check_file($local_file,$basename);
+    $self->MD5_check_file($lc_file);
 }
 
 #-> sub CPAN::Distribution::MD5_check_file ;
 sub MD5_check_file {
-    my($self,$lfile,$basename) = @_;
-    my($cksum);
-    my $fh = new FileHandle;
-    local($/)=undef;
-    if (open $fh, $lfile){
+    my($self,$chk_file) = @_;
+    my($cksum,$file,$basename);
+    $file =  $self->{localfile};
+    $basename = File::Basename::basename($file);
+    my $fh = FileHandle->new;
+    local($/);
+    if (open $fh, $chk_file){
        my $eval = <$fh>;
        close $fh;
        my($comp) = Safe->new();
        $cksum = $comp->reval($eval);
-       Carp::confess($@) if $@;
-       if ($cksum->{$basename}->{md5}) {
-           $self->debug("Found checksum for $basename: $cksum->{$basename}->{md5}\n")
-               if $CPAN::DEBUG;
-           my $file = $self->{localfile};
-           my $pipe = "$CPAN::Config->{gzip} --decompress --stdout $self->{localfile}|";
-           if (
-               open($fh, $file) && $self->eq_MD5($fh,$cksum->{$basename}->{md5})
-               or
-               open($fh, $pipe) && $self->eq_MD5($fh,$cksum->{$basename}->{'md5-ungz'})
-              ){
-               print "Checksum for $file ok\n";
-               return $self->{MD5_STATUS} = "OK";
-           } else {
-               print join(
-                          "",
-                          qq{Checksum mismatch for distribution file. },
-                          qq{Please investigate.\n\n}
-                         );
-               print $self->as_string;
-               print $CPAN::META->instance(
-                                           'CPAN::Author',
-                                           $self->{CPAN_USERID}
-                                          )->as_string;
-               my $wrap = qq{I\'d recommend removing $self->{'localfile'}}.
-                   qq{, put another URL at the top of the list of URLs to }.
-                   qq{visit, and restart CPAN.pm. If all this doesn\'t help, }.
-                   qq{please contact the author or your CPAN site admin};
-               print Text::Wrap::wrap("","",$wrap);
-               print "\n\n";
-               sleep 3;
-               return;
-           }
-           close $fh if fileno($fh);
+       if ($@) {
+           rename $chk_file, "$chk_file.bad";
+           Carp::confess($@) if $@;
+       }
+    } else {
+       Carp::carp "Could not open $chk_file for reading";
+    }
+    if ($cksum->{$basename}->{md5}) {
+       $self->debug("Found checksum for $basename:" .
+                    "$cksum->{$basename}->{md5}\n") if $CPAN::DEBUG;
+       my $pipe = "$CPAN::Config->{gzip} --decompress ".
+           "--stdout $file|";
+       if (
+           open($fh, $file) &&
+           binmode $fh &&
+           $self->eq_MD5($fh,$cksum->{$basename}->{md5})
+           or
+           open($fh, $pipe) &&
+           binmode $fh  &&
+           $self->eq_MD5($fh,$cksum->{$basename}->{'md5-ungz'})
+          ){
+           print "Checksum for $file ok\n";
+           return $self->{MD5_STATUS} = "OK";
        } else {
-           $self->{MD5_STATUS} ||= "";
-           if ($self->{MD5_STATUS} eq "NIL") {
-               print "\nNo md5 checksum for $basename in local $lfile.";
-               print "Removing $lfile\n";
-               unlink $lfile or print "Could not unlink: $!";
-               sleep 1;
-           }
-           $self->{MD5_STATUS} = "NIL";
+           print qq{Checksum mismatch for distribution file. }.
+               qq{Please investigate.\n\n};
+           print $self->as_string;
+           print $CPAN::META->instance(
+                                       'CPAN::Author',
+                                       $self->{CPAN_USERID}
+                                      )->as_string;
+           my $wrap = qq{I\'d recommend removing $file. It seems to
+be a bogus file.  Maybe you have configured your \`urllist\' with a
+bad URL.  Please check this array with \`o conf urllist\', and
+retry.};
+           print Text::Wrap::wrap("","",$wrap);
+           print "\n\n";
+           sleep 3;
            return;
        }
+       close $fh if fileno($fh);
     } else {
-       Carp::carp "Could not open $lfile for reading";
+       $self->{MD5_STATUS} ||= "";
+       if ($self->{MD5_STATUS} eq "NIL") {
+           print "\nNo md5 checksum for $basename in local $chk_file.";
+           print "Removing $chk_file\n";
+           unlink $chk_file or print "Could not unlink: $!";
+           sleep 1;
+       }
+       $self->{MD5_STATUS} = "NIL";
+       return;
     }
 }
 
 #-> sub CPAN::Distribution::eq_MD5 ;
 sub eq_MD5 {
     my($self,$fh,$expectMD5) = @_;
-    my $md5 = new MD5;
+    my $md5 = MD5->new;
     $md5->addfile($fh);
     my $hexdigest = $md5->hexdigest;
     $hexdigest eq $expectMD5;
@@ -2278,6 +2550,30 @@ sub force {
     delete $self->{'writemakefile'};
 }
 
+#-> sub CPAN::Distribution::perl ;
+sub perl {
+    my($self) = @_;
+    my($perl) = MM->file_name_is_absolute($^X) ? $^X : "";
+    my $getcwd = $CPAN::Config->{'getcwd'} || 'cwd';
+    my $pwd  = CPAN->$getcwd();
+    my $candidate = $CPAN::META->catfile($pwd,$^X);
+    $perl ||= $candidate if MM->maybe_command($candidate);
+    unless ($perl) {
+       my ($component,$perl_name);
+      DIST_PERLNAME: foreach $perl_name ($^X, 'perl', 'perl5', "perl$]") {
+           PATH_COMPONENT: foreach $component (MM->path(), $Config::Config{'binexp'}) {
+                 next unless defined($component) && $component;
+                 my($abs) = MM->catfile($component,$perl_name);
+                 if (MM->maybe_command($abs)) {
+                     $perl = $abs;
+                     last DIST_PERLNAME;
+                 }
+             }
+         }
+    }
+    $perl;
+}
+
 #-> sub CPAN::Distribution::make ;
 sub make {
     my($self) = @_;
@@ -2289,7 +2585,7 @@ sub make {
        $self->{archived} eq "NO" and push @e,
        "Is neither a tar nor a zip archive.";
 
-       $self->{unwrapped} eq "NO"   and push @e,
+       $self->{unwrapped} eq "NO" and push @e,
        "had problems unarchiving. Please build manually";
 
        exists $self->{writemakefile} &&
@@ -2310,62 +2606,54 @@ sub make {
     if ($self->{'configure'}) {
        $system = $self->{'configure'};
     } else {
-       my($perl) = MM->file_name_is_absolute($^X) ? $^X : "";
-       $perl ||= "$CPAN::Cwd/$^X" if -x "$CPAN::Cwd/$^X";
-       unless ($perl) {
-           my ($component,$perl_name);
-           DIST_PERLNAME: foreach $perl_name ($^X, 'perl', 'perl5', "perl$]") {
-                 DIST_COMPONENT: foreach $component (MM->path(), $Config::Config{'binexp'}) {
-                       next unless defined($component) && $component;
-                       my($abs) = MM->catfile($component,$perl_name);
-                       if (MM->maybe_command($abs)) {
-                           $perl = $abs;
-                           last DIST_PERLNAME;
-                       }
+       my($perl) = $self->perl or die "Couldn\'t find executable perl\n";
+       my $switch = "";
+# This needs a handler that can be turned on or off:
+#      $switch = "-MExtUtils::MakeMaker ".
+#          "-Mops=:default,:filesys_read,:filesys_open,require,chdir"
+#          if $] > 5.00310;
+       $system = "$perl $switch Makefile.PL $CPAN::Config->{makepl_arg}";
+    }
+    {
+       local($SIG{ALRM}) = sub { die "inactivity_timeout reached\n" };
+       my($ret,$pid);
+       $@ = "";
+       if ($CPAN::Config->{inactivity_timeout}) {
+           eval {
+               alarm $CPAN::Config->{inactivity_timeout};
+               local $SIG{CHLD} = sub { wait };
+               if (defined($pid = fork)) {
+                   if ($pid) { #parent
+                       wait;
+                   } else {    #child
+                       exec $system;
                    }
+               } else {
+                   print "Cannot fork: $!";
+                   return;
                }
-       }
-       die "Couldn\'t find executable perl\n" unless $perl;
-       $system = "$perl Makefile.PL $CPAN::Config->{makepl_arg}";
-   }
-    $SIG{ALRM} = sub { die "inactivity_timeout reached\n" };
-    my($ret,$pid);
-    $@ = "";
-    if ($CPAN::Config->{inactivity_timeout}) {
-       eval {
-           alarm $CPAN::Config->{inactivity_timeout};
-           #$SIG{CHLD} = \&REAPER;
-           if (defined($pid=fork)) {
-               if ($pid) { #parent
-                   wait;
-               } else {    #child
-                   exec $system;
-               }
-           } else {
-               print "Cannot fork: $!";
+           };
+           alarm 0;
+           if ($@){
+               kill 9, $pid;
+               waitpid $pid, 0;
+               print $@;
+               $self->{writemakefile} = "NO - $@";
+               $@ = "";
                return;
            }
+       } else {
            $ret = system($system);
-       };
-       alarm 0;
-    } else {
-       $ret = system($system);
-    }
-    if ($@){
-       kill 9, $pid;
-       waitpid $pid, 0;
-       print $@;
-       $self->{writemakefile} = "NO - $@";
-       $@ = "";
-       return;
-    } elsif ($ret != 0) {
-        $self->{writemakefile} = "NO";
-        return;
+           if ($ret != 0) {
+               $self->{writemakefile} = "NO";
+               return;
+           }
+       }
     }
     $self->{writemakefile} = "YES";
     return if $CPAN::Signal;
     $system = join " ", $CPAN::Config->{'make'}, $CPAN::Config->{make_arg};
-    if (system($system)==0) {
+    if (system($system) == 0) {
         print "  $system -- OK\n";
         $self->{'make'} = "YES";
     } else {
@@ -2396,7 +2684,7 @@ sub test {
     chdir $self->{'build_dir'} or Carp::croak("Couldn't chdir to $self->{'build_dir'}");
     $self->debug("Changed directory to $self->{'build_dir'}") if $CPAN::DEBUG;
     my $system = join " ", $CPAN::Config->{'make'}, "test";
-    if (system($system)==0) {
+    if (system($system) == 0) {
         print "  $system -- OK\n";
         $self->{'make_test'} = "YES";
     } else {
@@ -2417,7 +2705,7 @@ sub clean {
     chdir $self->{'build_dir'} or Carp::croak("Couldn't chdir to $self->{'build_dir'}");
     $self->debug("Changed directory to $self->{'build_dir'}") if $CPAN::DEBUG;
     my $system = join " ", $CPAN::Config->{'make'}, "clean";
-    if (system($system)==0) {
+    if (system($system) == 0) {
        print "  $system -- OK\n";
        $self->force;
     } else {
@@ -2442,6 +2730,11 @@ sub install {
            $self->{'make'} eq 'NO' and
                push @e, "Oops, make had returned bad status";
 
+       push @e, "make test had returned bad status, won't install without force"
+           if exists $self->{'make_test'} and
+           $self->{'make_test'} eq 'NO' and
+           ! $self->{'force_update'};
+
        exists $self->{'install'} and push @e,
        $self->{'install'} eq "YES" ?
            "Already done" : "Already tried without success";
@@ -2476,7 +2769,6 @@ sub dir {
 }
 
 package CPAN::Bundle;
-@CPAN::Bundle::ISA = qw(CPAN::Module);
 
 #-> sub CPAN::Bundle::as_string ;
 sub as_string {
@@ -2494,23 +2786,23 @@ sub contains {
        # Try to get at it in the cpan directory
        $self->debug("no parsefile") if $CPAN::DEBUG;
        my $dist = $CPAN::META->instance('CPAN::Distribution',$self->{'CPAN_FILE'});
-       $self->debug($dist->as_string) if $CPAN::DEBUG;
        $dist->get;
        $self->debug($dist->as_string) if $CPAN::DEBUG;
        my($todir) = $CPAN::META->catdir($CPAN::Config->{'cpan_home'},"Bundle");
        File::Path::mkpath($todir);
        my($me,$from,$to);
        ($me = $self->id) =~ s/.*://;
-       $from = $CPAN::META->catfile($dist->{'build_dir'},"$me.pm");
+       $from = $self->find_bundle_file($dist->{'build_dir'},"$me.pm");
        $to = $CPAN::META->catfile($todir,"$me.pm");
        File::Copy::copy($from, $to) or Carp::confess("Couldn't copy $from to $to: $!");
        $parsefile = $to;
     }
     my @result;
-    my $fh = new FileHandle;
+    my $fh = FileHandle->new;
     local $/ = "\n";
     open($fh,$parsefile) or die "Could not open '$parsefile': $!";
     my $inpod = 0;
+    $self->debug("parsefile[$parsefile]") if $CPAN::DEBUG;
     while (<$fh>) {
        $inpod = /^=(?!head1\s+CONTENTS)/ ? 0 : /^=head1\s+CONTENTS/ ? 1 : $inpod;
        next unless $inpod;
@@ -2521,10 +2813,38 @@ sub contains {
     }
     close $fh;
     delete $self->{STATUS};
-    $self->{CONTAINS} = [@result];
+    $self->{CONTAINS} = join ", ", @result;
+    $self->debug("CONTAINS[@result]") if $CPAN::DEBUG;
     @result;
 }
 
+#-> sub CPAN::Bundle::find_bundle_file
+sub find_bundle_file {
+    my($self,$where,$what) = @_;
+    my $bu = $CPAN::META->catfile($where,$what);
+    return $bu if -f $bu;
+    my $manifest = $CPAN::META->catfile($where,"MANIFEST");
+    unless (-f $manifest) {
+       require ExtUtils::Manifest;
+       my $getcwd = $CPAN::Config->{'getcwd'} || 'cwd';
+       my $cwd = CPAN->$getcwd();
+       chdir $where;
+       ExtUtils::Manifest::mkmanifest();
+       chdir $cwd;
+    }
+    my $fh = FileHandle->new($manifest) or Carp::croak("Couldn't open $manifest: $!");
+    local($/) = "\n";
+    while (<$fh>) {
+       next if /^\s*\#/;
+       my($file) = /(\S+)/;
+       if ($file =~ m|Bundle/$what$|) {
+           $bu = $file;
+           return $CPAN::META->catfile($where,$bu);
+       }
+    }
+    Carp::croak("Could't find a Bundle file in $where");
+}
+
 #-> sub CPAN::Bundle::inst_file ;
 sub inst_file {
     my($self) = @_;
@@ -2532,9 +2852,10 @@ sub inst_file {
     ($me = $self->id) =~ s/.*://;
     $inst_file = $CPAN::META->catfile($CPAN::Config->{'cpan_home'},"Bundle", "$me.pm");
     return $self->{'INST_FILE'} = $inst_file if -f $inst_file;
-    $inst_file = $self->SUPER::inst_file;
-    return $self->{'INST_FILE'} = $inst_file if -f $inst_file;
-    return $self->{'INST_FILE'}; # even if undefined?
+#    $inst_file =
+    $self->SUPER::inst_file;
+#    return $self->{'INST_FILE'} = $inst_file if -f $inst_file;
+#    return $self->{'INST_FILE'}; # even if undefined?
 }
 
 #-> sub CPAN::Bundle::rematein ;
@@ -2556,6 +2877,13 @@ explicitly a file $s.
     }
 }
 
+#sub CPAN::Bundle::xs_file
+sub xs_file {
+    # If a bundle contains another that contains an xs_file we have
+    # here, we just don't bother I suppose
+    return 0;
+}
+
 #-> sub CPAN::Bundle::force ;
 sub force   { shift->rematein('force',@_); }
 #-> sub CPAN::Bundle::get ;
@@ -2578,7 +2906,6 @@ sub readme  {
 }
 
 package CPAN::Module;
-@CPAN::Module::ISA = qw(CPAN::InfoObj);
 
 #-> sub CPAN::Module::as_glimpse ;
 sub as_glimpse {
@@ -2608,7 +2935,7 @@ sub as_string {
                         $sprintf2,
                         'CPAN_USERID',
                         $userid,
-                        $CPAN::META->instance(CPAN::Author,$userid)->fullname
+                        CPAN::Shell->expand('Author',$userid)->fullname
                        )
     }
     push @m, sprintf $sprintf, 'CPAN_VERSION', $self->{CPAN_VERSION} if $self->{CPAN_VERSION};
@@ -2652,7 +2979,10 @@ sub as_string {
        close $fh;
        $self->{MANPAGE} = join " ", @result;
     }
-    push @m, sprintf $sprintf, 'MANPAGE', $self->{MANPAGE} if $self->{MANPAGE};
+    my($item);
+    for $item (qw/MANPAGE CONTAINS/) {
+       push @m, sprintf $sprintf, $item, $self->{$item} if exists $self->{$item};
+    }
     push @m, sprintf $sprintf, 'INST_FILE', $local_file || "(not installed)";
     push @m, sprintf $sprintf, 'INST_VERSION', $self->inst_version if $local_file;
     join "", @m, "\n";
@@ -2720,10 +3050,14 @@ sub install {
     if (defined $inst_file) {
        $have = $self->inst_version;
     }
-    if ($inst_file && $have >= $latest && not exists $self->{'force_update'}) {
-       print $self->id, " is up to date.\n";
-    } else {
-       $doit = 1;
+    if (1){ # A block for scoping $^W, the if is just for the visual
+            # appeal
+       local($^W)=0;
+       if ($inst_file && $have >= $latest && not exists $self->{'force_update'}) {
+           print $self->id, " is up to date.\n";
+       } else {
+           $doit = 1;
+       }
     }
     $self->rematein('install') if $doit;
 }
@@ -2742,6 +3076,7 @@ sub inst_file {
            return $pmfile;
        }
     }
+    return;
 }
 
 #-> sub CPAN::Module::xs_file ;
@@ -2757,6 +3092,7 @@ sub xs_file {
            return $xsfile;
        }
     }
+    return;
 }
 
 #-> sub CPAN::Module::inst_version ;
@@ -2771,8 +3107,12 @@ sub inst_version {
     $have;
 }
 
+package CPAN;
+
 1;
 
+__END__
+
 =head1 NAME
 
 CPAN - query, download and build perl modules from CPAN sites
@@ -2849,13 +3189,13 @@ item is displayed. If the search finds one item, we display the result
 of object-E<gt>as_string, but if we find more than one, we display
 each as object-E<gt>as_glimpse. E.g.
 
-    cpan> a ANDK     
+    cpan> a ANDK
     Author id = ANDK
        EMAIL        a.koenig@franz.ww.TU-Berlin.DE
        FULLNAME     Andreas König
 
 
-    cpan> a /andk/   
+    cpan> a /andk/
     Author id = ANDK
        EMAIL        a.koenig@franz.ww.TU-Berlin.DE
        FULLNAME     Andreas König
@@ -2873,11 +3213,11 @@ be. Is it a distribution file (recognized by embedded slashes), this
 file is being processed. Is it a module, CPAN determines the
 distribution file where this module is included and processes that.
 
-Any C<make>, C<test>, and C<readme> are run unconditionally. A 
+Any C<make>, C<test>, and C<readme> are run unconditionally. A
 
   install <distribution_file>
 
-also is run unconditionally.  But for 
+also is run unconditionally.  But for
 
   install <module>
 
@@ -2947,13 +3287,91 @@ perl breaks binary compatibility. If one of the modules that CPAN uses
 is in turn depending on binary compatibility (so you cannot run CPAN
 commands), then you should try the CPAN::Nox module for recovery.
 
+=head2 The 4 C<CPAN::*> Classes: Author, Bundle, Module, Distribution
+
+Although it may be considered internal, the class hierarchie does
+matter for both users and programmer. CPAN.pm deals with above
+mentioned four classes, and all those classes share a set of
+methods. It is a classical single polymorphism that is in effect.  A
+metaclass object registers all objects of all kinds and indexes them
+with a string. The strings referencing objects have a separated
+namespace (well, not completely separated):
+
+         Namespace                         Class
+
+   words containing a "/" (slash)      Distribution
+    words starting with Bundle::          Bundle
+          everything else            Module or Author
+
+Modules know their associated Distribution objects. They always refer
+to the most recent official release. Developers may mark their
+releases as unstable development versions (by inserting an underbar
+into the visible version number), so not always is the default
+distribution for a given module the really hottest and newest. If a
+module Foo circulates on CPAN in both version 1.23 and 1.23_90,
+CPAN.pm offers a convenient way to install version 1.23 by saying
+
+    install Foo
+
+This would install the complete distribution file (say
+BAR/Foo-1.23.tar.gz) with all accompanying material in there. But if
+you would like to install version 1.23_90, you need to know where the
+distribution file resides on CPAN relative to the authors/id/
+directory. If the author is BAR, this might be BAR/Foo-1.23_90.tar.gz,
+so he would have to say
+
+    install BAR/Foo-1.23_90.tar.gz
+
+The first example will be driven by an object of the class
+CPAN::Module, the second by an object of class Distribution.
+
 =head2 ProgrammerE<39>s interface
 
 If you do not enter the shell, the available shell commands are both
 available as methods (C<CPAN::Shell-E<gt>install(...)>) and as
-functions in the calling package (C<install(...)>). The
-programmerE<39>s interface has beta status. Do not heavily rely on it,
-changes may still be necessary.
+functions in the calling package (C<install(...)>).
+
+There's currently only one class that has a stable interface,
+CPAN::Shell. All commands that are available in the CPAN shell are
+methods of the class CPAN::Shell. Each of the commands that produce
+listings of modules (C<r>, C<autobundle>, C<u>) returns a list of the
+IDs of all modules within the list.
+
+=over 2
+
+=item expand($type,@things)
+
+The IDs of all objects available within a program are strings that can
+be expanded to the corresponding real objects with the
+C<CPAN::Shell-E<gt>expand("Module",@things)> method. Expand returns a
+list of CPAN::Module objects according to the C<@things> arguments
+given. In scalar context it only returns the first element of the
+list.
+
+=item Programming Examples
+
+This enables the programmer to do operations that combine
+functionalities that are available in the shell.
+
+    # install everything that is outdated on my disk:
+    perl -MCPAN -e 'CPAN::Shell->install(CPAN::Shell->r)'
+
+    # install my favorite programs if necessary:
+    for $mod (qw(Net::FTP MD5 Data::Dumper)){
+        my $obj = CPAN::Shell->expand('Module',$mod);
+        $obj->install;
+    }
+
+    # list all modules on my disk that have no VERSION number
+    for $mod (CPAN::Shell->expand("Module","/./")){
+       next unless $mod->inst_file;
+       next if $mod->inst_version;
+       print "No VERSION in ", $mod->id, "\n";
+    }
+
+=back
+
+=head2 Methods in the four
 
 =head2 Cache Manager