]> git.saurik.com Git - apt.git/commitdiff
Merge remote-tracking branch 'upstream/debian/experimental' into feature/acq-trans
authorMichael Vogt <mvo@debian.org>
Mon, 29 Sep 2014 07:58:38 +0000 (09:58 +0200)
committerMichael Vogt <mvo@debian.org>
Mon, 29 Sep 2014 07:58:38 +0000 (09:58 +0200)
Conflicts:
apt-pkg/acquire-item.cc
apt-pkg/acquire-item.h
methods/gpgv.cc

1  2 
apt-pkg/acquire-item.cc
apt-pkg/acquire-item.h
apt-pkg/acquire.cc
methods/copy.cc
methods/gpgv.cc

diff --combined apt-pkg/acquire-item.cc
index b8317b13de849f048b65d25c48a4e79865068858,f46c8a6e46f9f2c8534d32f7269a52a5fe39c218..923a153a785ff762974f1bb3c7006048f75f32c0
@@@ -64,18 -64,21 +64,25 @@@ static void printHashSumComparision(std
                                                                        /*}}}*/
  
  // Acquire::Item::Item - Constructor                                  /*{{{*/
 -pkgAcquire::Item::Item(pkgAcquire *Owner, HashStringList const &ExpectedHashes) :
 -   Owner(Owner), FileSize(0), PartialSize(0), Mode(0), ID(0), Complete(false),
 -   Local(false), QueueCounter(0), ExpectedAdditionalItems(0),
 -   ExpectedHashes(ExpectedHashes)
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
 +pkgAcquire::Item::Item(pkgAcquire *Owner,
 +                       HashStringList const &ExpectedHashes,
 +                       pkgAcqMetaBase *TransactionManager)
 +   : Owner(Owner), FileSize(0), PartialSize(0), Mode(0), ID(0), Complete(false),
 +     Local(false), QueueCounter(0), TransactionManager(TransactionManager),
 +     ExpectedAdditionalItems(0), ExpectedHashes(ExpectedHashes)
  {
     Owner->Add(this);
     Status = StatIdle;
 +   if(TransactionManager != NULL)
 +      TransactionManager->Add(this);
  }
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
                                                                        /*}}}*/
  // Acquire::Item::~Item - Destructor                                  /*{{{*/
  // ---------------------------------------------------------------------
@@@ -92,8 -95,7 +99,8 @@@ pkgAcquire::Item::~Item(
  void pkgAcquire::Item::Failed(string Message,pkgAcquire::MethodConfig *Cnf)
  {
     Status = StatIdle;
 -   ErrorText = LookupTag(Message,"Message");
 +   if(ErrorText == "")
 +      ErrorText = LookupTag(Message,"Message");
     UsedMirror =  LookupTag(Message,"UsedMirror");
     if (QueueCounter <= 1)
     {
@@@ -139,7 -141,7 +146,7 @@@ void pkgAcquire::Item::Done(string Mess
  {
     // We just downloaded something..
     string FileName = LookupTag(Message,"Filename");
 -   UsedMirror =  LookupTag(Message,"UsedMirror");
 +   UsedMirror = LookupTag(Message,"UsedMirror");
     if (Complete == false && !Local && FileName == DestFile)
     {
        if (Owner->Log != 0)
  // ---------------------------------------------------------------------
  /* This helper function is used by a lot of item methods as their final
     step */
 -void pkgAcquire::Item::Rename(string From,string To)
 +bool pkgAcquire::Item::Rename(string From,string To)
  {
     if (rename(From.c_str(),To.c_str()) != 0)
     {
        snprintf(S,sizeof(S),_("rename failed, %s (%s -> %s)."),strerror(errno),
              From.c_str(),To.c_str());
        Status = StatError;
 -      ErrorText = S;
 +      ErrorText += S;
 +      return false;
     }   
 +   return true;
  }
                                                                        /*}}}*/
  bool pkgAcquire::Item::RenameOnError(pkgAcquire::Item::RenameOnErrorState const error)/*{{{*/
         Status = StatError;
         // do not report as usually its not the mirrors fault, but Portal/Proxy
         break;
 +      case SignatureError:
 +       ErrorText = _("Signature error");
 +       Status = StatError;
 +       break;
 +      case NotClearsigned:
 +         ErrorText = _("Does not start with a cleartext signature");
 +       Status = StatError;
 +       break;
     }
     return false;
  }
@@@ -249,6 -241,118 +256,6 @@@ void pkgAcquire::Item::ReportMirrorFail
     }
  }
                                                                        /*}}}*/
 -// AcqSubIndex::AcqSubIndex - Constructor                             /*{{{*/
 -// ---------------------------------------------------------------------
 -/* Get a sub-index file based on checksums from a 'master' file and
 -   possibly query additional files */
 -pkgAcqSubIndex::pkgAcqSubIndex(pkgAcquire *Owner, string const &URI,
 -                               string const &URIDesc, string const &ShortDesc,
 -                               HashStringList const &ExpectedHashes)
 -   : Item(Owner, ExpectedHashes)
 -{
 -   /* XXX: Beware: Currently this class does nothing (of value) anymore ! */
 -   Debug = _config->FindB("Debug::pkgAcquire::SubIndex",false);
 -
 -   DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 -
 -   Desc.URI = URI;
 -   Desc.Description = URIDesc;
 -   Desc.Owner = this;
 -   Desc.ShortDesc = ShortDesc;
 -
 -   QueueURI(Desc);
 -
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex: " << Desc.URI << std::endl;
 -}
 -                                                                      /*}}}*/
 -// AcqSubIndex::Custom600Headers - Insert custom request headers      /*{{{*/
 -// ---------------------------------------------------------------------
 -/* The only header we use is the last-modified header. */
 -string pkgAcqSubIndex::Custom600Headers() const
 -{
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(Desc.URI);
 -
 -   struct stat Buf;
 -   if (stat(Final.c_str(),&Buf) != 0)
 -      return "\nIndex-File: true\nFail-Ignore: true\n";
 -   return "\nIndex-File: true\nFail-Ignore: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
 -}
 -                                                                      /*}}}*/
 -void pkgAcqSubIndex::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)/*{{{*/
 -{
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex failed: " << Desc.URI << " with " << Message << std::endl;
 -
 -   Complete = false;
 -   Status = StatDone;
 -   Dequeue();
 -
 -   // No good Index is provided
 -}
 -                                                                      /*}}}*/
 -void pkgAcqSubIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,        /*{{{*/
 -                         pkgAcquire::MethodConfig *Cnf)
 -{
 -   if(Debug)
 -      std::clog << "pkgAcqSubIndex::Done(): " << Desc.URI << std::endl;
 -
 -   string FileName = LookupTag(Message,"Filename");
 -   if (FileName.empty() == true)
 -   {
 -      Status = StatError;
 -      ErrorText = "Method gave a blank filename";
 -      return;
 -   }
 -
 -   if (FileName != DestFile)
 -   {
 -      Local = true;
 -      Desc.URI = "copy:" + FileName;
 -      QueueURI(Desc);
 -      return;
 -   }
 -
 -   Item::Done(Message, Size, Hashes, Cnf);
 -
 -   string FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(Desc.URI);
 -
 -   /* Downloaded invalid transindex => Error (LP: #346386) (Closes: #627642) */
 -   indexRecords SubIndexParser;
 -   if (FileExists(DestFile) == true && !SubIndexParser.Load(DestFile)) {
 -      Status = StatError;
 -      ErrorText = SubIndexParser.ErrorText;
 -      return;
 -   }
 -
 -   // success in downloading the index
 -   // rename the index
 -   if(Debug)
 -      std::clog << "Renaming: " << DestFile << " -> " << FinalFile << std::endl;
 -   Rename(DestFile,FinalFile);
 -   chmod(FinalFile.c_str(),0644);
 -   DestFile = FinalFile;
 -
 -   if(ParseIndex(DestFile) == false)
 -      return Failed("", NULL);
 -
 -   Complete = true;
 -   Status = StatDone;
 -   Dequeue();
 -   return;
 -}
 -                                                                      /*}}}*/
 -bool pkgAcqSubIndex::ParseIndex(string const &IndexFile)              /*{{{*/
 -{
 -   indexRecords SubIndexParser;
 -   if (FileExists(IndexFile) == false || SubIndexParser.Load(IndexFile) == false)
 -      return false;
 -   // so something with the downloaded index
 -   return true;
 -}
 -                                                                      /*}}}*/
  // AcqDiffIndex::AcqDiffIndex - Constructor                           /*{{{*/
  // ---------------------------------------------------------------------
  /* Get the DiffIndex file first and see if there are patches available
   * the original packages file
   */
  pkgAcqDiffIndex::pkgAcqDiffIndex(pkgAcquire *Owner,
 +                                 pkgAcqMetaBase *TransactionManager,
                                   IndexTarget const * const Target,
                                 HashStringList const &ExpectedHashes,
                                   indexRecords *MetaIndexParser)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser)
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, 
 +                     MetaIndexParser), PackagesFileReadyInPartial(false)
  {
     
     Debug = _config->FindB("Debug::pkgAcquire::Diffs",false);
@@@ -358,14 -460,9 +365,14 @@@ bool pkgAcqDiffIndex::ParseDiffIndex(st
         // we have the same sha1 as the server so we are done here
         if(Debug)
            std::clog << "Package file is up-to-date" << std::endl;
 +         // ensure we have no leftovers from previous runs
 +         std::string Partial = _config->FindDir("Dir::State::lists");
 +         Partial += "partial/" + URItoFileName(RealURI);
 +         unlink(Partial.c_str());
         // list cleanup needs to know that this file as well as the already
         // present index is ours, so we create an empty diff to save it for us
 -       new pkgAcqIndexDiffs(Owner, Target, ExpectedHashes, MetaIndexParser, 
 +       new pkgAcqIndexDiffs(Owner, TransactionManager, Target, 
 +                              ExpectedHashes, MetaIndexParser, 
                                ServerSha1, available_patches);
         return true;
        }
        // we have something, queue the next diff
        if(found)
        {
 +         // FIXME: make this use the method
 +         PackagesFileReadyInPartial = true;
 +         std::string Partial = _config->FindDir("Dir::State::lists");
 +         Partial += "partial/" + URItoFileName(RealURI);
 +
 +         FileFd From(CurrentPackagesFile, FileFd::ReadOnly);
 +         FileFd To(Partial, FileFd::WriteEmpty);
 +         if(CopyFile(From, To) == false)
 +            return _error->Errno("CopyFile", "failed to copy");
 +
 +         if(Debug)
 +            std::cerr << "Done copying " << CurrentPackagesFile
 +                      << " -> " << Partial
 +                      << std::endl;
 +
         // queue the diffs
         string::size_type const last_space = Description.rfind(" ");
         if(last_space != string::npos)
  
         if (pdiff_merge == false)
           {
 -          new pkgAcqIndexDiffs(Owner, Target, ExpectedHashes, MetaIndexParser,
 +          new pkgAcqIndexDiffs(Owner, TransactionManager, Target, ExpectedHashes, 
 +                                 MetaIndexParser,
                                   ServerSha1, available_patches);
           }
           else
         {
            std::vector<pkgAcqIndexMergeDiffs*> *diffs = new std::vector<pkgAcqIndexMergeDiffs*>(available_patches.size());
            for(size_t i = 0; i < available_patches.size(); ++i)
 -             (*diffs)[i] = new pkgAcqIndexMergeDiffs(Owner, Target,
 +             (*diffs)[i] = new pkgAcqIndexMergeDiffs(Owner,
 +                                                       TransactionManager,
 +                                                       Target,
                                                         ExpectedHashes,
                                                         MetaIndexParser,
                                                         available_patches[i],
                                                         diffs);
         }
  
 -       Complete = false;
 -       Status = StatDone;
 -       Dequeue();
 -       return true;
 +         Complete = false;
 +         Status = StatDone;
 +         Dequeue();
 +         return true;
        }
     }
     
@@@ -505,7 -584,7 +512,7 @@@ void pkgAcqDiffIndex::Failed(string Mes
        std::clog << "pkgAcqDiffIndex failed: " << Desc.URI << " with " << Message << std::endl
                << "Falling back to normal index file acquire" << std::endl;
  
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
  
     Complete = false;
     Status = StatDone;
@@@ -548,13 -627,12 +555,13 @@@ void pkgAcqDiffIndex::Done(string Messa
   * for each diff and the index
   */
  pkgAcqIndexDiffs::pkgAcqIndexDiffs(pkgAcquire *Owner,
 +                                   pkgAcqMetaBase *TransactionManager,
                                     struct IndexTarget const * const Target,
                                     HashStringList const &ExpectedHashes,
                                     indexRecords *MetaIndexParser,
                                   string ServerSha1,
                                   vector<DiffInfo> diffs)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser),
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser),
       available_patches(diffs), ServerSha1(ServerSha1)
  {
     
  
     if(available_patches.empty() == true)
     {
 -      // we are done (yeah!)
 +      // we are done (yeah!), check hashes against the final file
 +      DestFile = _config->FindDir("Dir::State::lists");
 +      DestFile += URItoFileName(Target->URI);
        Finish(true);
     }
     else
@@@ -588,22 -664,20 +595,22 @@@ void pkgAcqIndexDiffs::Failed(string Me
     if(Debug)
        std::clog << "pkgAcqIndexDiffs failed: " << Desc.URI << " with " << Message << std::endl
                << "Falling back to normal index file acquire" << std::endl;
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
     Finish();
  }
                                                                        /*}}}*/
  // Finish - helper that cleans the item out of the fetcher queue      /*{{{*/
  void pkgAcqIndexDiffs::Finish(bool allDone)
  {
 +   if(Debug)
 +      std::clog << "pkgAcqIndexDiffs::Finish(): " 
 +                << allDone << " "
 +                << Desc.URI << std::endl;
 +
     // we restore the original name, this is required, otherwise
     // the file will be cleaned
     if(allDone) 
     {
 -      DestFile = _config->FindDir("Dir::State::lists");
 -      DestFile += URItoFileName(RealURI);
 -
        if(HashSums().usable() && !HashSums().VerifyFile(DestFile))
        {
         RenameOnError(HashSumMismatch);
         return;
        }
  
 +      // queue for copy
 +      PartialFile = _config->FindDir("Dir::State::lists")+"partial/"+URItoFileName(RealURI);
 +
 +      DestFile = _config->FindDir("Dir::State::lists");
 +      DestFile += URItoFileName(RealURI);
 +
 +      // this happens if we have a up-to-date indexfile
 +      if(!FileExists(PartialFile))
 +         PartialFile = DestFile;
 +      
        // this is for the "real" finish
        Complete = true;
        Status = StatDone;
                                                                        /*}}}*/
  bool pkgAcqIndexDiffs::QueueNextDiff()                                        /*{{{*/
  {
 -
     // calc sha1 of the just patched file
     string FinalFile = _config->FindDir("Dir::State::lists");
 -   FinalFile += URItoFileName(RealURI);
 +   FinalFile += "partial/" + URItoFileName(RealURI);
 +
 +   if(!FileExists(FinalFile))
 +   {
 +      Failed("No FinalFile " + FinalFile + " available", NULL);
 +      return false;
 +   }
  
     FileFd fd(FinalFile, FileFd::ReadOnly);
     SHA1Summation SHA1;
        std::clog << "QueueNextDiff: " 
                << FinalFile << " (" << local_sha1 << ")"<<std::endl;
  
 +
     // final file reached before all patches are applied
     if(local_sha1 == ServerSha1)
     {
     // error checking and falling back if no patch was found
     if(available_patches.empty() == true)
     {
 -      Failed("", NULL);
 +      Failed("No patches available", NULL);
        return false;
     }
  
@@@ -707,7 -765,7 +714,7 @@@ void pkgAcqIndexDiffs::Done(string Mess
     Item::Done(Message, Size, Hashes, Cnf);
  
     string FinalFile;
 -   FinalFile = _config->FindDir("Dir::State::lists")+URItoFileName(RealURI);
 +   FinalFile = _config->FindDir("Dir::State::lists")+"partial/"+URItoFileName(RealURI);
  
     // success in downloading a diff, enter ApplyDiff state
     if(State == StateFetchDiff)
        Local = true;
        Desc.URI = "rred:" + FinalFile;
        QueueURI(Desc);
+       ActiveSubprocess = "rred";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
        Mode = "rred";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
        return;
     } 
  
  
        // see if there is more to download
        if(available_patches.empty() == false) {
 -       new pkgAcqIndexDiffs(Owner, Target,
 +       new pkgAcqIndexDiffs(Owner, TransactionManager, Target,
                              ExpectedHashes, MetaIndexParser,
                                ServerSha1, available_patches);
         return Finish();
        } else 
 +         // update
 +         DestFile = FinalFile;
         return Finish(true);
     }
  }
                                                                        /*}}}*/
  // AcqIndexMergeDiffs::AcqIndexMergeDiffs - Constructor                       /*{{{*/
  pkgAcqIndexMergeDiffs::pkgAcqIndexMergeDiffs(pkgAcquire *Owner,
 +                                             pkgAcqMetaBase *TransactionManager,
                                               struct IndexTarget const * const Target,
                                               HashStringList const &ExpectedHashes,
                                               indexRecords *MetaIndexParser,
                                               DiffInfo const &patch,
                                               std::vector<pkgAcqIndexMergeDiffs*> const * const allPatches)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHashes, MetaIndexParser),
 +  : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser),
       patch(patch), allPatches(allPatches), State(StateFetchDiff)
  {
  
@@@ -808,7 -871,7 +823,7 @@@ void pkgAcqIndexMergeDiffs::Failed(stri
     // first failure means we should fallback
     State = StateErrorDiff;
     std::clog << "Falling back to normal index file acquire" << std::endl;
 -   new pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser);
 +   new pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser);
  }
                                                                        /*}}}*/
  void pkgAcqIndexMergeDiffs::Done(string Message,unsigned long long Size,HashStringList const &Hashes, /*{{{*/
  
     Item::Done(Message,Size,Hashes,Cnf);
  
 -   string const FinalFile = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +   string const FinalFile = _config->FindDir("Dir::State::lists") + "partial/" + URItoFileName(RealURI);
  
     if (State == StateFetchDiff)
     {
        Local = true;
        Desc.URI = "rred:" + FinalFile;
        QueueURI(Desc);
+       ActiveSubprocess = "rred";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
        Mode = "rred";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
        return;
     }
     // success in download/apply all diffs, clean up
         return;
        }
  
 +
 +      std::string FinalFile = _config->FindDir("Dir::State::lists");
 +      FinalFile += URItoFileName(RealURI);
 +
        // move the result into place
        if(Debug)
 -       std::clog << "Moving patched file in place: " << std::endl
 +       std::clog << "Queue patched file in place: " << std::endl
                   << DestFile << " -> " << FinalFile << std::endl;
 -      Rename(DestFile, FinalFile);
 -      chmod(FinalFile.c_str(), 0644);
  
 -      // otherwise lists cleanup will eat the file
 +      // queue for copy by the transaction manager
 +      PartialFile = DestFile;
        DestFile = FinalFile;
  
        // ensure the ed's are gone regardless of list-cleanup
        for (std::vector<pkgAcqIndexMergeDiffs *>::const_iterator I = allPatches->begin();
            I != allPatches->end(); ++I)
        {
 -          std::string patch = FinalFile + ".ed." + (*I)->patch.file + ".gz";
 +            std::string PartialFile = _config->FindDir("Dir::State::lists");
 +            PartialFile += "partial/" + URItoFileName(RealURI);
 +          std::string patch = PartialFile + ".ed." + (*I)->patch.file + ".gz";
 +            std::cerr << patch << std::endl;
            unlink(patch.c_str());
        }
  
     instantiated to fetch the revision file */   
  pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner,
                         string URI,string URIDesc,string ShortDesc,
 -                       HashStringList const  &ExpectedHash, string comprExt)
 -   : pkgAcqBaseIndex(Owner, NULL, ExpectedHash, NULL), RealURI(URI)
 +                       HashStringList const  &ExpectedHash)
 +   : pkgAcqBaseIndex(Owner, 0, NULL, ExpectedHash, NULL), RealURI(URI)
  {
 -   if(comprExt.empty() == true)
 -   {
 -      // autoselect the compression method
 -      std::vector<std::string> types = APT::Configuration::getCompressionTypes();
 -      for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
 -       comprExt.append(*t).append(" ");
 -      if (comprExt.empty() == false)
 -       comprExt.erase(comprExt.end()-1);
 -   }
 -   CompressionExtension = comprExt;
 -
 +   AutoSelectCompression();
     Init(URI, URIDesc, ShortDesc);
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgIndex with TransactionManager "
 +                << TransactionManager << std::endl;
  }
 -pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner, IndexTarget const *Target,
 +                                                                      /*}}}*/
 +// AcqIndex::AcqIndex - Constructor                                   /*{{{*/
 +// ---------------------------------------------------------------------
 +pkgAcqIndex::pkgAcqIndex(pkgAcquire *Owner,
 +                         pkgAcqMetaBase *TransactionManager,
 +                         IndexTarget const *Target,
                         HashStringList const &ExpectedHash, 
                           indexRecords *MetaIndexParser)
 -   : pkgAcqBaseIndex(Owner, Target, ExpectedHash, MetaIndexParser)
 -     RealURI(Target->URI)
 +   : pkgAcqBaseIndex(Owner, TransactionManager, Target, ExpectedHash
 +                     MetaIndexParser), RealURI(Target->URI)
  {
     // autoselect the compression method
 +   AutoSelectCompression();
 +   Init(Target->URI, Target->Description, Target->ShortDesc);
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgIndex with TransactionManager "
 +                << TransactionManager << std::endl;
 +}
 +                                                                      /*}}}*/
 +// AcqIndex::AutoSelectCompression - Select compression                       /*{{{*/
 +// ---------------------------------------------------------------------
 +void pkgAcqIndex::AutoSelectCompression()
 +{
     std::vector<std::string> types = APT::Configuration::getCompressionTypes();
     CompressionExtension = "";
     if (ExpectedHashes.usable())
     }
     if (CompressionExtension.empty() == false)
        CompressionExtension.erase(CompressionExtension.end()-1);
 -
 -   Init(Target->URI, Target->Description, Target->ShortDesc);
  }
 -                                                                      /*}}}*/
  // AcqIndex::Init - defered Constructor                                       /*{{{*/
 -void pkgAcqIndex::Init(string const &URI, string const &URIDesc, string const &ShortDesc) {
 +// ---------------------------------------------------------------------
 +void pkgAcqIndex::Init(string const &URI, string const &URIDesc, 
 +                       string const &ShortDesc)
 +{
     Decompression = false;
     Erase = false;
  
     DestFile += URItoFileName(URI);
  
     std::string const comprExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   std::string MetaKey;
     if (comprExt == "uncompressed")
     {
        Desc.URI = URI;
     else
     {
        Desc.URI = URI + '.' + comprExt;
+       DestFile = DestFile + '.' + comprExt;
        if(Target)
           MetaKey = string(Target->MetaKey) + '.' + comprExt;
     }
@@@ -1025,9 -1081,14 +1049,9 @@@ void pkgAcqIndex::InitByHashIfNeeded(co
  /* The only header we use is the last-modified header. */
  string pkgAcqIndex::Custom600Headers() const
  {
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (_config->FindB("Acquire::GzipIndexes",false))
 -      Final += compExt;
 +   string Final = GetFinalFilename();
     
     string msg = "\nIndex-File: true";
 -
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) == 0)
        msg += "\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
     return msg;
  }
                                                                        /*}}}*/
 +// pkgAcqIndex::Failed - getting the indexfile failed                 /*{{{*/
 +// ---------------------------------------------------------------------
 +/* */
  void pkgAcqIndex::Failed(string Message,pkgAcquire::MethodConfig *Cnf)        /*{{{*/
  {
     size_t const nextExt = CompressionExtension.find(' ');
     }
  
     Item::Failed(Message,Cnf);
 +
 +   /// cancel the entire transaction
 +   TransactionManager->AbortTransaction();
  }
                                                                        /*}}}*/
  // pkgAcqIndex::GetFinalFilename - Return the full final file path      /*{{{*/
 -std::string pkgAcqIndex::GetFinalFilename(std::string const &URI,
 -                                          std::string const &compExt)
 +// ---------------------------------------------------------------------
 +/* */
 +std::string pkgAcqIndex::GetFinalFilename() const
  {
 +   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
     std::string FinalFile = _config->FindDir("Dir::State::lists");
 -   FinalFile += URItoFileName(URI);
 +   FinalFile += URItoFileName(RealURI);
-    if (_config->FindB("Acquire::GzipIndexes",false) && compExt == "gz")
-       FinalFile += ".gz";
+    if (_config->FindB("Acquire::GzipIndexes",false) == true)
+       FinalFile += '.' + compExt;
     return FinalFile;
  }
 -                                                                      /*}}}*/
 -// AcqIndex::ReverifyAfterIMS - Reverify index after an ims-hit               /*{{{*/
 -void pkgAcqIndex::ReverifyAfterIMS(std::string const &FileName)
 +                                                                       /*}}}*/
 +// AcqIndex::ReverifyAfterIMS - Reverify index after an ims-hit        /*{{{*/
 +// ---------------------------------------------------------------------
 +/* */
 +void pkgAcqIndex::ReverifyAfterIMS()
  {
     std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
-    if (_config->FindB("Acquire::GzipIndexes",false) && compExt == "gz")
-       DestFile += ".gz";
+    if (_config->FindB("Acquire::GzipIndexes",false) == true)
+       DestFile += compExt;
  
 -   string FinalFile = GetFinalFilename(RealURI, compExt);
 -   Rename(FinalFile, FileName);
 +   // copy FinalFile into partial/ so that we check the hash again
 +   string FinalFile = GetFinalFilename();
     Decompression = true;
 -   Desc.URI = "copy:" + FileName;
 +   Desc.URI = "copy:" + FinalFile;
     QueueURI(Desc);
  }
 -                                                                      /*}}}*/
 +                                                                       /*}}}*/
  // AcqIndex::Done - Finished a fetch                                  /*{{{*/
  // ---------------------------------------------------------------------
  /* This goes through a number of states.. On the initial fetch the
     method could possibly return an alternate filename which points
     to the uncompressed version of the file. If this is so the file
     is copied into the partial directory. In all other cases the file
 -   is decompressed with a gzip uri. */
 -void pkgAcqIndex::Done(string Message,unsigned long long Size,HashStringList const &Hashes,
 +   is decompressed with a compressed uri. */
 +void pkgAcqIndex::Done(string Message, unsigned long long Size,
 +                       HashStringList const &Hashes,
                       pkgAcquire::MethodConfig *Cfg)
  {
     Item::Done(Message,Size,Hashes,Cfg);
           Desc.URI = RealURI;
         RenameOnError(HashSumMismatch);
         printHashSumComparision(RealURI, ExpectedHashes, Hashes);
 +         Failed(Message, Cfg);
           return;
        }
  
         * have a Package field) (LP: #346386) (Closes: #627642) 
         */
        FileFd fd(DestFile, FileFd::ReadOnly, FileFd::Extension);
 -      // Only test for correctness if the file is not empty (empty is ok)
 +      // Only test for correctness if the content of the file is not empty
 +      // (empty is ok)
        if (fd.Size() > 0)
        {
           pkgTagSection sec;
           if (_error->PendingError() == true || tag.Step(sec) == false || sec.Exists("Package") == false)
           {
              RenameOnError(InvalidFormat);
 +            Failed(Message, Cfg);
              return;
           }
        }
         
 -      // Done, move it into position
 -      string FinalFile = GetFinalFilename(RealURI, compExt);
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
 -
 -      /* We restore the original name to DestFile so that the clean operation
 -         will work OK */
 -      DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -      DestFile += URItoFileName(RealURI);
 -      if (_config->FindB("Acquire::GzipIndexes",false))
 -         DestFile += '.' + compExt;
 +      // FIXME: can we void the "Erase" bool here as its very non-local?
 +      std::string CompressedFile = _config->FindDir("Dir::State::lists") + "partial/";
 +      CompressedFile += URItoFileName(RealURI);
  
        // Remove the compressed version.
        if (Erase == true)
 -       unlink(DestFile.c_str());
 +       unlink(CompressedFile.c_str());
 +
 +      // Done, queue for rename on transaction finished
 +      PartialFile = DestFile;
 +      DestFile = GetFinalFilename();
  
        return;
     }
 +   
 +   // FIXME: use the same method to find 
 +   // check the compressed hash too
 +   if(MetaKey != "" && Hashes.size() > 0)
 +   {
 +      indexRecords::checkSum *Record = MetaIndexParser->Lookup(MetaKey);
 +      if(Record && Record->Hashes.usable() && Hashes != Record->Hashes)
 +      {
 +         RenameOnError(HashSumMismatch);
 +         printHashSumComparision(RealURI, Record->Hashes, Hashes);
 +         Failed(Message, Cfg);
 +         return;
 +      }
 +   }
  
     Erase = false;
     Complete = true;
        DestFile += ".decomp";
        Desc.URI = "copy:" + FileName;
        QueueURI(Desc);
+       ActiveSubprocess = "copy";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
        Mode = "copy";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
        return;
     }
  
         StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
        return;
  
 -   // The files timestamp matches, for non-local URLs reverify the local
 -   // file, for local file, uncompress again to ensure the hashsum is still
 -   // matching the Release file
 -   if (!Local && StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 +   // The files timestamp matches, reverify by copy into partial/
 +   if (StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
     {
 +      Erase = false;
 +      ReverifyAfterIMS();
++#if 0 // ???
+       // set destfile to the final destfile
+       if(_config->FindB("Acquire::GzipIndexes",false) == false)
+       {
+          DestFile = _config->FindDir("Dir::State::lists") + "partial/";
+          DestFile += URItoFileName(RealURI);
+       }
+       ReverifyAfterIMS(FileName);
++#endif
        return;
     }
     string decompProg;
  
     // If we enable compressed indexes, queue for hash verification
-    if (_config->FindB("Acquire::GzipIndexes",false) && compExt == "gz")
+    if (_config->FindB("Acquire::GzipIndexes",false))
     {
        DestFile = _config->FindDir("Dir::State::lists");
-       DestFile += URItoFileName(RealURI) + ".gz";
+       DestFile += URItoFileName(RealURI) + '.' + compExt;
  
        Decompression = true;
        Desc.URI = "copy:" + FileName;
     Desc.URI = decompProg + ":" + FileName;
     QueueURI(Desc);
  
-    // FIXME: this points to a c++ string that goes out of scope
-    Mode = decompProg.c_str();
+    ActiveSubprocess = decompProg;
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
+       Mode = ActiveSubprocess.c_str();
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
  }
                                                                        /*}}}*/
  // AcqIndexTrans::pkgAcqIndexTrans - Constructor                      /*{{{*/
  /* The Translation file is added to the queue */
  pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner,
                            string URI,string URIDesc,string ShortDesc) 
 -  : pkgAcqIndex(Owner, URI, URIDesc, ShortDesc, HashStringList(), "")
 +  : pkgAcqIndex(Owner, URI, URIDesc, ShortDesc, HashStringList())
  {
  }
 -pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner, IndexTarget const * const Target,
 -                       HashStringList const &ExpectedHashes, indexRecords *MetaIndexParser)
 -  : pkgAcqIndex(Owner, Target, ExpectedHashes, MetaIndexParser)
 +                                                                      /*}}}*/
 +pkgAcqIndexTrans::pkgAcqIndexTrans(pkgAcquire *Owner, 
 +                                   pkgAcqMetaBase *TransactionManager, 
 +                                   IndexTarget const * const Target,
 +                                   HashStringList const &ExpectedHashes, 
 +                                   indexRecords *MetaIndexParser)
 +   : pkgAcqIndex(Owner, TransactionManager, Target, ExpectedHashes, MetaIndexParser)
  {
     // load the filesize
     indexRecords::checkSum *Record = MetaIndexParser->Lookup(string(Target->MetaKey));
  // ---------------------------------------------------------------------
  string pkgAcqIndexTrans::Custom600Headers() const
  {
 -   std::string const compExt = CompressionExtension.substr(0, CompressionExtension.find(' '));
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (_config->FindB("Acquire::GzipIndexes",false))
 -      Final += compExt;
 +   string Final = GetFinalFilename();
  
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) != 0)
@@@ -1290,7 -1350,6 +1339,7 @@@ void pkgAcqIndexTrans::Failed(string Me
        return;
     }
  
 +   // FIXME: this is used often (e.g. in pkgAcqIndexTrans) so refactor
     if (Cnf->LocalOnly == true || 
         StringToBool(LookupTag(Message,"Transient-Failure"),false) == false)
     {      
     Item::Failed(Message,Cnf);
  }
                                                                        /*}}}*/
 -pkgAcqMetaSig::pkgAcqMetaSig(pkgAcquire *Owner,                               /*{{{*/
 +
 +void pkgAcqMetaBase::Add(Item *I)
 +{
 +   Transaction.push_back(I);
 +}
 +
 +void pkgAcqMetaBase::AbortTransaction()
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "AbortTransaction: " << TransactionManager << std::endl;
 +
 +   // ensure the toplevel is in error state too
 +   for (std::vector<Item*>::iterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +   {
 +      if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +         std::clog << "  Cancel: " << (*I)->DestFile << std::endl;
 +      // the transaction will abort, so stop anything that is idle
 +      if ((*I)->Status == pkgAcquire::Item::StatIdle)
 +         (*I)->Status = pkgAcquire::Item::StatDone;
 +   }
 +}
 +                                                                      /*}}}*/
 +bool pkgAcqMetaBase::TransactionHasError()
 +{
 +   for (pkgAcquire::ItemIterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +      if((*I)->Status != pkgAcquire::Item::StatDone &&
 +         (*I)->Status != pkgAcquire::Item::StatIdle)
 +         return true;
 +
 +   return false;
 +}
 +// Acquire::CommitTransaction - Commit a transaction                  /*{{{*/
 +void pkgAcqMetaBase::CommitTransaction()
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "CommitTransaction: " << this << std::endl;
 +
 +   // move new files into place *and* remove files that are not
 +   // part of the transaction but are still on disk
 +   for (std::vector<Item*>::iterator I = Transaction.begin();
 +        I != Transaction.end(); ++I)
 +   {
 +      if((*I)->PartialFile != "")
 +      {
 +         if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +            std::clog << "mv " 
 +                      << (*I)->PartialFile << " -> " 
 +                      <<  (*I)->DestFile << " " 
 +                      << (*I)->DescURI()
 +                      << std::endl;
 +         Rename((*I)->PartialFile, (*I)->DestFile);
 +         chmod((*I)->DestFile.c_str(),0644);
 +      } else {
 +         if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +            std::clog << "rm " 
 +                      <<  (*I)->DestFile
 +                      << " " 
 +                      << (*I)->DescURI()
 +                      << std::endl;
 +         unlink((*I)->DestFile.c_str());
 +      }
 +      // mark that this transaction is finished
 +      (*I)->TransactionManager = 0;
 +   }
 +}
 +
 +                                                                       /*{{{*/
 +bool pkgAcqMetaBase::GenerateAuthWarning(const std::string &RealURI,
 +                                         const std::string &Message)
 +{
 +   string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +   
 +   if(FileExists(Final))
 +   {
 +      Status = StatTransientNetworkError;
 +      _error->Warning(_("An error occurred during the signature "
 +                        "verification. The repository is not updated "
 +                        "and the previous index files will be used. "
 +                        "GPG error: %s: %s\n"),
 +                      Desc.Description.c_str(),
 +                      LookupTag(Message,"Message").c_str());
 +      RunScripts("APT::Update::Auth-Failure");
 +      return true;
 +   } else if (LookupTag(Message,"Message").find("NODATA") != string::npos) {
 +      /* Invalid signature file, reject (LP: #346386) (Closes: #627642) */
 +      _error->Error(_("GPG error: %s: %s"),
 +                    Desc.Description.c_str(),
 +                    LookupTag(Message,"Message").c_str());
 +      Status = StatError;
 +      return true;
 +   } else {
 +      _error->Warning(_("GPG error: %s: %s"),
 +                      Desc.Description.c_str(),
 +                      LookupTag(Message,"Message").c_str());
 +   }
 +   // gpgv method failed 
 +   ReportMirrorFailure("GPGFailure");
 +   return false;
 +}
 +                                                                      /*}}}*/
 +
 +
 +pkgAcqMetaSig::pkgAcqMetaSig(pkgAcquire *Owner,                       /*{{{*/
 +                             pkgAcqMetaBase *TransactionManager,
                             string URI,string URIDesc,string ShortDesc,
 -                           string MetaIndexURI, string MetaIndexURIDesc,
 -                           string MetaIndexShortDesc,
 +                             string MetaIndexFile,
                             const vector<IndexTarget*>* IndexTargets,
                             indexRecords* MetaIndexParser) :
 -   Item(Owner, HashStringList()), RealURI(URI), MetaIndexURI(MetaIndexURI),
 -   MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 -   MetaIndexParser(MetaIndexParser), IndexTargets(IndexTargets)
 +   pkgAcqMetaBase(Owner, HashStringList(), TransactionManager), RealURI(URI), 
 +   MetaIndexParser(MetaIndexParser), MetaIndexFile(MetaIndexFile),
 +   URIDesc(URIDesc), ShortDesc(ShortDesc),
 +   IndexTargets(IndexTargets), AuthPass(false), IMSHit(false)
  {
     DestFile = _config->FindDir("Dir::State::lists") + "partial/";
     DestFile += URItoFileName(URI);
     // partial download anyway
     unlink(DestFile.c_str());
  
 +   // set the TransactionManager
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgAcqMetaSig with TransactionManager "
 +                << TransactionManager << std::endl;
 +
     // Create the item
     Desc.Description = URIDesc;
     Desc.Owner = this;
     Desc.ShortDesc = ShortDesc;
     Desc.URI = URI;
 -      
 -   string Final = _config->FindDir("Dir::State::lists");
 -   Final += URItoFileName(RealURI);
 -   if (RealFileExists(Final) == true)
 -   {
 -      // File was already in place.  It needs to be re-downloaded/verified
 -      // because Release might have changed, we do give it a different
 -      // name than DestFile because otherwise the http method will
 -      // send If-Range requests and there are too many broken servers
 -      // out there that do not understand them
 -      LastGoodSig = DestFile+".reverify";
 -      Rename(Final,LastGoodSig);
 -   }
 -
 -   // we expect the indextargets + one additional Release file
 -   ExpectedAdditionalItems = IndexTargets->size() + 1;
  
     QueueURI(Desc);
  }
                                                                        /*}}}*/
  pkgAcqMetaSig::~pkgAcqMetaSig()                                               /*{{{*/
  {
 -   // if the file was never queued undo file-changes done in the constructor
 -   if (QueueCounter == 1 && Status == StatIdle && FileSize == 0 && Complete == false &&
 -       LastGoodSig.empty() == false)
 -   {
 -      string const Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -      if (RealFileExists(Final) == false && RealFileExists(LastGoodSig) == true)
 -       Rename(LastGoodSig, Final);
 -   }
 -
  }
                                                                        /*}}}*/
  // pkgAcqMetaSig::Custom600Headers - Insert custom request headers    /*{{{*/
  /* The only header we use is the last-modified header. */
  string pkgAcqMetaSig::Custom600Headers() const
  {
 +   string FinalFile = _config->FindDir("Dir::State::lists");
 +   FinalFile += URItoFileName(RealURI);
 +
     struct stat Buf;
 -   if (stat(LastGoodSig.c_str(),&Buf) != 0)
 +   if (stat(FinalFile.c_str(),&Buf) != 0)
        return "\nIndex-File: true";
  
     return "\nIndex-File: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
@@@ -1482,98 -1453,51 +1531,98 @@@ void pkgAcqMetaSig::Done(string Message
        return;
     }
  
 -   Complete = true;
 +   if(StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 +      IMSHit = true;
  
 -   // at this point pkgAcqMetaIndex takes over
 -   ExpectedAdditionalItems = 0;
 +   // adjust paths if its a ims-hit
 +   if(IMSHit)
 +   {
 +      string FinalFile = _config->FindDir("Dir::State::lists");
 +      FinalFile += URItoFileName(RealURI);
 +         
 +      DestFile = PartialFile = FinalFile;
 +   }
  
 -   // put the last known good file back on i-m-s hit (it will
 -   // be re-verified again)
 -   // Else do nothing, we have the new file in DestFile then
 -   if(StringToBool(LookupTag(Message,"IMS-Hit"),false) == true)
 -      Rename(LastGoodSig, DestFile);
 +   // queue for verify
 +   if(AuthPass == false)
 +   {
 +      AuthPass = true;
 +      Desc.URI = "gpgv:" + DestFile;
 +      DestFile = MetaIndexFile;
 +      QueueURI(Desc);
 +      return;
 +   }
  
 -   // queue a pkgAcqMetaIndex to be verified against the sig we just retrieved
 -   new pkgAcqMetaIndex(Owner, MetaIndexURI, MetaIndexURIDesc, 
 -                     MetaIndexShortDesc,  DestFile, IndexTargets, 
 -                     MetaIndexParser);
 +   // queue to copy the file in place if it was not a ims hit, on ims
 +   // hit the file is already at the right place
 +   if(IMSHit == false)
 +   {
 +      PartialFile = _config->FindDir("Dir::State::lists") + "partial/";
 +      PartialFile += URItoFileName(RealURI);
 +      
 +      DestFile = _config->FindDir("Dir::State::lists");
 +      DestFile += URItoFileName(RealURI);
 +   }
 +
 +   // we parse the MetaIndexFile here because at this point we can
 +   // trust the data
 +   if(AuthPass == true)
 +   {
 +      // load indexes and queue further downloads
 +      MetaIndexParser->Load(MetaIndexFile);
 +      ((pkgAcqMetaIndex*)TransactionManager)->QueueIndexes(true);
 +   }
  
 +   Complete = true;
  }
                                                                        /*}}}*/
  void pkgAcqMetaSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf)/*{{{*/
  {
     string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +   
 +   // FIXME: meh, this is not really elegant
 +   string InReleaseURI = RealURI.replace(RealURI.rfind("Release.gpg"), 12,
 +                                         "InRelease");
 +   string FinalInRelease = _config->FindDir("Dir::State::lists") + URItoFileName(InReleaseURI);
  
 -   // at this point pkgAcqMetaIndex takes over
 -   ExpectedAdditionalItems = 0;
 -
 -   // if we get a network error we fail gracefully
 -   if(Status == StatTransientNetworkError)
 +   if(RealFileExists(Final) || RealFileExists(FinalInRelease))
     {
 -      Item::Failed(Message,Cnf);
 -      // move the sigfile back on transient network failures 
 -      if(FileExists(LastGoodSig))
 -       Rename(LastGoodSig,Final);
 -
 -      // set the status back to , Item::Failed likes to reset it
 -      Status = pkgAcquire::Item::StatTransientNetworkError;
 +      _error->Error("The repository '%s' is no longer signed.",
 +                    URIDesc.c_str());
 +      Rename(MetaIndexFile, MetaIndexFile+".FAILED");
 +      Status = pkgAcquire::Item::StatError;
 +      TransactionManager->AbortTransaction();
        return;
     }
  
 -   // Delete any existing sigfile when the acquire failed
 -   unlink(Final.c_str());
 +   // this ensures that any file in the lists/ dir is removed by the
 +   // transaction
 +   DestFile =  _config->FindDir("Dir::State::lists") + "partial/";
 +   DestFile += URItoFileName(RealURI);
 +   PartialFile = "";
  
 -   // queue a pkgAcqMetaIndex with no sigfile
 -   new pkgAcqMetaIndex(Owner, MetaIndexURI, MetaIndexURIDesc, MetaIndexShortDesc,
 -                     "", IndexTargets, MetaIndexParser);
 +   // FIXME: duplicated code from pkgAcqMetaIndex
 +   if (AuthPass == true)
 +   {
 +      bool Stop = GenerateAuthWarning(RealURI, Message);
 +      if(Stop)
 +         return;
 +   }
  
 +   // only allow going further if the users explicitely wants it
 +   if(_config->FindB("APT::Get::AllowUnauthenticated", false) == true)
 +   {
 +      // we parse the indexes here because at this point the user wanted
 +      // a repository that may potentially harm him
 +      MetaIndexParser->Load(MetaIndexFile);
 +      ((pkgAcqMetaIndex*)TransactionManager)->QueueIndexes(true);
 +   } 
 +   else 
 +   {
 +      _error->Warning("Use --allow-unauthenticated to force the update");
 +   }
 +
 +   // FIXME: this is used often (e.g. in pkgAcqIndexTrans) so refactor
     if (Cnf->LocalOnly == true || 
         StringToBool(LookupTag(Message,"Transient-Failure"),false) == false)
     {      
        Dequeue();
        return;
     }
 -   
     Item::Failed(Message,Cnf);
  }
                                                                        /*}}}*/
  pkgAcqMetaIndex::pkgAcqMetaIndex(pkgAcquire *Owner,                   /*{{{*/
 +                                 pkgAcqMetaBase *TransactionManager,
                                 string URI,string URIDesc,string ShortDesc,
 -                               string SigFile,
 +                                 string MetaIndexSigURI,string MetaIndexSigURIDesc, string MetaIndexSigShortDesc,
                                 const vector<IndexTarget*>* IndexTargets,
                                 indexRecords* MetaIndexParser) :
 -   Item(Owner, HashStringList()), RealURI(URI), SigFile(SigFile), IndexTargets(IndexTargets),
 -   MetaIndexParser(MetaIndexParser), AuthPass(false), IMSHit(false)
 +   pkgAcqMetaBase(Owner, HashStringList(), TransactionManager), RealURI(URI), IndexTargets(IndexTargets),
 +   URIDesc(URIDesc), ShortDesc(ShortDesc),
 +   MetaIndexParser(MetaIndexParser), AuthPass(false), IMSHit(false),
 +   MetaIndexSigURI(MetaIndexSigURI), MetaIndexSigURIDesc(MetaIndexSigURIDesc),
 +   MetaIndexSigShortDesc(MetaIndexSigShortDesc)
 +{
 +   if(TransactionManager == NULL)
 +   {
 +      this->TransactionManager = this;
 +      this->TransactionManager->Add(this);
 +   }
 +
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "New pkgAcqMetaIndex with TransactionManager "
 +                << this->TransactionManager << std::endl;
 +
 +
 +   Init(URIDesc, ShortDesc);
 +}
 +                                                                      /*}}}*/
 +// pkgAcqMetaIndex::Init - Delayed constructor                         /*{{{*/
 +void pkgAcqMetaIndex::Init(std::string URIDesc, std::string ShortDesc)
  {
     DestFile = _config->FindDir("Dir::State::lists") + "partial/";
 -   DestFile += URItoFileName(URI);
 +   DestFile += URItoFileName(RealURI);
  
     // Create the item
     Desc.Description = URIDesc;
     Desc.Owner = this;
     Desc.ShortDesc = ShortDesc;
 -   Desc.URI = URI;
 +   Desc.URI = RealURI;
  
     // we expect more item
     ExpectedAdditionalItems = IndexTargets->size();
 -
     QueueURI(Desc);
  }
 -                                                                      /*}}}*/
  // pkgAcqMetaIndex::Custom600Headers - Insert custom request headers  /*{{{*/
  // ---------------------------------------------------------------------
  /* The only header we use is the last-modified header. */
@@@ -1666,8 -1572,27 +1715,8 @@@ void pkgAcqMetaIndex::Done(string Messa
           // Still more retrieving to do
           return;
  
 -      if (SigFile == "")
 -      {
 -         // There was no signature file, so we are finished.  Download
 -         // the indexes and do only hashsum verification if possible
 -         MetaIndexParser->Load(DestFile);
 -         QueueIndexes(false);
 -      }
 -      else
 +      if (SigFile != "")
        {
 -         // FIXME: move this into pkgAcqMetaClearSig::Done on the next
 -         //        ABI break
 -
 -         // if we expect a ClearTextSignature (InRelase), ensure that
 -         // this is what we get and if not fail to queue a 
 -         // Release/Release.gpg, see #346386
 -         if (SigFile == DestFile && !StartsWithGPGClearTextSignature(DestFile))
 -         {
 -            Failed(Message, Cfg);
 -            return;
 -         }
 -
           // There was a signature file, so pass it to gpgv for
           // verification
           if (_config->FindB("Debug::pkgAcquire::Auth", false))
           AuthPass = true;
           Desc.URI = "gpgv:" + SigFile;
           QueueURI(Desc);
-          Mode = "gpgv";
+        ActiveSubprocess = "gpgv";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic push
+       #pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+ #endif
+        Mode = "gpgv";
+ #if __GNUC__ >= 4
+       #pragma GCC diagnostic pop
+ #endif
         return;
        }
     }
        FinalFile += URItoFileName(RealURI);
        if (SigFile == DestFile)
         SigFile = FinalFile;
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
 +      // queue for copy in place
 +      PartialFile = DestFile;
        DestFile = FinalFile;
     }
  }
@@@ -1723,24 -1656,14 +1780,24 @@@ void pkgAcqMetaIndex::RetrievalDone(str
        if (SigFile == DestFile)
        {
         SigFile = FinalFile;
 +#if 0
         // constructor of pkgAcqMetaClearSig moved it out of the way,
         // now move it back in on IMS hit for the 'old' file
         string const OldClearSig = DestFile + ".reverify";
         if (RealFileExists(OldClearSig) == true)
            Rename(OldClearSig, FinalFile);
 +#endif
        }
        DestFile = FinalFile;
     }
 +
 +   // queue a signature
 +   if(SigFile != DestFile)
 +      new pkgAcqMetaSig(Owner, TransactionManager, 
 +                        MetaIndexSigURI, MetaIndexSigURIDesc,
 +                        MetaIndexSigShortDesc, DestFile, IndexTargets, 
 +                        MetaIndexParser);
 +
     Complete = true;
  }
                                                                        /*}}}*/
@@@ -1767,8 -1690,6 +1824,8 @@@ void pkgAcqMetaIndex::AuthDone(string M
        std::cerr << "Signature verification succeeded: "
                  << DestFile << std::endl;
  
 +// we ensure this by other means
 +#if 0 
     // do not trust any previously unverified content that we may have
     string LastGoodSigFile = _config->FindDir("Dir::State::lists").append("partial/").append(URItoFileName(RealURI));
     if (DestFile != SigFile)
              URItoFileName((*Target)->URI);
           unlink(index.c_str());
           // and also old gzipindexes
-          index += ".gz";
-          unlink(index.c_str());
+          std::vector<std::string> types = APT::Configuration::getCompressionTypes();
+          for (std::vector<std::string>::const_iterator t = types.begin(); t != types.end(); ++t)
+          {
+             index += '.' + (*t);
+             unlink(index.c_str());
+          }
        }
     }
 +#endif
  
 -
 -   // Download further indexes with verification
 +   // Download further indexes with verification 
 +   //
 +   // it would be really nice if we could simply do
 +   //    if (IMSHit == false) QueueIndexes(true)
 +   // and skip the download if the Release file has not changed
 +   // - but right now the list cleaner will needs to be tricked
 +   //   to not delete all our packages/source indexes in this case
     QueueIndexes(true);
  
 +#if 0
     // is it a clearsigned MetaIndex file?
     if (DestFile == SigFile)
        return;
        URItoFileName(RealURI) + ".gpg";
     Rename(SigFile,VerifiedSigFile);
     chmod(VerifiedSigFile.c_str(),0644);
 +#endif
  }
                                                                        /*}}}*/
  void pkgAcqMetaIndex::QueueIndexes(bool verify)                               /*{{{*/
  {
 -#if 0
 -   /* Reject invalid, existing Release files (LP: #346386) (Closes: #627642)
 -    * FIXME: Disabled; it breaks unsigned repositories without hashes */
 -   if (!verify && FileExists(DestFile) && !MetaIndexParser->Load(DestFile))
 -   {
 -      Status = StatError;
 -      ErrorText = MetaIndexParser->ErrorText;
 -      return;
 -   }
 -#endif
     bool transInRelease = false;
     {
        std::vector<std::string> const keys = MetaIndexParser->MetaKeys();
  
        if ((*Target)->IsOptional() == true)
        {
 -       if ((*Target)->IsSubIndex() == true)
 -          new pkgAcqSubIndex(Owner, (*Target)->URI, (*Target)->Description,
 -                              (*Target)->ShortDesc, ExpectedIndexHashes);
 -       else if (transInRelease == false || Record != NULL || compressedAvailable == true)
 +       if (transInRelease == false || Record != NULL || compressedAvailable == true)
         {
            if (_config->FindB("Acquire::PDiffs",true) == true && transInRelease == true &&
                MetaIndexParser->Exists((*Target)->MetaKey + ".diff/Index") == true)
 -             new pkgAcqDiffIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +             new pkgAcqDiffIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
            else
 -             new pkgAcqIndexTrans(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +             new pkgAcqIndexTrans(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
         }
         continue;
        }
           instead, but passing the required info to it is to much hassle */
        if(_config->FindB("Acquire::PDiffs",true) == true && (verify == false ||
          MetaIndexParser->Exists((*Target)->MetaKey + ".diff/Index") == true))
 -       new pkgAcqDiffIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +       new pkgAcqDiffIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
        else
 -       new pkgAcqIndex(Owner, *Target, ExpectedIndexHashes, MetaIndexParser);
 +       new pkgAcqIndex(Owner, TransactionManager, *Target, ExpectedIndexHashes, MetaIndexParser);
     }
  }
                                                                        /*}}}*/
@@@ -1979,22 -1909,49 +2040,22 @@@ bool pkgAcqMetaIndex::VerifyVendor(stri
  // pkgAcqMetaIndex::Failed - no Release file present or no signature file present     /*{{{*/
  // ---------------------------------------------------------------------
  /* */
 -void pkgAcqMetaIndex::Failed(string Message,pkgAcquire::MethodConfig * /*Cnf*/)
 +void pkgAcqMetaIndex::Failed(string Message,
 +                             pkgAcquire::MethodConfig * /*Cnf*/)
  {
 +   string Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 +
     if (AuthPass == true)
     {
 -      // gpgv method failed, if we have a good signature 
 -      string LastGoodSigFile = _config->FindDir("Dir::State::lists").append("partial/").append(URItoFileName(RealURI));
 -      if (DestFile != SigFile)
 -       LastGoodSigFile.append(".gpg");
 -      LastGoodSigFile.append(".reverify");
 -
 -      if(FileExists(LastGoodSigFile))
 -      {
 -       string VerifiedSigFile = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
 -       if (DestFile != SigFile)
 -          VerifiedSigFile.append(".gpg");
 -       Rename(LastGoodSigFile, VerifiedSigFile);
 -       Status = StatTransientNetworkError;
 -       _error->Warning(_("An error occurred during the signature "
 -                         "verification. The repository is not updated "
 -                         "and the previous index files will be used. "
 -                         "GPG error: %s: %s\n"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -       RunScripts("APT::Update::Auth-Failure");
 -       return;
 -      } else if (LookupTag(Message,"Message").find("NODATA") != string::npos) {
 -       /* Invalid signature file, reject (LP: #346386) (Closes: #627642) */
 -       _error->Error(_("GPG error: %s: %s"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -       return;
 -      } else {
 -       _error->Warning(_("GPG error: %s: %s"),
 -                       Desc.Description.c_str(),
 -                       LookupTag(Message,"Message").c_str());
 -      }
 -      // gpgv method failed 
 -      ReportMirrorFailure("GPGFailure");
 +      bool Stop = GenerateAuthWarning(RealURI, Message);
 +      if(Stop)
 +         return;
     }
  
     /* Always move the meta index, even if gpgv failed. This ensures
      * that PackageFile objects are correctly filled in */
 -   if (FileExists(DestFile)) {
 +   if (FileExists(DestFile)) 
 +   {
        string FinalFile = _config->FindDir("Dir::State::lists");
        FinalFile += URItoFileName(RealURI);
        /* InRelease files become Release files, otherwise
                                       "Release");
         SigFile = FinalFile;
        }
 -      Rename(DestFile,FinalFile);
 -      chmod(FinalFile.c_str(),0644);
  
 +      // Done, queue for rename on transaction finished
 +      PartialFile = DestFile;
        DestFile = FinalFile;
     }
  
 +   _error->Warning(_("The data from '%s' is not signed. Packages "
 +                     "from that repository can not be authenticated."),
 +                   URIDesc.c_str());
 +
     // No Release file was present, or verification failed, so fall
     // back to queueing Packages files without verification
 -   QueueIndexes(false);
 +   // only allow going further if the users explicitely wants it
 +   if(_config->FindB("APT::Get::AllowUnauthenticated", false) == true)
 +   {
 +      QueueIndexes(false);
 +   } else {
 +      // warn if the repository is unsinged
 +      _error->Warning("Use --allow-unauthenticated to force the update");
 +   } 
  }
                                                                        /*}}}*/
 +
 +void pkgAcqMetaIndex::Finished()
 +{
 +   if(_config->FindB("Debug::Acquire::Transaction", false) == true)
 +      std::clog << "Finished: " << DestFile <<std::endl;
 +   if(TransactionManager != NULL &&
 +      TransactionManager->TransactionHasError() == false)
 +      TransactionManager->CommitTransaction();
 +}
 +
 +
  pkgAcqMetaClearSig::pkgAcqMetaClearSig(pkgAcquire *Owner,             /*{{{*/
                string const &URI, string const &URIDesc, string const &ShortDesc,
                string const &MetaIndexURI, string const &MetaIndexURIDesc, string const &MetaIndexShortDesc,
                string const &MetaSigURI, string const &MetaSigURIDesc, string const &MetaSigShortDesc,
                const vector<IndexTarget*>* IndexTargets,
                indexRecords* MetaIndexParser) :
 -      pkgAcqMetaIndex(Owner, URI, URIDesc, ShortDesc, "", IndexTargets, MetaIndexParser),
 -      MetaIndexURI(MetaIndexURI), MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 -      MetaSigURI(MetaSigURI), MetaSigURIDesc(MetaSigURIDesc), MetaSigShortDesc(MetaSigShortDesc)
 +   pkgAcqMetaIndex(Owner, NULL, URI, URIDesc, ShortDesc, MetaSigURI, MetaSigURIDesc,MetaSigShortDesc, IndexTargets, MetaIndexParser),
 +       MetaIndexURI(MetaIndexURI), MetaIndexURIDesc(MetaIndexURIDesc), MetaIndexShortDesc(MetaIndexShortDesc),
 +       MetaSigURI(MetaSigURI), MetaSigURIDesc(MetaSigURIDesc), MetaSigShortDesc(MetaSigShortDesc)
  {
     SigFile = DestFile;
  
     // index targets + (worst case:) Release/Release.gpg
     ExpectedAdditionalItems = IndexTargets->size() + 2;
  
 -
 +#if 0
     // keep the old InRelease around in case of transistent network errors
     string const Final = _config->FindDir("Dir::State::lists") + URItoFileName(RealURI);
     if (RealFileExists(Final) == true)
        string const LastGoodSig = DestFile + ".reverify";
        Rename(Final,LastGoodSig);
     }
 +#endif
  }
                                                                        /*}}}*/
  pkgAcqMetaClearSig::~pkgAcqMetaClearSig()                             /*{{{*/
  {
 +#if 0
     // if the file was never queued undo file-changes done in the constructor
     if (QueueCounter == 1 && Status == StatIdle && FileSize == 0 && Complete == false)
     {
        if (RealFileExists(Final) == false && RealFileExists(LastGoodSig) == true)
         Rename(LastGoodSig, Final);
     }
 +#endif
  }
                                                                        /*}}}*/
  // pkgAcqMetaClearSig::Custom600Headers - Insert custom request headers       /*{{{*/
@@@ -2090,6 -2022,7 +2151,6 @@@ string pkgAcqMetaClearSig::Custom600Hea
     struct stat Buf;
     if (stat(Final.c_str(),&Buf) != 0)
     {
 -      Final = DestFile + ".reverify";
        if (stat(Final.c_str(),&Buf) != 0)
         return "\nIndex-File: true\nFail-Ignore: true\n";
     }
     return "\nIndex-File: true\nFail-Ignore: true\nLast-Modified: " + TimeRFC1123(Buf.st_mtime);
  }
                                                                        /*}}}*/
 +// pkgAcqMetaClearSig::Done - We got a file                           /*{{{*/
 +// ---------------------------------------------------------------------
 +void pkgAcqMetaClearSig::Done(std::string Message,unsigned long long Size, 
 +                              HashStringList const &Hashes,
 +                              pkgAcquire::MethodConfig *Cnf)
 +{
 +   // if we expect a ClearTextSignature (InRelase), ensure that
 +   // this is what we get and if not fail to queue a 
 +   // Release/Release.gpg, see #346386
 +   if (FileExists(DestFile) && !StartsWithGPGClearTextSignature(DestFile))
 +   {
 +      pkgAcquire::Item::Failed(Message, Cnf);
 +      RenameOnError(NotClearsigned);
 +      TransactionManager->AbortTransaction();
 +      return;
 +   }
 +   pkgAcqMetaIndex::Done(Message, Size, Hashes, Cnf);
 +}
 +                                                                      /*}}}*/
  void pkgAcqMetaClearSig::Failed(string Message,pkgAcquire::MethodConfig *Cnf) /*{{{*/
  {
     // we failed, we will not get additional items from this method
  
     if (AuthPass == false)
     {
 -      // Remove the 'old' InRelease file if we try Release.gpg now as otherwise
 -      // the file will stay around and gives a false-auth impression (CVE-2012-0214)
 +      // Queue the 'old' InRelease file for removal if we try Release.gpg
 +      // as otherwise the file will stay around and gives a false-auth
 +      // impression (CVE-2012-0214)
        string FinalFile = _config->FindDir("Dir::State::lists");
        FinalFile.append(URItoFileName(RealURI));
 -      if (FileExists(FinalFile))
 -       unlink(FinalFile.c_str());
 +      PartialFile = "";
 +      DestFile = FinalFile;
  
 -      new pkgAcqMetaSig(Owner,
 -                      MetaSigURI, MetaSigURIDesc, MetaSigShortDesc,
 +      new pkgAcqMetaIndex(Owner, TransactionManager,
                        MetaIndexURI, MetaIndexURIDesc, MetaIndexShortDesc,
 +                      MetaSigURI, MetaSigURIDesc, MetaSigShortDesc,
                        IndexTargets, MetaIndexParser);
        if (Cnf->LocalOnly == true ||
          StringToBool(LookupTag(Message, "Transient-Failure"), false) == false)
diff --combined apt-pkg/acquire-item.h
index cc156cf17d2469dae7bc261dba44bca462ea659b,74b5de67582fa4a6d3a33acc0360569bc4139a75..e6a22ce7b7ec92ec6113aefab23e8dec868ee348
@@@ -47,7 -47,6 +47,7 @@@ class indexRecords
  class pkgRecords;
  class pkgSourceList;
  class IndexTarget;
 +class pkgAcqMetaBase;
  
  /** \brief Represents the process by which a pkgAcquire object should {{{
   *  retrieve a file or a collection of files.
@@@ -63,8 -62,6 +63,8 @@@
   */
  class pkgAcquire::Item : public WeakPointable
  {  
 +   void *d;
 +
     protected:
     
     /** \brief The acquire object with which this item is associated. */
@@@ -90,7 -87,7 +90,7 @@@
      *  \param To The new name of \a From.  If \a To exists it will be
      *  overwritten.
      */
 -   void Rename(std::string From,std::string To);
 +   bool Rename(std::string From,std::string To);
  
     public:
  
         /** \brief The item was could not be downloaded because of 
        *  a transient network error (e.g. network down)
        */
 -       StatTransientNetworkError
 +       StatTransientNetworkError,
       } Status;
  
     /** \brief Contains a textual description of the error encountered
     /** \brief If not \b NULL, contains the name of a subprocess that
      *  is operating on this object (for instance, "gzip" or "gpgv").
      */
-    const char *Mode;
+    APT_DEPRECATED const char *Mode;
+    /** \brief contains the name of the subprocess that is operating on this object
+     * (for instance, "gzip", "rred" or "gpgv"). This is obsoleting #Mode from above
+     * as it can manage the lifetime of included string properly. */
+    std::string ActiveSubprocess;
  
     /** \brief A client-supplied unique identifier.
      * 
      */
     unsigned int QueueCounter;
  
 +   /** \brief TransactionManager */
 +   pkgAcqMetaBase *TransactionManager;
 +
     /** \brief The number of additional fetch items that are expected
      *  once this item is done.
      *
      */
     std::string DestFile;
  
 +   /** \brief storge name until a transaction is finished */
 +   std::string PartialFile;
 +
     /** \brief Invoked by the acquire worker when the object couldn't
      *  be fetched.
      *
      *
      *  \return a URI that should be used to describe what is being fetched.
      */
-    virtual std::string DescURI() const = 0;
+    virtual std::string DescURI() = 0;
     /** \brief Short item description.
      *
      *  \return a brief description of the object being fetched.
      */
-    virtual std::string ShortDesc() const {return DescURI();}
+    virtual std::string ShortDesc() {return DescURI();}
  
     /** \brief Invoked by the worker when the download is completely done. */
     virtual void Finished() {};
      *  \param ExpectedHashes of the file represented by this item
      */
     Item(pkgAcquire *Owner,
 -        HashStringList const &ExpectedHashes=HashStringList());
 +        HashStringList const &ExpectedHashes=HashStringList(),
 +        pkgAcqMetaBase *TransactionManager=NULL);
  
     /** \brief Remove this item from its owner's queue by invoking
      *  pkgAcquire::Remove.
     enum RenameOnErrorState {
        HashSumMismatch,
        SizeMismatch,
 -      InvalidFormat
 +      InvalidFormat,
 +      SignatureError,
 +      NotClearsigned,
     };
  
     /** \brief Rename failed file and set error
@@@ -342,270 -335,62 +347,270 @@@ struct DiffInfo 
     unsigned long size;
  };
                                                                        /*}}}*/
 -/** \brief An item that is responsible for fetching a SubIndex                {{{
 - *
 - *  The MetaIndex file includes only records for important indexes
 - *  and records for these SubIndex files so these can carry records
 - *  for addition files like PDiffs and Translations
 - */
 -class pkgAcqSubIndex : public pkgAcquire::Item
 +                                                                      /*}}}*/
 +
 +class pkgAcqMetaBase  : public pkgAcquire::Item
  {
 +   void *d;
 +
   protected:
 -   /** \brief If \b true, debugging information will be written to std::clog. */
 -   bool Debug;
 +   std::vector<Item*> Transaction;
  
   public:
 +   // transaction code
 +   void Add(Item *I);
 +   void AbortTransaction();
 +   bool TransactionHasError() APT_PURE;
 +   void CommitTransaction();
 +
 +   // helper for the signature warning
 +   bool GenerateAuthWarning(const std::string &RealURI,
 +                            const std::string &Message);
 +
 +
 +   pkgAcqMetaBase(pkgAcquire *Owner,
 +                  HashStringList const &ExpectedHashes=HashStringList(),
 +                  pkgAcqMetaBase *TransactionManager=NULL)
 +      : Item(Owner, ExpectedHashes, TransactionManager) {};
 +};
 +
 +/** \brief An acquire item that downloads the detached signature      {{{
 + *  of a meta-index (Release) file, then queues up the release
 + *  file itself.
 + *
 + *  \todo Why protected members?
 + *
 + *  \sa pkgAcqMetaIndex
 + */
 +class pkgAcqMetaSig : public pkgAcqMetaBase
 +{
 +   void *d;
 +
 +   protected:
 +
 +   /** \brief The URI of the signature file.  Unlike Desc.URI, this is
 +    *  never modified; it is used to determine the file that is being
 +    *  downloaded.
 +    */
 +   std::string RealURI;
 +
 +   std::string URIDesc;
 +   std::string ShortDesc;
 +
 +   /** \brief A package-system-specific parser for the meta-index file. */
 +   indexRecords* MetaIndexParser;
 +
 +   /** \brief The file we need to verify */
 +   std::string MetaIndexFile;
 +
 +   /** \brief The index files which should be looked up in the meta-index
 +    *  and then downloaded.
 +    *
 +    *  \todo Why a list of pointers instead of a list of structs?
 +    */
 +   const std::vector<IndexTarget*>* IndexTargets;
 +
 +   /** \brief If we are in fetching or download state */
 +   bool AuthPass;
 +
 +   /** \brief Was this file already on disk */
 +   bool IMSHit;
 +
 +   public:
 +   
     // Specialized action members
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string DescURI() {return Desc.URI;};
     virtual std::string Custom600Headers() const;
-    virtual std::string DescURI() const {return RealURI; };
 -   virtual bool ParseIndex(std::string const &IndexFile);
++   virtual std::string DescURI() {return RealURI; };
 +
 +   /** \brief Create a new pkgAcqMetaSig. */
 +   pkgAcqMetaSig(pkgAcquire *Owner,
 +                 pkgAcqMetaBase *TransactionManager,
 +                 std::string URI,std::string URIDesc, std::string ShortDesc,
 +                 std::string MetaIndexFile,
 +               const std::vector<IndexTarget*>* IndexTargets,
 +               indexRecords* MetaIndexParser);
 +   virtual ~pkgAcqMetaSig();
 +};
 +                                                                      /*}}}*/
 +
 +/** \brief An item that is responsible for downloading the meta-index {{{
 + *  file (i.e., Release) itself and verifying its signature.
 + *
 + *  Once the download and verification are complete, the downloads of
 + *  the individual index files are queued up using pkgAcqDiffIndex.
 + *  If the meta-index file had a valid signature, the expected hashsums
 + *  of the index files will be the md5sums listed in the meta-index;
 + *  otherwise, the expected hashsums will be "" (causing the
 + *  authentication of the index files to be bypassed).
 + */
 +class pkgAcqMetaIndex : public pkgAcqMetaBase
 +{
 +   void *d;
 +
 +   protected:
 +   /** \brief The URI that is actually being downloaded; never
 +    *  modified by pkgAcqMetaIndex.
 +    */
 +   std::string RealURI;
  
 -   /** \brief Create a new pkgAcqSubIndex.
 +   /** \brief The file in which the signature for this index was stored.
      *
 -    *  \param Owner The Acquire object that owns this item.
 +    *  If empty, the signature and the md5sums of the individual
 +    *  indices will not be checked.
 +    */
 +   std::string SigFile;
 +
 +   /** \brief The index files to download. */
 +   const std::vector<IndexTarget*>* IndexTargets;
 +
 +   /** \brief The parser for the meta-index file. */
 +   indexRecords* MetaIndexParser;
 +
 +   /** \brief If \b true, the index's signature is currently being verified.
 +    */
 +   bool AuthPass;
 +   // required to deal gracefully with problems caused by incorrect ims hits
 +   bool IMSHit; 
 +
 +   /** \brief Check that the release file is a release file for the
 +    *  correct distribution.
      *
 -    *  \param URI The URI of the list file to download.
 +    *  \return \b true if no fatal errors were encountered.
 +    */
 +   bool VerifyVendor(std::string Message);
 +
 +   /** \brief Called when a file is finished being retrieved.
      *
 -    *  \param URIDesc A long description of the list file to download.
 +    *  If the file was not downloaded to DestFile, a copy process is
 +    *  set up to copy it to DestFile; otherwise, Complete is set to \b
 +    *  true and the file is moved to its final location.
      *
 -    *  \param ShortDesc A short description of the list file to download.
 +    *  \param Message The message block received from the fetch
 +    *  subprocess.
 +    */
 +   void RetrievalDone(std::string Message);
 +
 +   /** \brief Called when authentication succeeded.
      *
 -    *  \param ExpectedHashes The list file's hashsums which are expected.
 +    *  Sanity-checks the authenticated file, queues up the individual
 +    *  index files for download, and saves the signature in the lists
 +    *  directory next to the authenticated list file.
 +    *
 +    *  \param Message The message block received from the fetch
 +    *  subprocess.
      */
 -   pkgAcqSubIndex(pkgAcquire *Owner, std::string const &URI,std::string const &URIDesc,
 -                 std::string const &ShortDesc, HashStringList const &ExpectedHashes);
 +   void AuthDone(std::string Message);
 +
 +   std::string URIDesc;
 +   std::string ShortDesc;
 +
 +   /** \brief The URI of the meta-index file for the detached signature */
 +   std::string MetaIndexSigURI;
 +
 +   /** \brief A "URI-style" description of the meta-index file */
 +   std::string MetaIndexSigURIDesc;
 +
 +   /** \brief A brief description of the meta-index file */
 +   std::string MetaIndexSigShortDesc;
 +
 +   /** \brief delayed constructor */
 +   void Init(std::string URIDesc, std::string ShortDesc);
 +   
 +   public:
 +
 +   /** \brief Starts downloading the individual index files.
 +    *
 +    *  \param verify If \b true, only indices whose expected hashsum
 +    *  can be determined from the meta-index will be downloaded, and
 +    *  the hashsums of indices will be checked (reporting
 +    *  #StatAuthError if there is a mismatch).  If verify is \b false,
 +    *  no hashsum checking will be performed.
 +    */
 +   void QueueIndexes(bool verify);
 +
 +   // Specialized action members
 +   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 +   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 +                   pkgAcquire::MethodConfig *Cnf);
 +   virtual std::string Custom600Headers() const;
-    virtual std::string DescURI() const {return RealURI; };
++   virtual std::string DescURI() {return RealURI; };
 +   virtual void Finished();
 +
 +   /** \brief Create a new pkgAcqMetaIndex. */
 +   pkgAcqMetaIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
 +                 std::string URI,std::string URIDesc, std::string ShortDesc,
 +                   std::string MetaIndexSigURI, std::string MetaIndexSigURIDesc, std::string MetaIndexSigShortDesc,
 +                 const std::vector<IndexTarget*>* IndexTargets,
 +                 indexRecords* MetaIndexParser);
 +};
 +                                                                      /*}}}*/
 +/** \brief An item repsonsible for downloading clearsigned metaindexes        {{{*/
 +class pkgAcqMetaClearSig : public pkgAcqMetaIndex
 +{
 +   void *d;
 +
 +   /** \brief The URI of the meta-index file for the detached signature */
 +   std::string MetaIndexURI;
 +
 +   /** \brief A "URI-style" description of the meta-index file */
 +   std::string MetaIndexURIDesc;
 +
 +   /** \brief A brief description of the meta-index file */
 +   std::string MetaIndexShortDesc;
 +
 +   /** \brief The URI of the detached meta-signature file if the clearsigned one failed. */
 +   std::string MetaSigURI;
 +
 +   /** \brief A "URI-style" description of the meta-signature file */
 +   std::string MetaSigURIDesc;
 +
 +   /** \brief A brief description of the meta-signature file */
 +   std::string MetaSigShortDesc;
 +
 +public:
 +   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 +   virtual std::string Custom600Headers() const;
 +   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 +                   pkgAcquire::MethodConfig *Cnf);
 +
 +   /** \brief Create a new pkgAcqMetaClearSig. */
 +   pkgAcqMetaClearSig(pkgAcquire *Owner,
 +              std::string const &URI, std::string const &URIDesc, std::string const &ShortDesc,
 +              std::string const &MetaIndexURI, std::string const &MetaIndexURIDesc, std::string const &MetaIndexShortDesc,
 +              std::string const &MetaSigURI, std::string const &MetaSigURIDesc, std::string const &MetaSigShortDesc,
 +              const std::vector<IndexTarget*>* IndexTargets,
 +              indexRecords* MetaIndexParser);
 +   virtual ~pkgAcqMetaClearSig();
  };
                                                                        /*}}}*/
  
 +
  /** \brief Common base class for all classes that deal with fetching  {{{
             indexes
   */
  class pkgAcqBaseIndex : public pkgAcquire::Item
  {
 +   void *d;
 +
   protected:
     /** \brief Pointer to the IndexTarget data
      */
     const struct IndexTarget * Target;
     indexRecords *MetaIndexParser;
 +   /** \brief The MetaIndex Key */
 +   std::string MetaKey;
  
     pkgAcqBaseIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
                     struct IndexTarget const * const Target,
                     HashStringList const &ExpectedHashes,
                     indexRecords *MetaIndexParser)
 -      : Item(Owner, ExpectedHashes), Target(Target), 
 +      : Item(Owner, ExpectedHashes, TransactionManager), Target(Target), 
          MetaIndexParser(MetaIndexParser) {};
 -
  };
                                                                        /*}}}*/
  /** \brief An item that is responsible for fetching an index file of  {{{
   */
  class pkgAcqDiffIndex : public pkgAcqBaseIndex
  {
 +   void *d;
 +
   protected:
     /** \brief If \b true, debugging information will be written to std::clog. */
     bool Debug;
      */
     std::string Description;
  
 +   /** \brief If the copy step of the packages file is done
 +    */
 +   bool PackagesFileReadyInPartial;
 +
   public:
     // Specialized action members
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
-    virtual std::string DescURI() const {return RealURI + "Index";};
+    virtual std::string DescURI() {return RealURI + "Index";};
     virtual std::string Custom600Headers() const;
  
     /** \brief Parse the Index file for a set of Packages diffs.
      *  \param ExpectedHashes The list file's hashsums which are expected.
      */
     pkgAcqDiffIndex(pkgAcquire *Owner,
 +                   pkgAcqMetaBase *TransactionManager,
                     struct IndexTarget const * const Target,
                     HashStringList const &ExpectedHashes,
                     indexRecords *MetaIndexParser);
   */
  class pkgAcqIndexMergeDiffs : public pkgAcqBaseIndex
  {
 +   void *d;
 +
     protected:
  
     /** \brief If \b true, debugging output will be written to
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
         pkgAcquire::MethodConfig *Cnf);
-    virtual std::string DescURI() const {return RealURI + "Index";};
+    virtual std::string DescURI() {return RealURI + "Index";};
  
     /** \brief Create an index merge-diff item.
      *
      *  check if it was the last one to complete the download step
      */
     pkgAcqIndexMergeDiffs(pkgAcquire *Owner,
 +                         pkgAcqMetaBase *TransactionManager,
                           struct IndexTarget const * const Target,
                           HashStringList const &ExpectedHash,
                           indexRecords *MetaIndexParser,
   */
  class pkgAcqIndexDiffs : public pkgAcqBaseIndex
  {
 +   void *d;
 +
     private:
  
     /** \brief Queue up the next diff download.
  
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
-    virtual std::string DescURI() const {return RealURI + "IndexDiffs";};
 -   virtual std::string DescURI() {return RealURI + "Index";};
++   virtual std::string DescURI() {return RealURI + "IndexDiffs";};
  
     /** \brief Create an index diff item.
      *
      *  that depends on it.
      */
     pkgAcqIndexDiffs(pkgAcquire *Owner,
 +                    pkgAcqMetaBase *TransactionManager,
                      struct IndexTarget const * const Target,
                      HashStringList const &ExpectedHash,
                      indexRecords *MetaIndexParser,
   */
  class pkgAcqIndex : public pkgAcqBaseIndex
  {
 +   void *d;
 +
     protected:
  
     /** \brief If \b true, the index file has been decompressed. */
      */
     bool Erase;
  
 -   // Unused, used to be used to verify that "Packages: " header was there
 -   bool __DELME_ON_NEXT_ABI_BREAK_Verify;
 -
     /** \brief The object that is actually being fetched (minus any
      *  compression-related extensions).
      */
     /** \brief Do the changes needed to fetch via AptByHash (if needed) */
     void InitByHashIfNeeded(const std::string MetaKey);
  
 -   /** \brief Get the full pathname of the final file for the given URI
 +   /** \brief Auto select the right compression to use */
 +   void AutoSelectCompression();
 +
 +   /** \brief Get the full pathname of the final file for the current URI
      */
 -   std::string GetFinalFilename(std::string const &URI,
 -                                std::string const &compExt);
 +   std::string GetFinalFilename() const;
  
     /** \brief Schedule file for verification after a IMS hit */
 -   void ReverifyAfterIMS(std::string const &FileName);
 +   void ReverifyAfterIMS();
  
     public:
     
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
     virtual std::string Custom600Headers() const;
-    virtual std::string DescURI() const {return Desc.URI;};
+    virtual std::string DescURI() {return Desc.URI;};
  
     /** \brief Create a pkgAcqIndex.
      *
      *  fallback is ".gz" or none.
      */
     pkgAcqIndex(pkgAcquire *Owner,std::string URI,std::string URIDesc,
 -             std::string ShortDesc, HashStringList const &ExpectedHashes,
 -             std::string compressExt="");
 -   pkgAcqIndex(pkgAcquire *Owner,
 +             std::string ShortDesc, HashStringList const &ExpectedHashes);
 +   pkgAcqIndex(pkgAcquire *Owner, pkgAcqMetaBase *TransactionManager,
                 IndexTarget const * const Target,
                 HashStringList const &ExpectedHash,
                 indexRecords *MetaIndexParser);
 +               
     void Init(std::string const &URI, std::string const &URIDesc,
               std::string const &ShortDesc);
  };
   */
  class pkgAcqIndexTrans : public pkgAcqIndex
  {
 +   void *d;
 +
     public:
    
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
      *
      *  \param ShortDesc A brief description of this index file.
      */
 -   pkgAcqIndexTrans(pkgAcquire *Owner,std::string URI,std::string URIDesc,
 +   pkgAcqIndexTrans(pkgAcquire *Owner,
 +                    std::string URI,std::string URIDesc,
                    std::string ShortDesc);
 -   pkgAcqIndexTrans(pkgAcquire *Owner, IndexTarget const * const Target,
 -                  HashStringList const &ExpectedHashes, indexRecords *MetaIndexParser);
 +   pkgAcqIndexTrans(pkgAcquire *Owner,
 +                    pkgAcqMetaBase *TransactionManager,
 +                    IndexTarget const * const Target,
 +                    HashStringList const &ExpectedHashes,
 +                    indexRecords *MetaIndexParser);
  };
                                                                        /*}}}*/
  /** \brief Information about an index file. */                                /*{{{*/
  class IndexTarget
  {
 +   void *d;
 +
   public:
     /** \brief A URI from which the index file can be downloaded. */
     std::string URI;
     virtual bool IsOptional() const {
        return false;
     }
 -   virtual bool IsSubIndex() const {
 -      return false;
 -   }
  };
                                                                        /*}}}*/
  /** \brief Information about an optional index file. */                       /*{{{*/
  class OptionalIndexTarget : public IndexTarget
  {
 +   void *d;
 +
     virtual bool IsOptional() const {
        return true;
     }
  };
                                                                        /*}}}*/
 -/** \brief Information about an subindex index file. */                       /*{{{*/
 -class SubIndexTarget : public IndexTarget
 -{
 -   virtual bool IsSubIndex() const {
 -      return true;
 -   }
 -};
 -                                                                      /*}}}*/
 -/** \brief Information about an subindex index file. */                       /*{{{*/
 -class OptionalSubIndexTarget : public OptionalIndexTarget
 -{
 -   virtual bool IsSubIndex() const {
 -      return true;
 -   }
 -};
 -                                                                      /*}}}*/
 -
 -/** \brief An acquire item that downloads the detached signature      {{{
 - *  of a meta-index (Release) file, then queues up the release
 - *  file itself.
 - *
 - *  \todo Why protected members?
 - *
 - *  \sa pkgAcqMetaIndex
 - */
 -class pkgAcqMetaSig : public pkgAcquire::Item
 -{
 -   protected:
 -   /** \brief The last good signature file */
 -   std::string LastGoodSig;
 -
 -   /** \brief The URI of the signature file.  Unlike Desc.URI, this is
 -    *  never modified; it is used to determine the file that is being
 -    *  downloaded.
 -    */
 -   std::string RealURI;
 -
 -   /** \brief The URI of the meta-index file to be fetched after the signature. */
 -   std::string MetaIndexURI;
 -
 -   /** \brief A "URI-style" description of the meta-index file to be
 -    *  fetched after the signature.
 -    */
 -   std::string MetaIndexURIDesc;
 -
 -   /** \brief A brief description of the meta-index file to be fetched
 -    *  after the signature.
 -    */
 -   std::string MetaIndexShortDesc;
 -
 -   /** \brief A package-system-specific parser for the meta-index file. */
 -   indexRecords* MetaIndexParser;
 -
 -   /** \brief The index files which should be looked up in the meta-index
 -    *  and then downloaded.
 -    *
 -    *  \todo Why a list of pointers instead of a list of structs?
 -    */
 -   const std::vector<IndexTarget*>* IndexTargets;
 -
 -   public:
 -   
 -   // Specialized action members
 -   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 -                   pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
 -   virtual std::string DescURI() {return RealURI; };
 -
 -   /** \brief Create a new pkgAcqMetaSig. */
 -   pkgAcqMetaSig(pkgAcquire *Owner,std::string URI,std::string URIDesc, std::string ShortDesc,
 -               std::string MetaIndexURI, std::string MetaIndexURIDesc, std::string MetaIndexShortDesc,
 -               const std::vector<IndexTarget*>* IndexTargets,
 -               indexRecords* MetaIndexParser);
 -   virtual ~pkgAcqMetaSig();
 -};
 -                                                                      /*}}}*/
 -/** \brief An item that is responsible for downloading the meta-index {{{
 - *  file (i.e., Release) itself and verifying its signature.
 - *
 - *  Once the download and verification are complete, the downloads of
 - *  the individual index files are queued up using pkgAcqDiffIndex.
 - *  If the meta-index file had a valid signature, the expected hashsums
 - *  of the index files will be the md5sums listed in the meta-index;
 - *  otherwise, the expected hashsums will be "" (causing the
 - *  authentication of the index files to be bypassed).
 - */
 -class pkgAcqMetaIndex : public pkgAcquire::Item
 -{
 -   protected:
 -   /** \brief The URI that is actually being downloaded; never
 -    *  modified by pkgAcqMetaIndex.
 -    */
 -   std::string RealURI;
 -
 -   /** \brief The file in which the signature for this index was stored.
 -    *
 -    *  If empty, the signature and the md5sums of the individual
 -    *  indices will not be checked.
 -    */
 -   std::string SigFile;
 -
 -   /** \brief The index files to download. */
 -   const std::vector<IndexTarget*>* IndexTargets;
 -
 -   /** \brief The parser for the meta-index file. */
 -   indexRecords* MetaIndexParser;
 -
 -   /** \brief If \b true, the index's signature is currently being verified.
 -    */
 -   bool AuthPass;
 -   // required to deal gracefully with problems caused by incorrect ims hits
 -   bool IMSHit; 
 -
 -   /** \brief Check that the release file is a release file for the
 -    *  correct distribution.
 -    *
 -    *  \return \b true if no fatal errors were encountered.
 -    */
 -   bool VerifyVendor(std::string Message);
 -
 -   /** \brief Called when a file is finished being retrieved.
 -    *
 -    *  If the file was not downloaded to DestFile, a copy process is
 -    *  set up to copy it to DestFile; otherwise, Complete is set to \b
 -    *  true and the file is moved to its final location.
 -    *
 -    *  \param Message The message block received from the fetch
 -    *  subprocess.
 -    */
 -   void RetrievalDone(std::string Message);
 -
 -   /** \brief Called when authentication succeeded.
 -    *
 -    *  Sanity-checks the authenticated file, queues up the individual
 -    *  index files for download, and saves the signature in the lists
 -    *  directory next to the authenticated list file.
 -    *
 -    *  \param Message The message block received from the fetch
 -    *  subprocess.
 -    */
 -   void AuthDone(std::string Message);
 -
 -   /** \brief Starts downloading the individual index files.
 -    *
 -    *  \param verify If \b true, only indices whose expected hashsum
 -    *  can be determined from the meta-index will be downloaded, and
 -    *  the hashsums of indices will be checked (reporting
 -    *  #StatAuthError if there is a mismatch).  If verify is \b false,
 -    *  no hashsum checking will be performed.
 -    */
 -   void QueueIndexes(bool verify);
 -   
 -   public:
 -   
 -   // Specialized action members
 -   virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
 -                   pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
 -   virtual std::string DescURI() {return RealURI; };
 -
 -   /** \brief Create a new pkgAcqMetaIndex. */
 -   pkgAcqMetaIndex(pkgAcquire *Owner,
 -                 std::string URI,std::string URIDesc, std::string ShortDesc,
 -                 std::string SigFile,
 -                 const std::vector<IndexTarget*>* IndexTargets,
 -                 indexRecords* MetaIndexParser);
 -};
 -                                                                      /*}}}*/
 -/** \brief An item repsonsible for downloading clearsigned metaindexes        {{{*/
 -class pkgAcqMetaClearSig : public pkgAcqMetaIndex
 -{
 -   /** \brief The URI of the meta-index file for the detached signature */
 -   std::string MetaIndexURI;
 -
 -   /** \brief A "URI-style" description of the meta-index file */
 -   std::string MetaIndexURIDesc;
 -
 -   /** \brief A brief description of the meta-index file */
 -   std::string MetaIndexShortDesc;
 -
 -   /** \brief The URI of the detached meta-signature file if the clearsigned one failed. */
 -   std::string MetaSigURI;
 -
 -   /** \brief A "URI-style" description of the meta-signature file */
 -   std::string MetaSigURIDesc;
 -
 -   /** \brief A brief description of the meta-signature file */
 -   std::string MetaSigShortDesc;
 -
 -public:
 -   void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
 -   virtual std::string Custom600Headers() const;
--
 -   /** \brief Create a new pkgAcqMetaClearSig. */
 -   pkgAcqMetaClearSig(pkgAcquire *Owner,
 -              std::string const &URI, std::string const &URIDesc, std::string const &ShortDesc,
 -              std::string const &MetaIndexURI, std::string const &MetaIndexURIDesc, std::string const &MetaIndexShortDesc,
 -              std::string const &MetaSigURI, std::string const &MetaSigURIDesc, std::string const &MetaSigShortDesc,
 -              const std::vector<IndexTarget*>* IndexTargets,
 -              indexRecords* MetaIndexParser);
 -   virtual ~pkgAcqMetaClearSig();
 -};
 -                                                                      /*}}}*/
  /** \brief An item that is responsible for fetching a package file.   {{{
   *
   *  If the package file already exists in the cache, nothing will be
   */
  class pkgAcqArchive : public pkgAcquire::Item
  {
 +   void *d;
 +
     protected:
     /** \brief The package version being fetched. */
     pkgCache::VerIterator Version;
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &Hashes,
                     pkgAcquire::MethodConfig *Cnf);
-    virtual std::string DescURI() const {return Desc.URI;};
-    virtual std::string ShortDesc() const {return Desc.ShortDesc;};
+    virtual std::string DescURI() {return Desc.URI;};
+    virtual std::string ShortDesc() {return Desc.ShortDesc;};
     virtual void Finished();
     virtual bool IsTrusted() const;
     
   */
  class pkgAcqFile : public pkgAcquire::Item
  {
 +   void *d;
 +
     /** \brief How many times to retry the download, set from
      *  Acquire::Retries.
      */
     virtual void Failed(std::string Message,pkgAcquire::MethodConfig *Cnf);
     virtual void Done(std::string Message,unsigned long long Size, HashStringList const &CalcHashes,
                     pkgAcquire::MethodConfig *Cnf);
-    virtual std::string DescURI() const {return Desc.URI;};
+    virtual std::string DescURI() {return Desc.URI;};
     virtual std::string Custom600Headers() const;
  
     /** \brief Create a new pkgAcqFile object.
diff --combined apt-pkg/acquire.cc
index 9060d492b8d5bce46a98d795219455b8bb05b3d2,8119e4487603553040b122f9ad5837e44f8cb91a..ec565fcfa6dcdbaea5ed21faa94057e89aeb2179
@@@ -37,7 -37,6 +37,7 @@@
  #include <sys/time.h>
  #include <sys/select.h>
  #include <errno.h>
 +#include <sys/stat.h>
  
  #include <apti18n.h>
                                                                        /*}}}*/
@@@ -74,23 -73,27 +74,27 @@@ pkgAcquire::pkgAcquire(pkgAcquireStatu
  // ---------------------------------------------------------------------
  /* Do everything needed to be a complete Acquire object and report the
     success (or failure) back so the user knows that something is wrong… */
- bool pkgAcquire::Setup(pkgAcquireStatus *Progress, string const &Lock)
+ bool pkgAcquire::Setup(pkgAcquireStatus *Progress, string const &Lock,
+       bool const createDirectories)
  {
     Log = Progress;
  
     // check for existence and possibly create auxiliary directories
-    string const listDir = _config->FindDir("Dir::State::lists");
-    string const partialListDir = listDir + "partial/";
-    string const archivesDir = _config->FindDir("Dir::Cache::Archives");
-    string const partialArchivesDir = archivesDir + "partial/";
-    if (CreateAPTDirectoryIfNeeded(_config->FindDir("Dir::State"), partialListDir) == false &&
-        CreateAPTDirectoryIfNeeded(listDir, partialListDir) == false)
-       return _error->Errno("Acquire", _("List directory %spartial is missing."), listDir.c_str());
-    if (CreateAPTDirectoryIfNeeded(_config->FindDir("Dir::Cache"), partialArchivesDir) == false &&
-        CreateAPTDirectoryIfNeeded(archivesDir, partialArchivesDir) == false)
-       return _error->Errno("Acquire", _("Archives directory %spartial is missing."), archivesDir.c_str());
+    if (createDirectories == true)
+    {
+       string const listDir = _config->FindDir("Dir::State::lists");
+       string const partialListDir = listDir + "partial/";
+       string const archivesDir = _config->FindDir("Dir::Cache::Archives");
+       string const partialArchivesDir = archivesDir + "partial/";
+       if (CreateAPTDirectoryIfNeeded(_config->FindDir("Dir::State"), partialListDir) == false &&
+           CreateAPTDirectoryIfNeeded(listDir, partialListDir) == false)
+        return _error->Errno("Acquire", _("List directory %spartial is missing."), listDir.c_str());
+       if (CreateAPTDirectoryIfNeeded(_config->FindDir("Dir::Cache"), partialArchivesDir) == false &&
+           CreateAPTDirectoryIfNeeded(archivesDir, partialArchivesDir) == false)
+        return _error->Errno("Acquire", _("Archives directory %spartial is missing."), archivesDir.c_str());
+    }
  
     if (Lock.empty() == true || _config->FindB("Debug::NoLocking", false) == true)
        return true;
@@@ -582,27 -585,18 +586,18 @@@ pkgAcquire::UriIterator pkgAcquire::Uri
  // Acquire::MethodConfig::MethodConfig - Constructor                  /*{{{*/
  // ---------------------------------------------------------------------
  /* */
- pkgAcquire::MethodConfig::MethodConfig()
+ pkgAcquire::MethodConfig::MethodConfig() : d(NULL), Next(0), SingleInstance(false),
+    Pipeline(false), SendConfig(false), LocalOnly(false), NeedsCleanup(false),
+    Removable(false)
  {
-    SingleInstance = false;
-    Pipeline = false;
-    SendConfig = false;
-    LocalOnly = false;
-    Removable = false;
-    Next = 0;
  }
                                                                        /*}}}*/
  // Queue::Queue - Constructor                                         /*{{{*/
  // ---------------------------------------------------------------------
  /* */
- pkgAcquire::Queue::Queue(string Name,pkgAcquire *Owner) : Name(Name), 
-             Owner(Owner)
+ pkgAcquire::Queue::Queue(string Name,pkgAcquire *Owner) : d(NULL), Next(0),
+    Name(Name), Items(0), Workers(0), Owner(Owner), PipeDepth(0), MaxPipeDepth(1)
  {
-    Items = 0;
-    Next = 0;
-    Workers = 0;
-    MaxPipeDepth = 1;
-    PipeDepth = 0;
  }
                                                                        /*}}}*/
  // Queue::~Queue - Destructor                                         /*{{{*/
@@@ -806,7 -800,7 +801,7 @@@ void pkgAcquire::Queue::Bump(
  // AcquireStatus::pkgAcquireStatus - Constructor                      /*{{{*/
  // ---------------------------------------------------------------------
  /* */
- pkgAcquireStatus::pkgAcquireStatus() : d(NULL), Update(true), MorePulses(false)
+ pkgAcquireStatus::pkgAcquireStatus() : d(NULL), Percent(0), Update(true), MorePulses(false)
  {
     Start();
  }
diff --combined methods/copy.cc
index faf330ace57abf56a7b4579f6eae1e036f51bc07,f542a27c0b2b34b8ee03e64ef5d2b3c72351dd8c..a23c0316c4717568950cacf3aebee238c4fdde30
@@@ -37,15 -37,12 +37,12 @@@ class CopyMethod : public pkgAcqMetho
  
  void CopyMethod::CalculateHashes(FetchResult &Res)
  {
-    // For gzip indexes we need to look inside the gzip for the hash
-    // We can not use the extension here as its not used in partial 
-    // on a IMS hit
-    FileFd::OpenMode OpenMode = FileFd::ReadOnly;
+    Hashes Hash;
+    FileFd::CompressMode CompressMode = FileFd::None;
     if (_config->FindB("Acquire::GzipIndexes", false) == true)
-       OpenMode = FileFd::ReadOnlyGzip;
+       CompressMode = FileFd::Extension;
  
-    Hashes Hash;
-    FileFd Fd(Res.Filename, OpenMode);
+    FileFd Fd(Res.Filename, FileFd::ReadOnly, CompressMode);
     Hash.AddFD(Fd);
     Res.TakeHashes(Hash);
  }
@@@ -70,14 -67,6 +67,14 @@@ bool CopyMethod::Fetch(FetchItem *Itm
     Res.LastModified = Buf.st_mtime;
     Res.IMSHit = false;      
     URIStart(Res);
 +
 +   // when the files are identical, just compute the hashes
 +   if(File == Itm->DestFile)
 +   {
 +      CalculateHashes(Res);
 +      URIDone(Res);
 +      return true;
 +   }
     
     // just calc the hashes if the source and destination are identical
     if (File == Itm->DestFile)
@@@ -128,6 -117,5 +125,5 @@@ int main(
  
     CopyMethod Mth;
  
-    Mth.DropPrivsOrDie();
     return Mth.Run();
  }
diff --combined methods/gpgv.cc
index 72e4c798706595ee97968f918bb700a64c81a9f4,02fb8c356fcc14d96df066237acfb07584b0b888..7e8500c51b865f74bf230bc9881582f653ee2d8a
@@@ -75,7 -75,7 +75,7 @@@ string GPGVMethod::VerifyGetSigners(con
  
     FILE *pipein = fdopen(fd[0], "r");
  
-    // Loop over the output of gpgv, and check the signatures.
+    // Loop over the output of apt-key (which really is gnupg), and check the signatures.
     size_t buffersize = 64;
     char *buffer = (char *) malloc(buffersize);
     size_t bufferoff = 0;
     waitpid(pid, &status, 0);
     if (Debug == true)
     {
 -      std::clog << "apt-key exited\n";
 +      ioprintf(std::clog, "gpgv exited with status %i\n", WEXITSTATUS(status));
     }
     
     if (WEXITSTATUS(status) == 0)
     else if (WEXITSTATUS(status) == 1)
        return _("At least one invalid signature was encountered.");
     else if (WEXITSTATUS(status) == 111)
-       return _("Could not execute 'gpgv' to verify signature (is gpgv installed?)");
+       return _("Could not execute 'apt-key' to verify signature (is gnupg installed?)");
     else if (WEXITSTATUS(status) == 112)
     {
        // acquire system checks for "NODATA" to generate GPG errors (the others are only warnings)
        return errmsg;
     }
     else
-       return _("Unknown error executing gpgv");
+       return _("Unknown error executing apt-key");
  }
  
  bool GPGVMethod::Fetch(FetchItem *Itm)
     Res.Filename = Itm->DestFile;
     URIStart(Res);
  
-    // Run gpgv on file, extract contents and get the key ID of the signer
+    // Run apt-key on file, extract contents and get the key ID of the signer
     string msg = VerifyGetSigners(Path.c_str(), Itm->DestFile.c_str(),
                                   GoodSigners, BadSigners, WorthlessSigners,
                                   NoPubKeySigners);
  
     if (_config->FindB("Debug::Acquire::gpgv", false))
     {
-       std::clog << "gpgv succeeded\n";
+       std::clog << "apt-key succeeded\n";
     }
  
     return true;