Merge branch 'feature/popen' into feature/apt-install-deb

[apt.git] / cmdline / apt-cache.cc
diff --git a/cmdline/apt-cache.cc b/cmdline/apt-cache.cc

index 07b95e3ca81bc90ca9c25cba4ce52ae658c39752..84b775390b31799d785439afbed07f370ae45514 100644 (file)
--- a/cmdline/apt-cache.cc
+++ b/cmdline/apt-cache.cc
@@ -13,43 +13,61 @@
     ##################################################################### */
                                                                         /*}}}*/
  // Include Files                                                       /*{{{*/
     ##################################################################### */
                                                                         /*}}}*/
  // Include Files                                                       /*{{{*/
+#include<config.h>
+
+#include <apt-pkg/algorithms.h>
+#include <apt-pkg/cachefile.h>
+#include <apt-pkg/cacheset.h>
+#include <apt-pkg/cmndline.h>
  #include <apt-pkg/error.h>
  #include <apt-pkg/error.h>
-#include <apt-pkg/pkgcachegen.h>
+#include <apt-pkg/fileutl.h>
+#include <apt-pkg/indexfile.h>
  #include <apt-pkg/init.h>
  #include <apt-pkg/init.h>
+#include <apt-pkg/metaindex.h>
+#include <apt-pkg/pkgrecords.h>
+#include <apt-pkg/pkgsystem.h>
+#include <apt-pkg/policy.h>
  #include <apt-pkg/progress.h>
  #include <apt-pkg/sourcelist.h>
  #include <apt-pkg/progress.h>
  #include <apt-pkg/sourcelist.h>
-#include <apt-pkg/cmndline.h>
-#include <apt-pkg/strutl.h>
-#include <apt-pkg/pkgrecords.h>
+#include <apt-pkg/sptr.h>
  #include <apt-pkg/srcrecords.h>
  #include <apt-pkg/srcrecords.h>
-#include <apt-pkg/version.h>
-#include <apt-pkg/policy.h>
+#include <apt-pkg/strutl.h>
  #include <apt-pkg/tagfile.h>
  #include <apt-pkg/tagfile.h>
-#include <apt-pkg/algorithms.h>
-#include <apt-pkg/sptr.h>
+#include <apt-pkg/version.h>
+#include <apt-pkg/cacheiterators.h>
+#include <apt-pkg/configuration.h>
+#include <apt-pkg/depcache.h>
+#include <apt-pkg/macros.h>
+#include <apt-pkg/mmap.h>
+#include <apt-pkg/pkgcache.h>
  
  
-#include <config.h>
-#include <apti18n.h>
+#include <apt-private/private-cacheset.h>
+#include <apt-private/private-cmndline.h>
  
  
-#include <locale.h>
-#include <iostream>
-#include <unistd.h>
-#include <errno.h>
  #include <regex.h>
  #include <regex.h>
+#include <stddef.h>
  #include <stdio.h>
  #include <stdio.h>
-
+#include <stdlib.h>
+#include <unistd.h>
+#include <algorithm>
+#include <cstring>
  #include <iomanip>
  #include <iomanip>
+#include <iostream>
+#include <list>
+#include <map>
+#include <set>
+#include <string>
+#include <vector>
+
+#include <apti18n.h>
                                                                         /*}}}*/
  
  using namespace std;
  
                                                                         /*}}}*/
  
  using namespace std;
  
-pkgCache *GCache = 0;
-pkgSourceList *SrcList = 0;
-
  // LocalitySort - Sort a version list by package file locality         /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // LocalitySort - Sort a version list by package file locality         /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-int LocalityCompare(const void *a, const void *b)
+static int LocalityCompare(const void *a, const void *b)
  {
     pkgCache::VerFile *A = *(pkgCache::VerFile **)a;
     pkgCache::VerFile *B = *(pkgCache::VerFile **)b;
  {
     pkgCache::VerFile *A = *(pkgCache::VerFile **)a;
     pkgCache::VerFile *B = *(pkgCache::VerFile **)b;
@@ -66,13 +84,13 @@ int LocalityCompare(const void *a, const void *b)
     return A->File - B->File;
  }
  
     return A->File - B->File;
  }
  
-void LocalitySort(pkgCache::VerFile **begin,
+static void LocalitySort(pkgCache::VerFile **begin,
                   unsigned long Count,size_t Size)
  {   
     qsort(begin,Count,Size,LocalityCompare);
  }
  
                   unsigned long Count,size_t Size)
  {   
     qsort(begin,Count,Size,LocalityCompare);
  }
  
-void LocalitySort(pkgCache::DescFile **begin,
+static void LocalitySort(pkgCache::DescFile **begin,
                   unsigned long Count,size_t Size)
  {   
     qsort(begin,Count,Size,LocalityCompare);
                   unsigned long Count,size_t Size)
  {   
     qsort(begin,Count,Size,LocalityCompare);
@@ -81,15 +99,8 @@ void LocalitySort(pkgCache::DescFile **begin,
  // UnMet - Show unmet dependencies                                     /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // UnMet - Show unmet dependencies                                     /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool UnMet(CommandLine &CmdL)
+static bool ShowUnMet(pkgCache::VerIterator const &V, bool const Important)
  {
  {
-   pkgCache &Cache = *GCache;
-   bool Important = _config->FindB("APT::Cache::Important",false);
-   
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
-   {
-      for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; V++)
-      {
          bool Header = false;
          for (pkgCache::DepIterator D = V.DependsList(); D.end() == false;)
          {
          bool Header = false;
          for (pkgCache::DepIterator D = V.DependsList(); D.end() == false;)
          {
@@ -97,20 +108,17 @@ bool UnMet(CommandLine &CmdL)
             pkgCache::DepIterator Start;
             pkgCache::DepIterator End;
             D.GlobOr(Start,End);
             pkgCache::DepIterator Start;
             pkgCache::DepIterator End;
             D.GlobOr(Start,End);
-           
-           // Skip conflicts and replaces
-           if (End->Type != pkgCache::Dep::PreDepends &&
-               End->Type != pkgCache::Dep::Depends && 
-               End->Type != pkgCache::Dep::Suggests &&
-               End->Type != pkgCache::Dep::Recommends)
-              continue;
  
             // Important deps only
             if (Important == true)
                if (End->Type != pkgCache::Dep::PreDepends &&
                    End->Type != pkgCache::Dep::Depends)
                   continue;
  
             // Important deps only
             if (Important == true)
                if (End->Type != pkgCache::Dep::PreDepends &&
                    End->Type != pkgCache::Dep::Depends)
                   continue;
-           
+
+           // Skip conflicts and replaces
+           if (End.IsNegative() == true)
+              continue;
+
             // Verify the or group
             bool OK = false;
             pkgCache::DepIterator RealStart = Start;
             // Verify the or group
             bool OK = false;
             pkgCache::DepIterator RealStart = Start;
@@ -128,7 +136,7 @@ bool UnMet(CommandLine &CmdL)
                
                if (Start == End)
                   break;
                
                if (Start == End)
                   break;
-              Start++;
+              ++Start;
             }
             while (1);
  
             }
             while (1);
  
@@ -139,7 +147,7 @@ bool UnMet(CommandLine &CmdL)
             // Oops, it failed..
             if (Header == false)
                ioprintf(cout,_("Package %s version %s has an unmet dep:\n"),
             // Oops, it failed..
             if (Header == false)
                ioprintf(cout,_("Package %s version %s has an unmet dep:\n"),
-                       P.FullName(true).c_str(),V.VerStr());
+                       V.ParentPkg().FullName(true).c_str(),V.VerStr());
             Header = true;
             
             // Print out the dep type
             Header = true;
             
             // Print out the dep type
@@ -156,41 +164,61 @@ bool UnMet(CommandLine &CmdL)
                if (Start == End)
                   break;
                cout << " | ";
                if (Start == End)
                   break;
                cout << " | ";
-              Start++;
+              ++Start;
             }
             while (1);
             
             cout << endl;
             }
             while (1);
             
             cout << endl;
-        }       
-      }
-   }   
+        }
+   return true;
+}
+static bool UnMet(CommandLine &CmdL)
+{
+   bool const Important = _config->FindB("APT::Cache::Important",false);
+
+   pkgCacheFile CacheFile;
+   if (unlikely(CacheFile.GetPkgCache() == NULL))
+      return false;
+
+   if (CmdL.FileSize() <= 1)
+   {
+      for (pkgCache::PkgIterator P = CacheFile.GetPkgCache()->PkgBegin(); P.end() == false; ++P)
+        for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; ++V)
+           if (ShowUnMet(V, Important) == false)
+              return false;
+   }
+   else
+   {
+      CacheSetHelperVirtuals helper(true, GlobalError::NOTICE);
+      APT::VersionList verset = APT::VersionList::FromCommandLine(CacheFile, CmdL.FileList + 1,
+                               APT::VersionList::CANDIDATE, helper);
+      for (APT::VersionList::iterator V = verset.begin(); V != verset.end(); ++V)
+        if (ShowUnMet(V, Important) == false)
+           return false;
+   }
     return true;
  }
                                                                         /*}}}*/
  // DumpPackage - Show a dump of a package record                       /*{{{*/
  // ---------------------------------------------------------------------
  /* */
     return true;
  }
                                                                         /*}}}*/
  // DumpPackage - Show a dump of a package record                       /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool DumpPackage(CommandLine &CmdL)
-{   
-   pkgCache &Cache = *GCache;
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
-   {
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(*I);
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
+static bool DumpPackage(CommandLine &CmdL)
+{
+   pkgCacheFile CacheFile;
+   APT::CacheSetHelper helper(true, GlobalError::NOTICE);
+   APT::PackageList pkgset = APT::PackageList::FromCommandLine(CacheFile, CmdL.FileList + 1, helper);
  
  
+   for (APT::PackageList::const_iterator Pkg = pkgset.begin(); Pkg != pkgset.end(); ++Pkg)
+   {
        cout << "Package: " << Pkg.FullName(true) << endl;
        cout << "Versions: " << endl;
        cout << "Package: " << Pkg.FullName(true) << endl;
        cout << "Versions: " << endl;
-      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; Cur++)
+      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; ++Cur)
        {
          cout << Cur.VerStr();
        {
          cout << Cur.VerStr();
-        for (pkgCache::VerFileIterator Vf = Cur.FileList(); Vf.end() == false; Vf++)
+        for (pkgCache::VerFileIterator Vf = Cur.FileList(); Vf.end() == false; ++Vf)
             cout << " (" << Vf.File().FileName() << ")";
          cout << endl;
             cout << " (" << Vf.File().FileName() << ")";
          cout << endl;
-        for (pkgCache::DescIterator D = Cur.DescriptionList(); D.end() == false; D++)
+        for (pkgCache::DescIterator D = Cur.DescriptionList(); D.end() == false; ++D)
          {
             cout << " Description Language: " << D.LanguageCode() << endl
                  << "                 File: " << D.FileList().File().FileName() << endl
          {
             cout << " Description Language: " << D.LanguageCode() << endl
                  << "                 File: " << D.FileList().File().FileName() << endl
@@ -202,7 +230,7 @@ bool DumpPackage(CommandLine &CmdL)
        cout << endl;
        
        cout << "Reverse Depends: " << endl;
        cout << endl;
        
        cout << "Reverse Depends: " << endl;
-      for (pkgCache::DepIterator D = Pkg.RevDependsList(); D.end() != true; D++)
+      for (pkgCache::DepIterator D = Pkg.RevDependsList(); D.end() != true; ++D)
        {
          cout << "  " << D.ParentPkg().FullName(true) << ',' << D.TargetPkg().FullName(true);
          if (D->Version != 0)
        {
          cout << "  " << D.ParentPkg().FullName(true) << ',' << D.TargetPkg().FullName(true);
          if (D->Version != 0)
@@ -212,24 +240,24 @@ bool DumpPackage(CommandLine &CmdL)
        }
        
        cout << "Dependencies: " << endl;
        }
        
        cout << "Dependencies: " << endl;
-      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; Cur++)
+      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; ++Cur)
        {
          cout << Cur.VerStr() << " - ";
        {
          cout << Cur.VerStr() << " - ";
-        for (pkgCache::DepIterator Dep = Cur.DependsList(); Dep.end() != true; Dep++)
+        for (pkgCache::DepIterator Dep = Cur.DependsList(); Dep.end() != true; ++Dep)
             cout << Dep.TargetPkg().FullName(true) << " (" << (int)Dep->CompareOp << " " << DeNull(Dep.TargetVer()) << ") ";
          cout << endl;
        }      
  
        cout << "Provides: " << endl;
             cout << Dep.TargetPkg().FullName(true) << " (" << (int)Dep->CompareOp << " " << DeNull(Dep.TargetVer()) << ") ";
          cout << endl;
        }      
  
        cout << "Provides: " << endl;
-      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; Cur++)
+      for (pkgCache::VerIterator Cur = Pkg.VersionList(); Cur.end() != true; ++Cur)
        {
          cout << Cur.VerStr() << " - ";
        {
          cout << Cur.VerStr() << " - ";
-        for (pkgCache::PrvIterator Prv = Cur.ProvidesList(); Prv.end() != true; Prv++)
+        for (pkgCache::PrvIterator Prv = Cur.ProvidesList(); Prv.end() != true; ++Prv)
             cout << Prv.ParentPkg().FullName(true) << " ";
          cout << endl;
        }
        cout << "Reverse Provides: " << endl;
             cout << Prv.ParentPkg().FullName(true) << " ";
          cout << endl;
        }
        cout << "Reverse Provides: " << endl;
-      for (pkgCache::PrvIterator Prv = Pkg.ProvidesList(); Prv.end() != true; Prv++)
+      for (pkgCache::PrvIterator Prv = Pkg.ProvidesList(); Prv.end() != true; ++Prv)
          cout << Prv.OwnerPkg().FullName(true) << " " << Prv.OwnerVer().VerStr() << endl;
     }
  
          cout << Prv.OwnerPkg().FullName(true) << " " << Prv.OwnerVer().VerStr() << endl;
     }
  
@@ -239,19 +267,25 @@ bool DumpPackage(CommandLine &CmdL)
  // Stats - Dump some nice statistics                                   /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // Stats - Dump some nice statistics                                   /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool Stats(CommandLine &Cmd)
+static bool Stats(CommandLine &)
  {
  {
-   pkgCache &Cache = *GCache;
-   cout << _("Total package names: ") << Cache.Head().PackageCount << " (" <<
-      SizeToStr(Cache.Head().PackageCount*Cache.Head().PackageSz) << ')' << endl;
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
+   cout << _("Total package names: ") << Cache->Head().GroupCount << " (" <<
+      SizeToStr(Cache->Head().GroupCount*Cache->Head().GroupSz) << ')' << endl
+        << _("Total package structures: ") << Cache->Head().PackageCount << " (" <<
+      SizeToStr(Cache->Head().PackageCount*Cache->Head().PackageSz) << ')' << endl;
  
     int Normal = 0;
     int Virtual = 0;
     int NVirt = 0;
     int DVirt = 0;
     int Missing = 0;
  
     int Normal = 0;
     int Virtual = 0;
     int NVirt = 0;
     int DVirt = 0;
     int Missing = 0;
-   pkgCache::PkgIterator I = Cache.PkgBegin();
-   for (;I.end() != true; I++)
+   pkgCache::PkgIterator I = Cache->PkgBegin();
+   for (;I.end() != true; ++I)
     {
        if (I->VersionList != 0 && I->ProvidesList == 0)
        {
     {
        if (I->VersionList != 0 && I->ProvidesList == 0)
        {
@@ -288,37 +322,37 @@ bool Stats(CommandLine &Cmd)
     cout << _("  Mixed virtual packages: ") << NVirt << endl;
     cout << _("  Missing: ") << Missing << endl;
     
     cout << _("  Mixed virtual packages: ") << NVirt << endl;
     cout << _("  Missing: ") << Missing << endl;
     
-   cout << _("Total distinct versions: ") << Cache.Head().VersionCount << " (" <<
-      SizeToStr(Cache.Head().VersionCount*Cache.Head().VersionSz) << ')' << endl;
-   cout << _("Total distinct descriptions: ") << Cache.Head().DescriptionCount << " (" <<
-      SizeToStr(Cache.Head().DescriptionCount*Cache.Head().DescriptionSz) << ')' << endl;
-   cout << _("Total dependencies: ") << Cache.Head().DependsCount << " (" << 
-      SizeToStr(Cache.Head().DependsCount*Cache.Head().DependencySz) << ')' << endl;
+   cout << _("Total distinct versions: ") << Cache->Head().VersionCount << " (" <<
+      SizeToStr(Cache->Head().VersionCount*Cache->Head().VersionSz) << ')' << endl;
+   cout << _("Total distinct descriptions: ") << Cache->Head().DescriptionCount << " (" <<
+      SizeToStr(Cache->Head().DescriptionCount*Cache->Head().DescriptionSz) << ')' << endl;
+   cout << _("Total dependencies: ") << Cache->Head().DependsCount << " (" << 
+      SizeToStr(Cache->Head().DependsCount*Cache->Head().DependencySz) << ')' << endl;
     
     
-   cout << _("Total ver/file relations: ") << Cache.Head().VerFileCount << " (" <<
-      SizeToStr(Cache.Head().VerFileCount*Cache.Head().VerFileSz) << ')' << endl;
-   cout << _("Total Desc/File relations: ") << Cache.Head().DescFileCount << " (" <<
-      SizeToStr(Cache.Head().DescFileCount*Cache.Head().DescFileSz) << ')' << endl;
-   cout << _("Total Provides mappings: ") << Cache.Head().ProvidesCount << " (" <<
-      SizeToStr(Cache.Head().ProvidesCount*Cache.Head().ProvidesSz) << ')' << endl;
+   cout << _("Total ver/file relations: ") << Cache->Head().VerFileCount << " (" <<
+      SizeToStr(Cache->Head().VerFileCount*Cache->Head().VerFileSz) << ')' << endl;
+   cout << _("Total Desc/File relations: ") << Cache->Head().DescFileCount << " (" <<
+      SizeToStr(Cache->Head().DescFileCount*Cache->Head().DescFileSz) << ')' << endl;
+   cout << _("Total Provides mappings: ") << Cache->Head().ProvidesCount << " (" <<
+      SizeToStr(Cache->Head().ProvidesCount*Cache->Head().ProvidesSz) << ')' << endl;
     
     // String list stats
     unsigned long Size = 0;
     unsigned long Count = 0;
     
     // String list stats
     unsigned long Size = 0;
     unsigned long Count = 0;
-   for (pkgCache::StringItem *I = Cache.StringItemP + Cache.Head().StringList;
-        I!= Cache.StringItemP; I = Cache.StringItemP + I->NextItem)
+   for (pkgCache::StringItem *I = Cache->StringItemP + Cache->Head().StringList;
+        I!= Cache->StringItemP; I = Cache->StringItemP + I->NextItem)
     {
        Count++;
     {
        Count++;
-      Size += strlen(Cache.StrP + I->String) + 1;
+      Size += strlen(Cache->StrP + I->String) + 1;
     }
     cout << _("Total globbed strings: ") << Count << " (" << SizeToStr(Size) << ')' << endl;
  
     unsigned long DepVerSize = 0;
     }
     cout << _("Total globbed strings: ") << Count << " (" << SizeToStr(Size) << ')' << endl;
  
     unsigned long DepVerSize = 0;
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
+   for (pkgCache::PkgIterator P = Cache->PkgBegin(); P.end() == false; ++P)
     {
     {
-      for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; V++)
+      for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; ++V)
        {
        {
-        for (pkgCache::DepIterator D = V.DependsList(); D.end() == false; D++)
+        for (pkgCache::DepIterator D = V.DependsList(); D.end() == false; ++D)
          {
             if (D->Version != 0)
                DepVerSize += strlen(D.TargetVer()) + 1;
          {
             if (D->Version != 0)
                DepVerSize += strlen(D.TargetVer()) + 1;
@@ -329,15 +363,15 @@ bool Stats(CommandLine &Cmd)
     
     unsigned long Slack = 0;
     for (int I = 0; I != 7; I++)
     
     unsigned long Slack = 0;
     for (int I = 0; I != 7; I++)
-      Slack += Cache.Head().Pools[I].ItemSize*Cache.Head().Pools[I].Count;
+      Slack += Cache->Head().Pools[I].ItemSize*Cache->Head().Pools[I].Count;
     cout << _("Total slack space: ") << SizeToStr(Slack) << endl;
     
     unsigned long Total = 0;
     cout << _("Total slack space: ") << SizeToStr(Slack) << endl;
     
     unsigned long Total = 0;
-   Total = Slack + Size + Cache.Head().DependsCount*Cache.Head().DependencySz + 
-           Cache.Head().VersionCount*Cache.Head().VersionSz +
-           Cache.Head().PackageCount*Cache.Head().PackageSz + 
-           Cache.Head().VerFileCount*Cache.Head().VerFileSz +
-           Cache.Head().ProvidesCount*Cache.Head().ProvidesSz;
+   Total = Slack + Size + Cache->Head().DependsCount*Cache->Head().DependencySz + 
+           Cache->Head().VersionCount*Cache->Head().VersionSz +
+           Cache->Head().PackageCount*Cache->Head().PackageSz + 
+           Cache->Head().VerFileCount*Cache->Head().VerFileSz +
+           Cache->Head().ProvidesCount*Cache->Head().ProvidesSz;
     cout << _("Total space accounted for: ") << SizeToStr(Total) << endl;
     
     return true;
     cout << _("Total space accounted for: ") << SizeToStr(Total) << endl;
     
     return true;
@@ -346,45 +380,49 @@ bool Stats(CommandLine &Cmd)
  // Dump - show everything                                              /*{{{*/
  // ---------------------------------------------------------------------
  /* This is worthless except fer debugging things */
  // Dump - show everything                                              /*{{{*/
  // ---------------------------------------------------------------------
  /* This is worthless except fer debugging things */
-bool Dump(CommandLine &Cmd)
+static bool Dump(CommandLine &)
  {
  {
-   pkgCache &Cache = *GCache;
-   cout << "Using Versioning System: " << Cache.VS->Label << endl;
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
+   std::cout << "Using Versioning System: " << Cache->VS->Label << std::endl;
     
     
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
+   for (pkgCache::PkgIterator P = Cache->PkgBegin(); P.end() == false; ++P)
     {
     {
-      cout << "Package: " << P.FullName(true) << endl;
-      for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; V++)
+      std::cout << "Package: " << P.FullName(true) << std::endl;
+      for (pkgCache::VerIterator V = P.VersionList(); V.end() == false; ++V)
        {
        {
-        cout << " Version: " << V.VerStr() << endl;
-        cout << "     File: " << V.FileList().File().FileName() << endl;
-        for (pkgCache::DepIterator D = V.DependsList(); D.end() == false; D++)
-           cout << "  Depends: " << D.TargetPkg().FullName(true) << ' ' << 
-                            DeNull(D.TargetVer()) << endl;
-        for (pkgCache::DescIterator D = V.DescriptionList(); D.end() == false; D++)
+        std::cout << " Version: " << V.VerStr() << std::endl;
+        std::cout << "     File: " << V.FileList().File().FileName() << std::endl;
+        for (pkgCache::DepIterator D = V.DependsList(); D.end() == false; ++D)
+           std::cout << "  Depends: " << D.TargetPkg().FullName(true) << ' ' << 
+                            DeNull(D.TargetVer()) << std::endl;
+        for (pkgCache::DescIterator D = V.DescriptionList(); D.end() == false; ++D)
          {
          {
-           cout << " Description Language: " << D.LanguageCode() << endl
-                << "                 File: " << D.FileList().File().FileName() << endl
-                << "                  MD5: " << D.md5() << endl;
+           std::cout << " Description Language: " << D.LanguageCode() << std::endl
+                << "                 File: " << D.FileList().File().FileName() << std::endl
+                << "                  MD5: " << D.md5() << std::endl;
          } 
        }      
     }
  
          } 
        }      
     }
  
-   for (pkgCache::PkgFileIterator F = Cache.FileBegin(); F.end() == false; F++)
+   for (pkgCache::PkgFileIterator F = Cache->FileBegin(); F.end() == false; ++F)
     {
     {
-      cout << "File: " << F.FileName() << endl;
-      cout << " Type: " << F.IndexType() << endl;
-      cout << " Size: " << F->Size << endl;
-      cout << " ID: " << F->ID << endl;
-      cout << " Flags: " << F->Flags << endl;
-      cout << " Time: " << TimeRFC1123(F->mtime) << endl;
-      cout << " Archive: " << DeNull(F.Archive()) << endl;
-      cout << " Component: " << DeNull(F.Component()) << endl;
-      cout << " Version: " << DeNull(F.Version()) << endl;
-      cout << " Origin: " << DeNull(F.Origin()) << endl;
-      cout << " Site: " << DeNull(F.Site()) << endl;
-      cout << " Label: " << DeNull(F.Label()) << endl;
-      cout << " Architecture: " << DeNull(F.Architecture()) << endl;
+      std::cout << "File: " << F.FileName() << std::endl;
+      std::cout << " Type: " << F.IndexType() << std::endl;
+      std::cout << " Size: " << F->Size << std::endl;
+      std::cout << " ID: " << F->ID << std::endl;
+      std::cout << " Flags: " << F->Flags << std::endl;
+      std::cout << " Time: " << TimeRFC1123(F->mtime) << std::endl;
+      std::cout << " Archive: " << DeNull(F.Archive()) << std::endl;
+      std::cout << " Component: " << DeNull(F.Component()) << std::endl;
+      std::cout << " Version: " << DeNull(F.Version()) << std::endl;
+      std::cout << " Origin: " << DeNull(F.Origin()) << std::endl;
+      std::cout << " Site: " << DeNull(F.Site()) << std::endl;
+      std::cout << " Label: " << DeNull(F.Label()) << std::endl;
+      std::cout << " Architecture: " << DeNull(F.Architecture()) << std::endl;
     }
  
     return true;
     }
  
     return true;
@@ -394,20 +432,19 @@ bool Dump(CommandLine &Cmd)
  // ---------------------------------------------------------------------
  /* This is needed to make dpkg --merge happy.. I spent a bit of time to 
     make this run really fast, perhaps I went a little overboard.. */
  // ---------------------------------------------------------------------
  /* This is needed to make dpkg --merge happy.. I spent a bit of time to 
     make this run really fast, perhaps I went a little overboard.. */
-bool DumpAvail(CommandLine &Cmd)
+static bool DumpAvail(CommandLine &)
  {
  {
-   pkgCache &Cache = *GCache;
-
-   pkgPolicy Plcy(&Cache);
-   if (ReadPinFile(Plcy) == false || ReadPinDir(Plcy) == false)
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL || CacheFile.BuildPolicy() == false))
        return false;
        return false;
-   
-   unsigned long Count = Cache.HeaderP->PackageCount+1;
+
+   unsigned long Count = Cache->HeaderP->PackageCount+1;
     pkgCache::VerFile **VFList = new pkgCache::VerFile *[Count];
     memset(VFList,0,sizeof(*VFList)*Count);
     
     // Map versions that we want to write out onto the VerList array.
     pkgCache::VerFile **VFList = new pkgCache::VerFile *[Count];
     memset(VFList,0,sizeof(*VFList)*Count);
     
     // Map versions that we want to write out onto the VerList array.
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
+   for (pkgCache::PkgIterator P = Cache->PkgBegin(); P.end() == false; ++P)
     {    
        if (P->VersionList == 0)
          continue;
     {    
        if (P->VersionList == 0)
          continue;
@@ -415,7 +452,7 @@ bool DumpAvail(CommandLine &Cmd)
        /* Find the proper version to use. If the policy says there are no
           possible selections we return the installed version, if available..
                  This prevents dselect from making it obsolete. */
        /* Find the proper version to use. If the policy says there are no
           possible selections we return the installed version, if available..
                  This prevents dselect from making it obsolete. */
-      pkgCache::VerIterator V = Plcy.GetCandidateVer(P);
+      pkgCache::VerIterator V = CacheFile.GetPolicy()->GetCandidateVer(P);
        if (V.end() == true)
        {
          if (P->CurrentVer == 0)
        if (V.end() == true)
        {
          if (P->CurrentVer == 0)
@@ -424,7 +461,7 @@ bool DumpAvail(CommandLine &Cmd)
        }
        
        pkgCache::VerFileIterator VF = V.FileList();
        }
        
        pkgCache::VerFileIterator VF = V.FileList();
-      for (; VF.end() == false ; VF++)
+      for (; VF.end() == false ; ++VF)
          if ((VF.File()->Flags & pkgCache::Flag::NotSource) == 0)
             break;
        
          if ((VF.File()->Flags & pkgCache::Flag::NotSource) == 0)
             break;
        
@@ -438,9 +475,9 @@ bool DumpAvail(CommandLine &Cmd)
           handling works OK. */
        if (VF.end() == true)
        {
           handling works OK. */
        if (VF.end() == true)
        {
-        for (pkgCache::VerIterator Cur = P.VersionList(); Cur.end() != true; Cur++)
+        for (pkgCache::VerIterator Cur = P.VersionList(); Cur.end() != true; ++Cur)
          {
          {
-           for (VF = Cur.FileList(); VF.end() == false; VF++)
+           for (VF = Cur.FileList(); VF.end() == false; ++VF)
             {    
                if ((VF.File()->Flags & pkgCache::Flag::NotSource) == 0)
                {
             {    
                if ((VF.File()->Flags & pkgCache::Flag::NotSource) == 0)
                {
@@ -460,10 +497,10 @@ bool DumpAvail(CommandLine &Cmd)
     LocalitySort(VFList,Count,sizeof(*VFList));
  
     // Iterate over all the package files and write them out.
     LocalitySort(VFList,Count,sizeof(*VFList));
  
     // Iterate over all the package files and write them out.
-   char *Buffer = new char[Cache.HeaderP->MaxVerFileSize+10];
+   char *Buffer = new char[Cache->HeaderP->MaxVerFileSize+10];
     for (pkgCache::VerFile **J = VFList; *J != 0;)
     {
     for (pkgCache::VerFile **J = VFList; *J != 0;)
     {
-      pkgCache::PkgFileIterator File(Cache,(*J)->File + Cache.PkgFileP);
+      pkgCache::PkgFileIterator File(*Cache,(*J)->File + Cache->PkgFileP);
        if (File.IsOk() == false)
        {
          _error->Error(_("Package file %s is out of sync."),File.FileName());
        if (File.IsOk() == false)
        {
          _error->Error(_("Package file %s is out of sync."),File.FileName());
@@ -482,7 +519,7 @@ bool DumpAvail(CommandLine &Cmd)
        unsigned long Pos = 0;
        for (; *J != 0; J++)
        {
        unsigned long Pos = 0;
        for (; *J != 0; J++)
        {
-        if ((*J)->File + Cache.PkgFileP != File)
+        if ((*J)->File + Cache->PkgFileP != File)
             break;
          
          const pkgCache::VerFile &VF = **J;
             break;
          
          const pkgCache::VerFile &VF = **J;
@@ -504,7 +541,7 @@ bool DumpAvail(CommandLine &Cmd)
          if ((File->Flags & pkgCache::Flag::NotSource) == pkgCache::Flag::NotSource)
          {
             pkgTagSection Tags;
          if ((File->Flags & pkgCache::Flag::NotSource) == pkgCache::Flag::NotSource)
          {
             pkgTagSection Tags;
-           TFRewriteData RW[] = {{"Status",0},{"Config-Version",0},{}};
+           TFRewriteData RW[] = {{"Status", NULL, NULL},{"Config-Version", NULL, NULL},{NULL, NULL, NULL}};
             const char *Zero = 0;
             if (Tags.Scan(Buffer+Jitter,VF.Size+1) == false ||
                 TFRewrite(stdout,Tags,&Zero,RW) == false)
             const char *Zero = 0;
             if (Tags.Scan(Buffer+Jitter,VF.Size+1) == false ||
                 TFRewrite(stdout,Tags,&Zero,RW) == false)
@@ -533,195 +570,145 @@ bool DumpAvail(CommandLine &Cmd)
     return !_error->PendingError();
  }
                                                                         /*}}}*/
     return !_error->PendingError();
  }
                                                                         /*}}}*/
-// Depends - Print out a dependency tree                               /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-bool Depends(CommandLine &CmdL)
+// ShowDepends - Helper for printing out a dependency tree             /*{{{*/
+static bool ShowDepends(CommandLine &CmdL, bool const RevDepends)
  {
  {
-   pkgCache &Cache = *GCache;
-   SPtrArray<unsigned> Colours = new unsigned[Cache.Head().PackageCount];
-   memset(Colours,0,sizeof(*Colours)*Cache.Head().PackageCount);
-   
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
-   {
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(*I);
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
-      Colours[Pkg->ID] = 1;
-   }
-   
-   bool Recurse = _config->FindB("APT::Cache::RecurseDepends",false);
-   bool Installed = _config->FindB("APT::Cache::Installed",false);
-   bool Important = _config->FindB("APT::Cache::Important",false);
-   bool DidSomething;
-   do
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
+   CacheSetHelperVirtuals helper(false);
+   APT::VersionList verset = APT::VersionList::FromCommandLine(CacheFile, CmdL.FileList + 1, APT::VersionList::CANDIDATE, helper);
+   if (verset.empty() == true && helper.virtualPkgs.empty() == true)
+      return _error->Error(_("No packages found"));
+   std::vector<bool> Shown(Cache->Head().PackageCount);
+
+   bool const Recurse = _config->FindB("APT::Cache::RecurseDepends", false);
+   bool const Installed = _config->FindB("APT::Cache::Installed", false);
+   bool const Important = _config->FindB("APT::Cache::Important", false);
+   bool const ShowDepType = _config->FindB("APT::Cache::ShowDependencyType", RevDepends == false);
+   bool const ShowVersion = _config->FindB("APT::Cache::ShowVersion", false);
+   bool const ShowPreDepends = _config->FindB("APT::Cache::ShowPre-Depends", true);
+   bool const ShowDepends = _config->FindB("APT::Cache::ShowDepends", true);
+   bool const ShowRecommends = _config->FindB("APT::Cache::ShowRecommends", Important == false);
+   bool const ShowSuggests = _config->FindB("APT::Cache::ShowSuggests", Important == false);
+   bool const ShowReplaces = _config->FindB("APT::Cache::ShowReplaces", Important == false);
+   bool const ShowConflicts = _config->FindB("APT::Cache::ShowConflicts", Important == false);
+   bool const ShowBreaks = _config->FindB("APT::Cache::ShowBreaks", Important == false);
+   bool const ShowEnhances = _config->FindB("APT::Cache::ShowEnhances", Important == false);
+   bool const ShowOnlyFirstOr = _config->FindB("APT::Cache::ShowOnlyFirstOr", false);
+
+   while (verset.empty() != true)
     {
     {
-      DidSomething = false;
-      for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
-      {
-        if (Colours[Pkg->ID] != 1)
-           continue;
-        Colours[Pkg->ID] = 2;
-        DidSomething = true;
-        
-        pkgCache::VerIterator Ver = Pkg.VersionList();
-        if (Ver.end() == true)
-        {
-           cout << '<' << Pkg.FullName(true) << '>' << endl;
-           continue;
-        }
-        
+      pkgCache::VerIterator Ver = *verset.begin();
+      verset.erase(verset.begin());
+      pkgCache::PkgIterator Pkg = Ver.ParentPkg();
+      Shown[Pkg->ID] = true;
+
          cout << Pkg.FullName(true) << endl;
          cout << Pkg.FullName(true) << endl;
-        
-        for (pkgCache::DepIterator D = Ver.DependsList(); D.end() == false; D++)
+
+        if (RevDepends == true)
+           cout << "Reverse Depends:" << endl;
+        for (pkgCache::DepIterator D = RevDepends ? Pkg.RevDependsList() : Ver.DependsList();
+             D.end() == false; ++D)
          {
          {
-           // Important deps only
-           if (Important == true)
-              if (D->Type != pkgCache::Dep::PreDepends &&
-                  D->Type != pkgCache::Dep::Depends)
-                 continue;
-                 
-           pkgCache::PkgIterator Trg = D.TargetPkg();
+           switch (D->Type) {
+           case pkgCache::Dep::PreDepends: if (!ShowPreDepends) continue; break;
+           case pkgCache::Dep::Depends: if (!ShowDepends) continue; break;
+           case pkgCache::Dep::Recommends: if (!ShowRecommends) continue; break;
+           case pkgCache::Dep::Suggests: if (!ShowSuggests) continue; break;
+           case pkgCache::Dep::Replaces: if (!ShowReplaces) continue; break;       case pkgCache::Dep::Conflicts: if (!ShowConflicts) continue; break;
+           case pkgCache::Dep::DpkgBreaks: if (!ShowBreaks) continue; break;
+           case pkgCache::Dep::Enhances: if (!ShowEnhances) continue; break;
+           }
+
+           pkgCache::PkgIterator Trg = RevDepends ? D.ParentPkg() : D.TargetPkg();
  
             if((Installed && Trg->CurrentVer != 0) || !Installed)
               {
  
  
             if((Installed && Trg->CurrentVer != 0) || !Installed)
               {
  
-               if ((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or)
+               if ((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or && ShowOnlyFirstOr == false)
                   cout << " |";
                 else
                   cout << "  ";
             
                 // Show the package
                   cout << " |";
                 else
                   cout << "  ";
             
                 // Show the package
+               if (ShowDepType == true)
+                 cout << D.DepType() << ": ";
                 if (Trg->VersionList == 0)
                 if (Trg->VersionList == 0)
-                 cout << D.DepType() << ": <" << Trg.FullName(true) << ">" << endl;
+                 cout << "<" << Trg.FullName(true) << ">";
                 else
                 else
-                 cout << D.DepType() << ": " << Trg.FullName(true) << endl;
-           
-               if (Recurse == true)
-                 Colours[D.TargetPkg()->ID]++;
+                 cout << Trg.FullName(true);
+               if (ShowVersion == true && D->Version != 0)
+                  cout << " (" << pkgCache::CompTypeDeb(D->CompareOp) << ' ' << D.TargetVer() << ')';
+               cout << std::endl;
+
+               if (Recurse == true && Shown[Trg->ID] == false)
+               {
+                 Shown[Trg->ID] = true;
+                 verset.insert(APT::VersionSet::FromPackage(CacheFile, Trg, APT::VersionSet::CANDIDATE, helper));
+               }
  
               }
             
             // Display all solutions
             SPtrArray<pkgCache::Version *> List = D.AllTargets();
  
               }
             
             // Display all solutions
             SPtrArray<pkgCache::Version *> List = D.AllTargets();
-           pkgPrioSortList(Cache,List);
+           pkgPrioSortList(*Cache,List);
             for (pkgCache::Version **I = List; *I != 0; I++)
             {
             for (pkgCache::Version **I = List; *I != 0; I++)
             {
-              pkgCache::VerIterator V(Cache,*I);
-              if (V != Cache.VerP + V.ParentPkg()->VersionList ||
+              pkgCache::VerIterator V(*Cache,*I);
+              if (V != Cache->VerP + V.ParentPkg()->VersionList ||
                    V->ParentPkg == D->Package)
                   continue;
                cout << "    " << V.ParentPkg().FullName(true) << endl;
                    V->ParentPkg == D->Package)
                   continue;
                cout << "    " << V.ParentPkg().FullName(true) << endl;
-              
-              if (Recurse == true)
-                 Colours[D.ParentPkg()->ID]++;
+
+               if (Recurse == true && Shown[V.ParentPkg()->ID] == false)
+               {
+                 Shown[V.ParentPkg()->ID] = true;
+                 verset.insert(APT::VersionSet::FromPackage(CacheFile, V.ParentPkg(), APT::VersionSet::CANDIDATE, helper));
+               }
             }
             }
+
+           if (ShowOnlyFirstOr == true)
+              while ((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or) ++D;
          }
          }
-      }      
-   }   
-   while (DidSomething == true);
-   
+   }
+
+   for (APT::PackageSet::const_iterator Pkg = helper.virtualPkgs.begin();
+       Pkg != helper.virtualPkgs.end(); ++Pkg)
+      cout << '<' << Pkg.FullName(true) << '>' << endl;
+
     return true;
  }
                                                                         /*}}}*/
     return true;
  }
                                                                         /*}}}*/
-// RDepends - Print out a reverse dependency tree - mbc                        /*{{{*/
+// Depends - Print out a dependency tree                               /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // ---------------------------------------------------------------------
  /* */
-bool RDepends(CommandLine &CmdL)
+static bool Depends(CommandLine &CmdL)
  {
  {
-   pkgCache &Cache = *GCache;
-   SPtrArray<unsigned> Colours = new unsigned[Cache.Head().PackageCount];
-   memset(Colours,0,sizeof(*Colours)*Cache.Head().PackageCount);
-   
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
-   {
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(*I);
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
-      Colours[Pkg->ID] = 1;
-   }
-   
-   bool Recurse = _config->FindB("APT::Cache::RecurseDepends",false);
-   bool Installed = _config->FindB("APT::Cache::Installed",false);
-   bool DidSomething;
-   do
-   {
-      DidSomething = false;
-      for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
-      {
-        if (Colours[Pkg->ID] != 1)
-           continue;
-        Colours[Pkg->ID] = 2;
-        DidSomething = true;
-        
-        pkgCache::VerIterator Ver = Pkg.VersionList();
-        if (Ver.end() == true)
-        {
-           cout << '<' << Pkg.FullName(true) << '>' << endl;
-           continue;
-        }
-        
-        cout << Pkg.FullName(true) << endl;
-        
-        cout << "Reverse Depends:" << endl;
-        for (pkgCache::DepIterator D = Pkg.RevDependsList(); D.end() == false; D++)
-        {          
-           // Show the package
-           pkgCache::PkgIterator Trg = D.ParentPkg();
-
-           if((Installed && Trg->CurrentVer != 0) || !Installed)
-             {
-
-               if ((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or)
-                 cout << " |";
-               else
-                 cout << "  ";
-
-               if (Trg->VersionList == 0)
-                 cout << D.DepType() << ": <" << Trg.FullName(true) << ">" << endl;
-               else
-                 cout << Trg.FullName(true) << endl;
-
-               if (Recurse == true)
-                 Colours[D.ParentPkg()->ID]++;
-
-             }
-           
-           // Display all solutions
-           SPtrArray<pkgCache::Version *> List = D.AllTargets();
-           pkgPrioSortList(Cache,List);
-           for (pkgCache::Version **I = List; *I != 0; I++)
-           {
-              pkgCache::VerIterator V(Cache,*I);
-              if (V != Cache.VerP + V.ParentPkg()->VersionList ||
-                  V->ParentPkg == D->Package)
-                 continue;
-              cout << "    " << V.ParentPkg().FullName(true) << endl;
-              
-              if (Recurse == true)
-                 Colours[D.ParentPkg()->ID]++;
-           }
-        }
-      }      
-   }   
-   while (DidSomething == true);
-   
-   return true;
+   return ShowDepends(CmdL, false);
+}
+                                                                       /*}}}*/
+// RDepends - Print out a reverse dependency tree                      /*{{{*/
+// ---------------------------------------------------------------------
+/* */
+static bool RDepends(CommandLine &CmdL)
+{
+   return ShowDepends(CmdL, true);
  }
                                                                         /*}}}*/
  // xvcg - Generate a graph for xvcg                                    /*{{{*/
  // ---------------------------------------------------------------------
  // Code contributed from Junichi Uekawa <dancer@debian.org> on 20 June 2002.
  
  }
                                                                         /*}}}*/
  // xvcg - Generate a graph for xvcg                                    /*{{{*/
  // ---------------------------------------------------------------------
  // Code contributed from Junichi Uekawa <dancer@debian.org> on 20 June 2002.
  
-bool XVcg(CommandLine &CmdL)
+static bool XVcg(CommandLine &CmdL)
  {
  {
-   pkgCache &Cache = *GCache;
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
     bool GivenOnly = _config->FindB("APT::Cache::GivenOnly",false);
     
     /* Normal packages are boxes
     bool GivenOnly = _config->FindB("APT::Cache::GivenOnly",false);
     
     /* Normal packages are boxes
@@ -738,21 +725,21 @@ bool XVcg(CommandLine &CmdL)
        0 = None */
     enum States {None=0, ToShow, ToShowNR, DoneNR, Done};
     enum TheFlags {ForceNR=(1<<0)};
        0 = None */
     enum States {None=0, ToShow, ToShowNR, DoneNR, Done};
     enum TheFlags {ForceNR=(1<<0)};
-   unsigned char *Show = new unsigned char[Cache.Head().PackageCount];
-   unsigned char *Flags = new unsigned char[Cache.Head().PackageCount];
-   unsigned char *ShapeMap = new unsigned char[Cache.Head().PackageCount];
+   unsigned char *Show = new unsigned char[Cache->Head().PackageCount];
+   unsigned char *Flags = new unsigned char[Cache->Head().PackageCount];
+   unsigned char *ShapeMap = new unsigned char[Cache->Head().PackageCount];
     
     // Show everything if no arguments given
     if (CmdL.FileList[1] == 0)
     
     // Show everything if no arguments given
     if (CmdL.FileList[1] == 0)
-      for (unsigned long I = 0; I != Cache.Head().PackageCount; I++)
+      for (unsigned long I = 0; I != Cache->Head().PackageCount; I++)
          Show[I] = ToShow;
     else
          Show[I] = ToShow;
     else
-      for (unsigned long I = 0; I != Cache.Head().PackageCount; I++)
+      for (unsigned long I = 0; I != Cache->Head().PackageCount; I++)
          Show[I] = None;
          Show[I] = None;
-   memset(Flags,0,sizeof(*Flags)*Cache.Head().PackageCount);
+   memset(Flags,0,sizeof(*Flags)*Cache->Head().PackageCount);
     
     // Map the shapes
     
     // Map the shapes
-   for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+   for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
     {   
        if (Pkg->VersionList == 0)
        {
     {   
        if (Pkg->VersionList == 0)
        {
@@ -771,38 +758,25 @@ bool XVcg(CommandLine &CmdL)
             ShapeMap[Pkg->ID] = 3;
        }
     }
             ShapeMap[Pkg->ID] = 3;
        }
     }
-   
+
     // Load the list of packages from the command line into the show list
     // Load the list of packages from the command line into the show list
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
+   APT::CacheSetHelper helper(true, GlobalError::NOTICE);
+   std::list<APT::PackageSet::Modifier> mods;
+   mods.push_back(APT::PackageSet::Modifier(0, ",", APT::PackageSet::Modifier::POSTFIX));
+   mods.push_back(APT::PackageSet::Modifier(1, "^", APT::PackageSet::Modifier::POSTFIX));
+   std::map<unsigned short, APT::PackageSet> pkgsets =
+               APT::PackageSet::GroupedFromCommandLine(CacheFile, CmdL.FileList + 1, mods, 0, helper);
+
+   for (APT::PackageSet::const_iterator Pkg = pkgsets[0].begin();
+       Pkg != pkgsets[0].end(); ++Pkg)
+      Show[Pkg->ID] = ToShow;
+   for (APT::PackageSet::const_iterator Pkg = pkgsets[1].begin();
+       Pkg != pkgsets[1].end(); ++Pkg)
     {
     {
-      // Process per-package flags
-      string P = *I;
-      bool Force = false;
-      if (P.length() > 3)
-      {
-        if (P.end()[-1] == '^')
-        {
-           Force = true;
-           P.erase(P.end()-1);
-        }
-        
-        if (P.end()[-1] == ',')
-           P.erase(P.end()-1);
-      }
-      
-      // Locate the package
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(P);
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
        Show[Pkg->ID] = ToShow;
        Show[Pkg->ID] = ToShow;
-      
-      if (Force == true)
-        Flags[Pkg->ID] |= ForceNR;
+      Flags[Pkg->ID] |= ForceNR;
     }
     }
-   
+
     // Little header
     cout << "graph: { title: \"packages\"" << endl <<
       "xmax: 700 ymax: 700 x: 30 y: 30" << endl <<
     // Little header
     cout << "graph: { title: \"packages\"" << endl <<
       "xmax: 700 ymax: 700 x: 30 y: 30" << endl <<
@@ -812,7 +786,7 @@ bool XVcg(CommandLine &CmdL)
     while (Act == true)
     {
        Act = false;
     while (Act == true)
     {
        Act = false;
-      for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+      for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
        {
          // See we need to show this package
          if (Show[Pkg->ID] == None || Show[Pkg->ID] >= DoneNR)
        {
          // See we need to show this package
          if (Show[Pkg->ID] == None || Show[Pkg->ID] >= DoneNR)
@@ -838,24 +812,24 @@ bool XVcg(CommandLine &CmdL)
             continue;
          
          pkgCache::VerIterator Ver = Pkg.VersionList();
             continue;
          
          pkgCache::VerIterator Ver = Pkg.VersionList();
-        for (pkgCache::DepIterator D = Ver.DependsList(); D.end() == false; D++)
+        for (pkgCache::DepIterator D = Ver.DependsList(); D.end() == false; ++D)
          {
             // See if anything can meet this dep
             // Walk along the actual package providing versions
             bool Hit = false;
             pkgCache::PkgIterator DPkg = D.TargetPkg();
             for (pkgCache::VerIterator I = DPkg.VersionList();
          {
             // See if anything can meet this dep
             // Walk along the actual package providing versions
             bool Hit = false;
             pkgCache::PkgIterator DPkg = D.TargetPkg();
             for (pkgCache::VerIterator I = DPkg.VersionList();
-                     I.end() == false && Hit == false; I++)
+                     I.end() == false && Hit == false; ++I)
             {
             {
-              if (Cache.VS->CheckDep(I.VerStr(),D->CompareOp,D.TargetVer()) == true)
+              if (Cache->VS->CheckDep(I.VerStr(),D->CompareOp,D.TargetVer()) == true)
                   Hit = true;
             }
             
             // Follow all provides
             for (pkgCache::PrvIterator I = DPkg.ProvidesList(); 
                   Hit = true;
             }
             
             // Follow all provides
             for (pkgCache::PrvIterator I = DPkg.ProvidesList(); 
-                     I.end() == false && Hit == false; I++)
+                     I.end() == false && Hit == false; ++I)
             {
             {
-              if (Cache.VS->CheckDep(I.ProvideVersion(),D->CompareOp,D.TargetVer()) == false)
+              if (Cache->VS->CheckDep(I.ProvideVersion(),D->CompareOp,D.TargetVer()) == false)
                   Hit = true;
             }
             
                   Hit = true;
             }
             
@@ -870,10 +844,7 @@ bool XVcg(CommandLine &CmdL)
                {
                   /* If a conflicts does not meet anything in the database
                      then show the relation but do not recurse */
                {
                   /* If a conflicts does not meet anything in the database
                      then show the relation but do not recurse */
-                 if (Hit == false && 
-                     (D->Type == pkgCache::Dep::Conflicts ||
-                      D->Type == pkgCache::Dep::DpkgBreaks ||
-                      D->Type == pkgCache::Dep::Obsoletes))
+                 if (Hit == false && D.IsNegative() == true)
                   {
                      if (Show[D.TargetPkg()->ID] == None && 
                          Show[D.TargetPkg()->ID] != ToShow)
                   {
                      if (Show[D.TargetPkg()->ID] == None && 
                          Show[D.TargetPkg()->ID] != ToShow)
@@ -916,7 +887,7 @@ bool XVcg(CommandLine &CmdL)
     
     /* Draw the box colours after the fact since we can not tell what colour
        they should be until everything is finished drawing */
     
     /* Draw the box colours after the fact since we can not tell what colour
        they should be until everything is finished drawing */
-   for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+   for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
     {
        if (Show[Pkg->ID] < DoneNR)
          continue;
     {
        if (Show[Pkg->ID] < DoneNR)
          continue;
@@ -943,9 +914,13 @@ bool XVcg(CommandLine &CmdL)
  /* Dotty is the graphvis program for generating graphs. It is a fairly
     simple queuing algorithm that just writes dependencies and nodes. 
     http://www.research.att.com/sw/tools/graphviz/ */
  /* Dotty is the graphvis program for generating graphs. It is a fairly
     simple queuing algorithm that just writes dependencies and nodes. 
     http://www.research.att.com/sw/tools/graphviz/ */
-bool Dotty(CommandLine &CmdL)
+static bool Dotty(CommandLine &CmdL)
  {
  {
-   pkgCache &Cache = *GCache;
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
     bool GivenOnly = _config->FindB("APT::Cache::GivenOnly",false);
     
     /* Normal packages are boxes
     bool GivenOnly = _config->FindB("APT::Cache::GivenOnly",false);
     
     /* Normal packages are boxes
@@ -962,21 +937,21 @@ bool Dotty(CommandLine &CmdL)
        0 = None */
     enum States {None=0, ToShow, ToShowNR, DoneNR, Done};
     enum TheFlags {ForceNR=(1<<0)};
        0 = None */
     enum States {None=0, ToShow, ToShowNR, DoneNR, Done};
     enum TheFlags {ForceNR=(1<<0)};
-   unsigned char *Show = new unsigned char[Cache.Head().PackageCount];
-   unsigned char *Flags = new unsigned char[Cache.Head().PackageCount];
-   unsigned char *ShapeMap = new unsigned char[Cache.Head().PackageCount];
+   unsigned char *Show = new unsigned char[Cache->Head().PackageCount];
+   unsigned char *Flags = new unsigned char[Cache->Head().PackageCount];
+   unsigned char *ShapeMap = new unsigned char[Cache->Head().PackageCount];
     
     // Show everything if no arguments given
     if (CmdL.FileList[1] == 0)
     
     // Show everything if no arguments given
     if (CmdL.FileList[1] == 0)
-      for (unsigned long I = 0; I != Cache.Head().PackageCount; I++)
+      for (unsigned long I = 0; I != Cache->Head().PackageCount; I++)
          Show[I] = ToShow;
     else
          Show[I] = ToShow;
     else
-      for (unsigned long I = 0; I != Cache.Head().PackageCount; I++)
+      for (unsigned long I = 0; I != Cache->Head().PackageCount; I++)
          Show[I] = None;
          Show[I] = None;
-   memset(Flags,0,sizeof(*Flags)*Cache.Head().PackageCount);
+   memset(Flags,0,sizeof(*Flags)*Cache->Head().PackageCount);
     
     // Map the shapes
     
     // Map the shapes
-   for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+   for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
     {   
        if (Pkg->VersionList == 0)
        {
     {   
        if (Pkg->VersionList == 0)
        {
@@ -995,38 +970,25 @@ bool Dotty(CommandLine &CmdL)
             ShapeMap[Pkg->ID] = 3;
        }
     }
             ShapeMap[Pkg->ID] = 3;
        }
     }
-   
+
     // Load the list of packages from the command line into the show list
     // Load the list of packages from the command line into the show list
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
+   APT::CacheSetHelper helper(true, GlobalError::NOTICE);
+   std::list<APT::PackageSet::Modifier> mods;
+   mods.push_back(APT::PackageSet::Modifier(0, ",", APT::PackageSet::Modifier::POSTFIX));
+   mods.push_back(APT::PackageSet::Modifier(1, "^", APT::PackageSet::Modifier::POSTFIX));
+   std::map<unsigned short, APT::PackageSet> pkgsets =
+               APT::PackageSet::GroupedFromCommandLine(CacheFile, CmdL.FileList + 1, mods, 0, helper);
+
+   for (APT::PackageSet::const_iterator Pkg = pkgsets[0].begin();
+       Pkg != pkgsets[0].end(); ++Pkg)
+      Show[Pkg->ID] = ToShow;
+   for (APT::PackageSet::const_iterator Pkg = pkgsets[1].begin();
+       Pkg != pkgsets[1].end(); ++Pkg)
     {
     {
-      // Process per-package flags
-      string P = *I;
-      bool Force = false;
-      if (P.length() > 3)
-      {
-        if (P.end()[-1] == '^')
-        {
-           Force = true;
-           P.erase(P.end()-1);
-        }
-        
-        if (P.end()[-1] == ',')
-           P.erase(P.end()-1);
-      }
-      
-      // Locate the package
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(P);
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
        Show[Pkg->ID] = ToShow;
        Show[Pkg->ID] = ToShow;
-      
-      if (Force == true)
-        Flags[Pkg->ID] |= ForceNR;
+      Flags[Pkg->ID] |= ForceNR;
     }
     }
-   
+
     // Little header
     printf("digraph packages {\n");
     printf("concentrate=true;\n");
     // Little header
     printf("digraph packages {\n");
     printf("concentrate=true;\n");
@@ -1036,7 +998,7 @@ bool Dotty(CommandLine &CmdL)
     while (Act == true)
     {
        Act = false;
     while (Act == true)
     {
        Act = false;
-      for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+      for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
        {
          // See we need to show this package
          if (Show[Pkg->ID] == None || Show[Pkg->ID] >= DoneNR)
        {
          // See we need to show this package
          if (Show[Pkg->ID] == None || Show[Pkg->ID] >= DoneNR)
@@ -1060,24 +1022,24 @@ bool Dotty(CommandLine &CmdL)
             continue;
          
          pkgCache::VerIterator Ver = Pkg.VersionList();
             continue;
          
          pkgCache::VerIterator Ver = Pkg.VersionList();
-        for (pkgCache::DepIterator D = Ver.DependsList(); D.end() == false; D++)
+        for (pkgCache::DepIterator D = Ver.DependsList(); D.end() == false; ++D)
          {
             // See if anything can meet this dep
             // Walk along the actual package providing versions
             bool Hit = false;
             pkgCache::PkgIterator DPkg = D.TargetPkg();
             for (pkgCache::VerIterator I = DPkg.VersionList();
          {
             // See if anything can meet this dep
             // Walk along the actual package providing versions
             bool Hit = false;
             pkgCache::PkgIterator DPkg = D.TargetPkg();
             for (pkgCache::VerIterator I = DPkg.VersionList();
-                     I.end() == false && Hit == false; I++)
+                     I.end() == false && Hit == false; ++I)
             {
             {
-              if (Cache.VS->CheckDep(I.VerStr(),D->CompareOp,D.TargetVer()) == true)
+              if (Cache->VS->CheckDep(I.VerStr(),D->CompareOp,D.TargetVer()) == true)
                   Hit = true;
             }
             
             // Follow all provides
             for (pkgCache::PrvIterator I = DPkg.ProvidesList(); 
                   Hit = true;
             }
             
             // Follow all provides
             for (pkgCache::PrvIterator I = DPkg.ProvidesList(); 
-                     I.end() == false && Hit == false; I++)
+                     I.end() == false && Hit == false; ++I)
             {
             {
-              if (Cache.VS->CheckDep(I.ProvideVersion(),D->CompareOp,D.TargetVer()) == false)
+              if (Cache->VS->CheckDep(I.ProvideVersion(),D->CompareOp,D.TargetVer()) == false)
                   Hit = true;
             }
             
                   Hit = true;
             }
             
@@ -1091,9 +1053,7 @@ bool Dotty(CommandLine &CmdL)
                {
                   /* If a conflicts does not meet anything in the database
                      then show the relation but do not recurse */
                {
                   /* If a conflicts does not meet anything in the database
                      then show the relation but do not recurse */
-                 if (Hit == false && 
-                     (D->Type == pkgCache::Dep::Conflicts ||
-                      D->Type == pkgCache::Dep::Obsoletes))
+                 if (Hit == false && D.IsNegative() == true)
                   {
                      if (Show[D.TargetPkg()->ID] == None && 
                          Show[D.TargetPkg()->ID] != ToShow)
                   {
                      if (Show[D.TargetPkg()->ID] == None && 
                          Show[D.TargetPkg()->ID] != ToShow)
@@ -1113,6 +1073,7 @@ bool Dotty(CommandLine &CmdL)
                {
                   case pkgCache::Dep::Conflicts:
                   case pkgCache::Dep::Obsoletes:
                {
                   case pkgCache::Dep::Conflicts:
                   case pkgCache::Dep::Obsoletes:
+                 case pkgCache::Dep::DpkgBreaks:
                   printf("[color=springgreen];\n");
                   break;
                   
                   printf("[color=springgreen];\n");
                   break;
                   
@@ -1131,7 +1092,7 @@ bool Dotty(CommandLine &CmdL)
     
     /* Draw the box colours after the fact since we can not tell what colour
        they should be until everything is finished drawing */
     
     /* Draw the box colours after the fact since we can not tell what colour
        they should be until everything is finished drawing */
-   for (pkgCache::PkgIterator Pkg = Cache.PkgBegin(); Pkg.end() == false; Pkg++)
+   for (pkgCache::PkgIterator Pkg = Cache->PkgBegin(); Pkg.end() == false; ++Pkg)
     {
        if (Show[Pkg->ID] < DoneNR)
          continue;
     {
        if (Show[Pkg->ID] < DoneNR)
          continue;
@@ -1146,70 +1107,43 @@ bool Dotty(CommandLine &CmdL)
     }
     
     printf("}\n");
     }
     
     printf("}\n");
+   delete[] Show;
+   delete[] Flags;
+   delete[] ShapeMap;
     return true;
  }
                                                                         /*}}}*/
     return true;
  }
                                                                         /*}}}*/
-// DoAdd - Perform an adding operation                                 /*{{{*/
+// DisplayRecord - Displays the complete record for the package                /*{{{*/
  // ---------------------------------------------------------------------
  // ---------------------------------------------------------------------
-/* */
-bool DoAdd(CommandLine &CmdL)
-{
-   return _error->Error("Unimplemented");
-#if 0   
-   // Make sure there is at least one argument
-   if (CmdL.FileSize() <= 1)
-      return _error->Error("You must give at least one file name");
-   
-   // Open the cache
-   FileFd CacheF(_config->FindFile("Dir::Cache::pkgcache"),FileFd::WriteAny);
-   if (_error->PendingError() == true)
-      return false;
-   
-   DynamicMMap Map(CacheF,MMap::Public);
-   if (_error->PendingError() == true)
-      return false;
-
-   OpTextProgress Progress(*_config);
-   pkgCacheGenerator Gen(Map,Progress);
-   if (_error->PendingError() == true)
-      return false;
+/* This displays the package record from the proper package index file. 
+   It is not used by DumpAvail for performance reasons. */
  
  
-   unsigned long Length = CmdL.FileSize() - 1;
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
+static APT_PURE unsigned char const* skipDescriptionFields(unsigned char const * DescP)
+{
+   char const * const TagName = "\nDescription";
+   size_t const TagLen = strlen(TagName);
+   while ((DescP = (unsigned char*)strchr((char*)DescP, '\n')) != NULL)
     {
     {
-      Progress.OverallProgress(I - CmdL.FileList,Length,1,"Generating cache");
-      Progress.SubProgress(Length);
-
-      // Do the merge
-      FileFd TagF(*I,FileFd::ReadOnly);
-      debListParser Parser(TagF);
-      if (_error->PendingError() == true)
-        return _error->Error("Problem opening %s",*I);
-      
-      if (Gen.SelectFile(*I,"") == false)
-        return _error->Error("Problem with SelectFile");
-        
-      if (Gen.MergeList(Parser) == false)
-        return _error->Error("Problem with MergeList");
+      if (DescP[1] == ' ')
+        DescP += 2;
+      else if (strncmp((char*)DescP, TagName, TagLen) == 0)
+        DescP += TagLen;
+      else
+        break;
     }
     }
-
-   Progress.Done();
-   GCache = &Gen.GetCache();
-   Stats(CmdL);
-   
-   return true;
-#endif   
+   if (DescP != NULL)
+      ++DescP;
+   return DescP;
  }
  }
-                                                                       /*}}}*/
-// DisplayRecord - Displays the complete record for the package                /*{{{*/
-// ---------------------------------------------------------------------
-/* This displays the package record from the proper package index file. 
-   It is not used by DumpAvail for performance reasons. */
-bool DisplayRecord(pkgCache::VerIterator V)
+static bool DisplayRecord(pkgCacheFile &CacheFile, pkgCache::VerIterator V)
  {
  {
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   if (unlikely(Cache == NULL))
+      return false;
+
     // Find an appropriate file
     pkgCache::VerFileIterator Vf = V.FileList();
     // Find an appropriate file
     pkgCache::VerFileIterator Vf = V.FileList();
-   for (; Vf.end() == false; Vf++)
+   for (; Vf.end() == false; ++Vf)
        if ((Vf.File()->Flags & pkgCache::Flag::NotSource) == 0)
          break;
     if (Vf.end() == true)
        if ((Vf.File()->Flags & pkgCache::Flag::NotSource) == 0)
          break;
     if (Vf.end() == true)
@@ -1219,82 +1153,105 @@ bool DisplayRecord(pkgCache::VerIterator V)
     pkgCache::PkgFileIterator I = Vf.File();
     if (I.IsOk() == false)
        return _error->Error(_("Package file %s is out of sync."),I.FileName());
     pkgCache::PkgFileIterator I = Vf.File();
     if (I.IsOk() == false)
        return _error->Error(_("Package file %s is out of sync."),I.FileName());
-   
-   FileFd PkgF(I.FileName(),FileFd::ReadOnly);
-   if (_error->PendingError() == true)
+
+   FileFd PkgF;
+   if (PkgF.Open(I.FileName(), FileFd::ReadOnly, FileFd::Extension) == false)
        return false;
        return false;
-   
-   // Read the record
-   unsigned char *Buffer = new unsigned char[GCache->HeaderP->MaxVerFileSize+1];
-   Buffer[V.FileList()->Size] = '\n';
-   if (PkgF.Seek(V.FileList()->Offset) == false ||
-       PkgF.Read(Buffer,V.FileList()->Size) == false)
+
+   // Read the record (and ensure that it ends with a newline and NUL)
+   unsigned char *Buffer = new unsigned char[Cache->HeaderP->MaxVerFileSize+2];
+   Buffer[Vf->Size] = '\n';
+   Buffer[Vf->Size+1] = '\0';
+   if (PkgF.Seek(Vf->Offset) == false ||
+       PkgF.Read(Buffer,Vf->Size) == false)
     {
        delete [] Buffer;
        return false;
     }
  
     // Get a pointer to start of Description field
     {
        delete [] Buffer;
        return false;
     }
  
     // Get a pointer to start of Description field
-   const unsigned char *DescP = (unsigned char*)strstr((char*)Buffer, "Description:");
+   const unsigned char *DescP = (unsigned char*)strstr((char*)Buffer, "\nDescription");
+   if (DescP != NULL)
+      ++DescP;
+   else
+      DescP = Buffer + Vf->Size;
  
     // Write all but Description
  
     // Write all but Description
-   if (fwrite(Buffer,1,DescP - Buffer,stdout) < (size_t)(DescP - Buffer))
+   size_t const length = DescP - Buffer;
+   if (length != 0 && FileFd::Write(STDOUT_FILENO, Buffer, length) == false)
     {
        delete [] Buffer;
        return false;
     }
  
     // Show the right description
     {
        delete [] Buffer;
        return false;
     }
  
     // Show the right description
-   pkgRecords Recs(*GCache);
+   pkgRecords Recs(*Cache);
     pkgCache::DescIterator Desc = V.TranslatedDescription();
     pkgCache::DescIterator Desc = V.TranslatedDescription();
-   pkgRecords::Parser &P = Recs.Lookup(Desc.FileList());
-   cout << "Description" << ( (strcmp(Desc.LanguageCode(),"") != 0) ? "-" : "" ) << Desc.LanguageCode() << ": " << P.LongDesc();
+   if (Desc.end() == false)
+   {
+      pkgRecords::Parser &P = Recs.Lookup(Desc.FileList());
+      cout << "Description" << ( (strcmp(Desc.LanguageCode(),"") != 0) ? "-" : "" ) << Desc.LanguageCode() << ": " << P.LongDesc();
+      cout << std::endl << "Description-md5: " << Desc.md5() << std::endl;
  
  
-   // Find the first field after the description (if there is any)
-   for(DescP++;DescP != &Buffer[V.FileList()->Size];DescP++) 
+      // Find the first field after the description (if there is any)
+      DescP = skipDescriptionFields(DescP);
+   }
+   // else we have no translation, so we found a lonely Description-md5 -> don't skip it
+
+   // write the rest of the buffer, but skip mixed in Descriptions* fields
+   while (DescP != NULL)
     {
     {
-      if(*DescP == '\n' && *(DescP+1) != ' ') 
+      const unsigned char * const Start = DescP;
+      const unsigned char *End = (unsigned char*)strstr((char*)DescP, "\nDescription");
+      if (End == NULL)
        {
        {
-        // write the rest of the buffer
-        const unsigned char *end=&Buffer[V.FileList()->Size];
-        if (fwrite(DescP,1,end-DescP,stdout) < (size_t)(end-DescP)) 
-        {
-           delete [] Buffer;
-           return false;
-        }
-
-        break;
+        End = &Buffer[Vf->Size];
+        DescP = NULL;
+      }
+      else
+      {
+        ++End; // get the newline into the output
+        DescP = skipDescriptionFields(End + strlen("Description"));
+      }
+      size_t const length = End - Start;
+      if (length != 0 && FileFd::Write(STDOUT_FILENO, Start, length) == false)
+      {
+        delete [] Buffer;
+        return false;
        }
     }
        }
     }
-   // write a final newline (after the description)
+
+   // write a final newline after the last field
     cout<<endl;
     cout<<endl;
-   delete [] Buffer;
  
  
+   delete [] Buffer;
     return true;
  }
                                                                         /*}}}*/
     return true;
  }
                                                                         /*}}}*/
-
  struct ExDescFile
  {
     pkgCache::DescFile *Df;
  struct ExDescFile
  {
     pkgCache::DescFile *Df;
-   bool NameMatch;
+   map_ptrloc ID;
  };
  
  // Search - Perform a search                                           /*{{{*/
  // ---------------------------------------------------------------------
  /* This searches the package names and package descriptions for a pattern */
  };
  
  // Search - Perform a search                                           /*{{{*/
  // ---------------------------------------------------------------------
  /* This searches the package names and package descriptions for a pattern */
-bool Search(CommandLine &CmdL)
+static bool Search(CommandLine &CmdL)
  {
  {
-   pkgCache &Cache = *GCache;
-   bool ShowFull = _config->FindB("APT::Cache::ShowFull",false);
-   bool NamesOnly = _config->FindB("APT::Cache::NamesOnly",false);
-   unsigned NumPatterns = CmdL.FileSize() -1;
-   
-   pkgDepCache::Policy Plcy;
+   bool const ShowFull = _config->FindB("APT::Cache::ShowFull",false);
+   bool const NamesOnly = _config->FindB("APT::Cache::NamesOnly",false);
+   unsigned int const NumPatterns = CmdL.FileSize() -1;
     
     
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   pkgDepCache::Policy *Plcy = CacheFile.GetPolicy();
+   if (unlikely(Cache == NULL || Plcy == NULL))
+      return false;
+
     // Make sure there is at least one argument
     if (NumPatterns < 1)
     // Make sure there is at least one argument
     if (NumPatterns < 1)
-      return _error->Error(_("You must give exactly one pattern"));
+      return _error->Error(_("You must give at least one search pattern"));
     
     // Compile the regex pattern
     regex_t *Patterns = new regex_t[NumPatterns];
     
     // Compile the regex pattern
     regex_t *Patterns = new regex_t[NumPatterns];
@@ -1310,8 +1267,6 @@ bool Search(CommandLine &CmdL)
        }      
     }
     
        }      
     }
     
-   // Create the text record parser
-   pkgRecords Recs(Cache);
     if (_error->PendingError() == true)
     {
        for (unsigned I = 0; I != NumPatterns; I++)
     if (_error->PendingError() == true)
     {
        for (unsigned I = 0; I != NumPatterns; I++)
@@ -1319,70 +1274,106 @@ bool Search(CommandLine &CmdL)
        return false;
     }
     
        return false;
     }
     
-   ExDescFile *DFList = new ExDescFile[Cache.HeaderP->PackageCount+1];
-   memset(DFList,0,sizeof(*DFList)*Cache.HeaderP->PackageCount+1);
+   size_t const descCount = Cache->HeaderP->GroupCount + 1;
+   ExDescFile *DFList = new ExDescFile[descCount];
+   memset(DFList,0,sizeof(*DFList) * descCount);
+
+   bool PatternMatch[descCount * NumPatterns];
+   memset(PatternMatch,false,sizeof(PatternMatch));
  
     // Map versions that we want to write out onto the VerList array.
  
     // Map versions that we want to write out onto the VerList array.
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
+   for (pkgCache::GrpIterator G = Cache->GrpBegin(); G.end() == false; ++G)
     {
     {
-      DFList[P->ID].NameMatch = NumPatterns != 0;
-      for (unsigned I = 0; I != NumPatterns; I++)
+      size_t const PatternOffset = G->ID * NumPatterns;
+      size_t unmatched = 0, matched = 0;
+      for (unsigned I = 0; I < NumPatterns; ++I)
        {
        {
-        if (regexec(&Patterns[I],P.Name(),0,0,0) == 0)
-           DFList[P->ID].NameMatch &= true;
+        if (PatternMatch[PatternOffset + I] == true)
+           ++matched;
+        else if (regexec(&Patterns[I],G.Name(),0,0,0) == 0)
+           PatternMatch[PatternOffset + I] = true;
          else
          else
-           DFList[P->ID].NameMatch = false;
+           ++unmatched;
        }
        }
-        
-      // Doing names only, drop any that dont match..
-      if (NamesOnly == true && DFList[P->ID].NameMatch == false)
+
+      // already dealt with this package?
+      if (matched == NumPatterns)
          continue;
          continue;
-        
-      // Find the proper version to use. 
-      pkgCache::VerIterator V = Plcy.GetCandidateVer(P);
+
+      // Doing names only, drop any that don't match..
+      if (NamesOnly == true && unmatched == NumPatterns)
+        continue;
+
+      // Find the proper version to use
+      pkgCache::PkgIterator P = G.FindPreferredPkg();
+      if (P.end() == true)
+        continue;
+      pkgCache::VerIterator V = Plcy->GetCandidateVer(P);
        if (V.end() == false)
        if (V.end() == false)
-        DFList[P->ID].Df = V.DescriptionList().FileList();
-   }
-      
-   // Include all the packages that provide matching names too
-   for (pkgCache::PkgIterator P = Cache.PkgBegin(); P.end() == false; P++)
-   {
-      if (DFList[P->ID].NameMatch == false)
+      {
+        pkgCache::DescIterator const D = V.TranslatedDescription();
+        //FIXME: packages without a description can't be found
+        if (D.end() == true)
+           continue;
+        DFList[G->ID].Df = D.FileList();
+        DFList[G->ID].ID = G->ID;
+      }
+
+      if (unmatched == NumPatterns)
          continue;
  
          continue;
  
-      for (pkgCache::PrvIterator Prv = P.ProvidesList() ; Prv.end() == false; Prv++)
+      // Include all the packages that provide matching names too
+      for (pkgCache::PrvIterator Prv = P.ProvidesList() ; Prv.end() == false; ++Prv)
        {
        {
-        pkgCache::VerIterator V = Plcy.GetCandidateVer(Prv.OwnerPkg());
-        if (V.end() == false)
-        {
-           DFList[Prv.OwnerPkg()->ID].Df = V.DescriptionList().FileList();
-           DFList[Prv.OwnerPkg()->ID].NameMatch = true;
-        }
+        pkgCache::VerIterator V = Plcy->GetCandidateVer(Prv.OwnerPkg());
+        if (V.end() == true)
+           continue;
+
+        unsigned long id = Prv.OwnerPkg().Group()->ID;
+        pkgCache::DescIterator const D = V.TranslatedDescription();
+        //FIXME: packages without a description can't be found
+        if (D.end() == true)
+           continue;
+        DFList[id].Df = D.FileList();
+        DFList[id].ID = id;
+
+        size_t const PrvPatternOffset = id * NumPatterns;
+        for (unsigned I = 0; I < NumPatterns; ++I)
+           PatternMatch[PrvPatternOffset + I] = PatternMatch[PatternOffset + I];
        }
     }
        }
     }
-   
-   LocalitySort(&DFList->Df,Cache.HeaderP->PackageCount,sizeof(*DFList));
  
  
+   LocalitySort(&DFList->Df,Cache->HeaderP->GroupCount,sizeof(*DFList));
+
+   // Create the text record parser
+   pkgRecords Recs(*Cache);
     // Iterate over all the version records and check them
     // Iterate over all the version records and check them
-   for (ExDescFile *J = DFList; J->Df != 0; J++)
+   for (ExDescFile *J = DFList; J->Df != 0; ++J)
     {
     {
-      pkgRecords::Parser &P = Recs.Lookup(pkgCache::DescFileIterator(Cache,J->Df));
+      pkgRecords::Parser &P = Recs.Lookup(pkgCache::DescFileIterator(*Cache,J->Df));
+      size_t const PatternOffset = J->ID * NumPatterns;
  
  
-      bool Match = true;
-      if (J->NameMatch == false)
+      if (NamesOnly == false)
        {
        {
-        string LongDesc = P.LongDesc();
-        Match = NumPatterns != 0;
-        for (unsigned I = 0; I != NumPatterns; I++)
+        string const LongDesc = P.LongDesc();
+        for (unsigned I = 0; I < NumPatterns; ++I)
          {
          {
-           if (regexec(&Patterns[I],LongDesc.c_str(),0,0,0) == 0)
-              Match &= true;
-           else
-              Match = false;
+           if (PatternMatch[PatternOffset + I] == true)
+              continue;
+           else if (regexec(&Patterns[I],LongDesc.c_str(),0,0,0) == 0)
+              PatternMatch[PatternOffset + I] = true;
          }
        }
          }
        }
-      
-      if (Match == true)
+
+      bool matchedAll = true;
+      for (unsigned I = 0; I < NumPatterns; ++I)
+        if (PatternMatch[PatternOffset + I] == false)
+        {
+           matchedAll = false;
+           break;
+        }
+
+      if (matchedAll == true)
        {
          if (ShowFull == true)
          {
        {
          if (ShowFull == true)
          {
@@ -1405,68 +1396,69 @@ bool Search(CommandLine &CmdL)
     return true;
  }
                                                                         /*}}}*/
     return true;
  }
                                                                         /*}}}*/
-// ShowPackage - Dump the package record to the screen                 /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-bool ShowPackage(CommandLine &CmdL)
-{   
-   pkgCache &Cache = *GCache;
-   pkgDepCache::Policy Plcy;
+/* ShowAuto - show automatically installed packages (sorted)           {{{*/
+static bool ShowAuto(CommandLine &)
+{
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   pkgDepCache *DepCache = CacheFile.GetDepCache();
+   if (unlikely(Cache == NULL || DepCache == NULL))
+      return false;
  
  
-   unsigned found = 0;
+   std::vector<string> packages;
+   packages.reserve(Cache->HeaderP->PackageCount / 3);
     
     
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
-   {
-      // FIXME: Handle the case in which pkgname name:arch is not found
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(*I);
-      if (Pkg.end() == true)
-      {
-        Pkg = Cache.FindPkg(*I, "any");
-        if (Pkg.end() == true) {
-               _error->Warning(_("Unable to locate package %s"),*I);
-               continue;
-        }
-      }
+   for (pkgCache::PkgIterator P = Cache->PkgBegin(); P.end() == false; ++P)
+      if ((*DepCache)[P].Flags & pkgCache::Flag::Auto)
+         packages.push_back(P.Name());
  
  
-      ++found;
+    std::sort(packages.begin(), packages.end());
+    
+    for (vector<string>::iterator I = packages.begin(); I != packages.end(); ++I)
+            cout << *I << "\n";
  
  
-      // Find the proper version to use.
-      if (_config->FindB("APT::Cache::AllVersions","true") == true)
-      {
-        pkgCache::VerIterator V;
-        for (V = Pkg.VersionList(); V.end() == false; V++)
-        {
-           if (DisplayRecord(V) == false)
-              return false;
-        }
-      }
+   _error->Notice(_("This command is deprecated. Please use 'apt-mark showauto' instead."));
+   return true;
+}
+                                                                       /*}}}*/
+// ShowPackage - Dump the package record to the screen                 /*{{{*/
+// ---------------------------------------------------------------------
+/* */
+static bool ShowPackage(CommandLine &CmdL)
+{
+   pkgCacheFile CacheFile;
+   CacheSetHelperVirtuals helper(true, GlobalError::NOTICE);
+   APT::VersionList::Version const select = _config->FindB("APT::Cache::AllVersions", true) ?
+                       APT::VersionList::ALL : APT::VersionList::CANDIDATE;
+   APT::VersionList const verset = APT::VersionList::FromCommandLine(CacheFile, CmdL.FileList + 1, select, helper);
+   for (APT::VersionList::const_iterator Ver = verset.begin(); Ver != verset.end(); ++Ver)
+      if (DisplayRecord(CacheFile, Ver) == false)
+        return false;
+
+   if (verset.empty() == true)
+   {
+      if (helper.virtualPkgs.empty() == true)
+        return _error->Error(_("No packages found"));
        else
        else
-      {
-        pkgCache::VerIterator V = Plcy.GetCandidateVer(Pkg);
-        if (V.end() == true || V.FileList().end() == true)
-           continue;
-        if (DisplayRecord(V) == false)
-           return false;
-      }      
+        _error->Notice(_("No packages found"));
     }
     }
-
-   if (found > 0)
-        return true;
-   return _error->Error(_("No packages found"));
+   return true;
  }
                                                                         /*}}}*/
  // ShowPkgNames - Show package names                                   /*{{{*/
  // ---------------------------------------------------------------------
  /* This does a prefix match on the first argument */
  }
                                                                         /*}}}*/
  // ShowPkgNames - Show package names                                   /*{{{*/
  // ---------------------------------------------------------------------
  /* This does a prefix match on the first argument */
-bool ShowPkgNames(CommandLine &CmdL)
+static bool ShowPkgNames(CommandLine &CmdL)
  {
  {
-   pkgCache &Cache = *GCache;
-   pkgCache::GrpIterator I = Cache.GrpBegin();
+   pkgCacheFile CacheFile;
+   if (unlikely(CacheFile.BuildCaches(NULL, false) == false))
+      return false;
+   pkgCache::GrpIterator I = CacheFile.GetPkgCache()->GrpBegin();
     bool const All = _config->FindB("APT::Cache::AllNames","false");
  
     if (CmdL.FileList[1] != 0)
     {
     bool const All = _config->FindB("APT::Cache::AllNames","false");
  
     if (CmdL.FileList[1] != 0)
     {
-      for (;I.end() != true; I++)
+      for (;I.end() != true; ++I)
        {
          if (All == false && I->FirstPackage == 0)
             continue;
        {
          if (All == false && I->FirstPackage == 0)
             continue;
@@ -1480,7 +1472,7 @@ bool ShowPkgNames(CommandLine &CmdL)
     }
     
     // Show all pkgs
     }
     
     // Show all pkgs
-   for (;I.end() != true; I++)
+   for (;I.end() != true; ++I)
     {
        if (All == false && I->FirstPackage == 0)
          continue;
     {
        if (All == false && I->FirstPackage == 0)
          continue;
@@ -1495,13 +1487,15 @@ bool ShowPkgNames(CommandLine &CmdL)
  // ShowSrcPackage - Show source package records                                /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // ShowSrcPackage - Show source package records                                /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool ShowSrcPackage(CommandLine &CmdL)
+static bool ShowSrcPackage(CommandLine &CmdL)
  {
  {
-   pkgSourceList List;
-   List.ReadMainList();
-   
+   pkgCacheFile CacheFile;
+   pkgSourceList *List = CacheFile.GetSourceList();
+   if (unlikely(List == NULL))
+      return false;
+
     // Create the text record parsers
     // Create the text record parsers
-   pkgSrcRecords SrcRecs(List);
+   pkgSrcRecords SrcRecs(*List);
     if (_error->PendingError() == true)
        return false;
  
     if (_error->PendingError() == true)
        return false;
  
@@ -1521,30 +1515,35 @@ bool ShowSrcPackage(CommandLine &CmdL)
          _error->Warning(_("Unable to locate package %s"),*I);
          continue;
        }
          _error->Warning(_("Unable to locate package %s"),*I);
          continue;
        }
-   }      
-   if (found > 0)
-        return true;
-   return _error->Error(_("No packages found"));
+   }
+   if (found == 0)
+      _error->Notice(_("No packages found"));
+   return true;
  }
                                                                         /*}}}*/
  // Policy - Show the results of the preferences file                   /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  }
                                                                         /*}}}*/
  // Policy - Show the results of the preferences file                   /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool Policy(CommandLine &CmdL)
+static bool Policy(CommandLine &CmdL)
  {
  {
-   if (SrcList == 0)
-      return _error->Error("Generate must be enabled for this function");
-   
-   pkgCache &Cache = *GCache;
-   pkgPolicy Plcy(&Cache);
-   if (ReadPinFile(Plcy) == false || ReadPinDir(Plcy) == false)
+   pkgCacheFile CacheFile;
+   pkgCache *Cache = CacheFile.GetPkgCache();
+   pkgPolicy *Plcy = CacheFile.GetPolicy();
+   pkgSourceList *SrcList = CacheFile.GetSourceList();
+   if (unlikely(Cache == NULL || Plcy == NULL || SrcList == NULL))
        return false;
        return false;
-   
+
+   /* Should the MultiArchKiller be run to see which pseudo packages for an
+      arch all package are currently installed? Activating it gives a speed
+      penality for no real gain beside enhanced debugging, so in general no. */
+   if (_config->FindB("APT::Cache::Policy::DepCache", false) == true)
+      CacheFile.GetDepCache();
+
     // Print out all of the package files
     if (CmdL.FileList[1] == 0)
     {
        cout << _("Package files:") << endl;   
     // Print out all of the package files
     if (CmdL.FileList[1] == 0)
     {
        cout << _("Package files:") << endl;   
-      for (pkgCache::PkgFileIterator F = Cache.FileBegin(); F.end() == false; F++)
+      for (pkgCache::PkgFileIterator F = Cache->FileBegin(); F.end() == false; ++F)
        {
          // Locate the associated index files so we can derive a description
          pkgIndexFile *Indx;
        {
          // Locate the associated index files so we can derive a description
          pkgIndexFile *Indx;
@@ -1553,7 +1552,7 @@ bool Policy(CommandLine &CmdL)
             return _error->Error(_("Cache is out of sync, can't x-ref a package file"));
          
          printf("%4i %s\n",
             return _error->Error(_("Cache is out of sync, can't x-ref a package file"));
          
          printf("%4i %s\n",
-               Plcy.GetPriority(F),Indx->Describe(true).c_str());
+               Plcy->GetPriority(F),Indx->Describe(true).c_str());
          
          // Print the reference information for the package
          string Str = F.RelStr();
          
          // Print the reference information for the package
          string Str = F.RelStr();
@@ -1565,16 +1564,16 @@ bool Policy(CommandLine &CmdL)
        
        // Show any packages have explicit pins
        cout << _("Pinned packages:") << endl;
        
        // Show any packages have explicit pins
        cout << _("Pinned packages:") << endl;
-      pkgCache::PkgIterator I = Cache.PkgBegin();
-      for (;I.end() != true; I++)
+      pkgCache::PkgIterator I = Cache->PkgBegin();
+      for (;I.end() != true; ++I)
        {
        {
-        if (Plcy.GetPriority(I) == 0)
+        if (Plcy->GetPriority(I) == 0)
             continue;
  
          // Print the package name and the version we are forcing to
          cout << "     " << I.FullName(true) << " -> ";
          
             continue;
  
          // Print the package name and the version we are forcing to
          cout << "     " << I.FullName(true) << " -> ";
          
-        pkgCache::VerIterator V = Plcy.GetMatch(I);
+        pkgCache::VerIterator V = Plcy->GetMatch(I);
          if (V.end() == true)
             cout << _("(not found)") << endl;
          else
          if (V.end() == true)
             cout << _("(not found)") << endl;
          else
@@ -1584,45 +1583,42 @@ bool Policy(CommandLine &CmdL)
        return true;
     }
  
        return true;
     }
  
-   string const myArch = _config->Find("APT::Architecture");
+   char const * const msgInstalled = _("  Installed: ");
+   char const * const msgCandidate = _("  Candidate: ");
+   short const InstalledLessCandidate =
+               mbstowcs(NULL, msgInstalled, 0) - mbstowcs(NULL, msgCandidate, 0);
+   short const deepInstalled =
+               (InstalledLessCandidate < 0 ? (InstalledLessCandidate*-1) : 0) - 1;
+   short const deepCandidate =
+               (InstalledLessCandidate > 0 ? (InstalledLessCandidate) : 0) - 1;
  
     // Print out detailed information for each package
  
     // Print out detailed information for each package
-   for (const char **I = CmdL.FileList + 1; *I != 0; I++)
+   APT::CacheSetHelper helper(true, GlobalError::NOTICE);
+   APT::PackageList pkgset = APT::PackageList::FromCommandLine(CacheFile, CmdL.FileList + 1, helper);
+   for (APT::PackageList::const_iterator Pkg = pkgset.begin(); Pkg != pkgset.end(); ++Pkg)
     {
     {
-      pkgCache::GrpIterator Grp = Cache.FindGrp(*I);
-      pkgCache::PkgIterator Pkg = Grp.FindPkg("any");
-      if (Pkg.end() == true)
-      {
-        _error->Warning(_("Unable to locate package %s"),*I);
-        continue;
-      }
-
-      for (; Pkg.end() != true; Pkg = Grp.NextPkg(Pkg)) {
-      if (strcmp(Pkg.Arch(),"all") == 0)
-        continue;
-
        cout << Pkg.FullName(true) << ":" << endl;
  
        // Installed version
        cout << Pkg.FullName(true) << ":" << endl;
  
        // Installed version
-      cout << _("  Installed: ");
+      cout << msgInstalled << OutputInDepth(deepInstalled, " ");
        if (Pkg->CurrentVer == 0)
          cout << _("(none)") << endl;
        else
          cout << Pkg.CurrentVer().VerStr() << endl;
        
        // Candidate Version 
        if (Pkg->CurrentVer == 0)
          cout << _("(none)") << endl;
        else
          cout << Pkg.CurrentVer().VerStr() << endl;
        
        // Candidate Version 
-      cout << _("  Candidate: ");
-      pkgCache::VerIterator V = Plcy.GetCandidateVer(Pkg);
+      cout << msgCandidate << OutputInDepth(deepCandidate, " ");
+      pkgCache::VerIterator V = Plcy->GetCandidateVer(Pkg);
        if (V.end() == true)
          cout << _("(none)") << endl;
        else
          cout << V.VerStr() << endl;
  
        // Pinned version
        if (V.end() == true)
          cout << _("(none)") << endl;
        else
          cout << V.VerStr() << endl;
  
        // Pinned version
-      if (Plcy.GetPriority(Pkg) != 0)
+      if (Plcy->GetPriority(Pkg) != 0)
        {
          cout << _("  Package pin: ");
        {
          cout << _("  Package pin: ");
-        V = Plcy.GetMatch(Pkg);
+        V = Plcy->GetMatch(Pkg);
          if (V.end() == true)
             cout << _("(not found)") << endl;
          else
          if (V.end() == true)
             cout << _("(not found)") << endl;
          else
@@ -1631,25 +1627,24 @@ bool Policy(CommandLine &CmdL)
        
        // Show the priority tables
        cout << _("  Version table:") << endl;
        
        // Show the priority tables
        cout << _("  Version table:") << endl;
-      for (V = Pkg.VersionList(); V.end() == false; V++)
+      for (V = Pkg.VersionList(); V.end() == false; ++V)
        {
          if (Pkg.CurrentVer() == V)
             cout << " *** " << V.VerStr();
          else
             cout << "     " << V.VerStr();
        {
          if (Pkg.CurrentVer() == V)
             cout << " *** " << V.VerStr();
          else
             cout << "     " << V.VerStr();
-        cout << " " << Plcy.GetPriority(Pkg) << endl;
-        for (pkgCache::VerFileIterator VF = V.FileList(); VF.end() == false; VF++)
+        cout << " " << Plcy->GetPriority(Pkg) << endl;
+        for (pkgCache::VerFileIterator VF = V.FileList(); VF.end() == false; ++VF)
          {
             // Locate the associated index files so we can derive a description
             pkgIndexFile *Indx;
             if (SrcList->FindIndex(VF.File(),Indx) == false &&
                 _system->FindIndex(VF.File(),Indx) == false)
                return _error->Error(_("Cache is out of sync, can't x-ref a package file"));
          {
             // Locate the associated index files so we can derive a description
             pkgIndexFile *Indx;
             if (SrcList->FindIndex(VF.File(),Indx) == false &&
                 _system->FindIndex(VF.File(),Indx) == false)
                return _error->Error(_("Cache is out of sync, can't x-ref a package file"));
-           printf("       %4i %s\n",Plcy.GetPriority(VF.File()),
+           printf("       %4i %s\n",Plcy->GetPriority(VF.File()),
                    Indx->Describe(true).c_str());
          }
        }
                    Indx->Describe(true).c_str());
          }
        }
-      }
     }
     
     return true;
     }
     
     return true;
@@ -1658,14 +1653,13 @@ bool Policy(CommandLine &CmdL)
  // Madison - Look a bit like katie's madison                           /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // Madison - Look a bit like katie's madison                           /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool Madison(CommandLine &CmdL)
+static bool Madison(CommandLine &CmdL)
  {
  {
-   if (SrcList == 0)
-      return _error->Error("Generate must be enabled for this function");
-
-   SrcList->ReadMainList();
+   pkgCacheFile CacheFile;
+   pkgSourceList *SrcList = CacheFile.GetSourceList();
  
  
-   pkgCache &Cache = *GCache;
+   if (SrcList == 0)
+      return false;
  
     // Create the src text record parsers and ignore errors about missing
     // deb-src lines that are generated from pkgSrcRecords::pkgSrcRecords
  
     // Create the src text record parsers and ignore errors about missing
     // deb-src lines that are generated from pkgSrcRecords::pkgSrcRecords
@@ -1673,15 +1667,16 @@ bool Madison(CommandLine &CmdL)
     if (_error->PendingError() == true)
        _error->Discard();
  
     if (_error->PendingError() == true)
        _error->Discard();
  
+   APT::CacheSetHelper helper(true, GlobalError::NOTICE);
     for (const char **I = CmdL.FileList + 1; *I != 0; I++)
     {
     for (const char **I = CmdL.FileList + 1; *I != 0; I++)
     {
-      pkgCache::PkgIterator Pkg = Cache.FindPkg(*I);
-
-      if (Pkg.end() == false)
+      _error->PushToStack();
+      APT::PackageList pkgset = APT::PackageList::FromString(CacheFile, *I, helper);
+      for (APT::PackageList::const_iterator Pkg = pkgset.begin(); Pkg != pkgset.end(); ++Pkg)
        {
        {
-         for (pkgCache::VerIterator V = Pkg.VersionList(); V.end() == false; V++)
+         for (pkgCache::VerIterator V = Pkg.VersionList(); V.end() == false; ++V)
           {
           {
-            for (pkgCache::VerFileIterator VF = V.FileList(); VF.end() == false; VF++)
+            for (pkgCache::VerFileIterator VF = V.FileList(); VF.end() == false; ++VF)
              {
  // This might be nice, but wouldn't uniquely identify the source -mdz
  //                if (VF.File().Archive() != 0)
              {
  // This might be nice, but wouldn't uniquely identify the source -mdz
  //                if (VF.File().Archive() != 0)
@@ -1691,11 +1686,11 @@ bool Madison(CommandLine &CmdL)
  //                }
  
                 // Locate the associated index files so we can derive a description
  //                }
  
                 // Locate the associated index files so we can derive a description
-               for (pkgSourceList::const_iterator S = SrcList->begin(); S != SrcList->end(); S++)
+               for (pkgSourceList::const_iterator S = SrcList->begin(); S != SrcList->end(); ++S)
                 {
                      vector<pkgIndexFile *> *Indexes = (*S)->GetIndexFiles();
                      for (vector<pkgIndexFile *>::const_iterator IF = Indexes->begin();
                 {
                      vector<pkgIndexFile *> *Indexes = (*S)->GetIndexFiles();
                      for (vector<pkgIndexFile *>::const_iterator IF = Indexes->begin();
-                         IF != Indexes->end(); IF++)
+                         IF != Indexes->end(); ++IF)
                      {
                           if ((*IF)->FindInCache(*(VF.File().Cache())) == VF.File())
                           {
                      {
                           if ((*IF)->FindInCache(*(VF.File().Cache())) == VF.File())
                           {
@@ -1708,16 +1703,21 @@ bool Madison(CommandLine &CmdL)
           }
        }
  
           }
        }
  
-      
        SrcRecs.Restart();
        pkgSrcRecords::Parser *SrcParser;
        SrcRecs.Restart();
        pkgSrcRecords::Parser *SrcParser;
-      while ((SrcParser = SrcRecs.Find(*I,false)) != 0)
+      bool foundSomething = false;
+      while ((SrcParser = SrcRecs.Find(*I, false)) != 0)
        {
        {
+         foundSomething = true;
           // Maybe support Release info here too eventually
           cout << setw(10) << SrcParser->Package() << " | "
                << setw(10) << SrcParser->Version() << " | "
                << SrcParser->Index().Describe(true) << endl;
        }
           // Maybe support Release info here too eventually
           cout << setw(10) << SrcParser->Package() << " | "
                << setw(10) << SrcParser->Version() << " | "
                << SrcParser->Index().Describe(true) << endl;
        }
+      if (foundSomething == true)
+        _error->RevertToStack();
+      else
+        _error->MergeWithStack();
     }
  
     return true;
     }
  
     return true;
@@ -1726,22 +1726,20 @@ bool Madison(CommandLine &CmdL)
  // GenCaches - Call the main cache generator                           /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  // GenCaches - Call the main cache generator                           /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool GenCaches(CommandLine &Cmd)
+static bool GenCaches(CommandLine &)
  {
     OpTextProgress Progress(*_config);
  {
     OpTextProgress Progress(*_config);
-   
-   pkgSourceList List;
-   if (List.ReadMainList() == false)
-      return false;   
-   return pkgMakeStatusCache(List,Progress);
+
+   pkgCacheFile CacheFile;
+   return CacheFile.BuildCaches(&Progress, true);
  }
                                                                         /*}}}*/
  // ShowHelp - Show a help screen                                       /*{{{*/
  // ---------------------------------------------------------------------
  /* */
  }
                                                                         /*}}}*/
  // ShowHelp - Show a help screen                                       /*{{{*/
  // ---------------------------------------------------------------------
  /* */
-bool ShowHelp(CommandLine &Cmd)
+static bool ShowHelp(CommandLine &)
  {
  {
-   ioprintf(cout,_("%s %s for %s compiled on %s %s\n"),PACKAGE,VERSION,
+   ioprintf(cout,_("%s %s for %s compiled on %s %s\n"),PACKAGE,PACKAGE_VERSION,
             COMMON_ARCH,__DATE__,__TIME__);
     
     if (_config->FindB("version") == true)
             COMMON_ARCH,__DATE__,__TIME__);
     
     if (_config->FindB("version") == true)
@@ -1749,15 +1747,13 @@ bool ShowHelp(CommandLine &Cmd)
  
     cout << 
      _("Usage: apt-cache [options] command\n"
  
     cout << 
      _("Usage: apt-cache [options] command\n"
-      "       apt-cache [options] add file1 [file2 ...]\n"
        "       apt-cache [options] showpkg pkg1 [pkg2 ...]\n"
        "       apt-cache [options] showsrc pkg1 [pkg2 ...]\n"
        "\n"
        "       apt-cache [options] showpkg pkg1 [pkg2 ...]\n"
        "       apt-cache [options] showsrc pkg1 [pkg2 ...]\n"
        "\n"
-      "apt-cache is a low-level tool used to manipulate APT's binary\n"
-      "cache files, and query information from them\n"
+      "apt-cache is a low-level tool used to query information\n"
+      "from APT's binary cache files\n"
        "\n"
        "Commands:\n"
        "\n"
        "Commands:\n"
-      "   add - Add a package file to the source cache\n"
        "   gencaches - Build both the package and source cache\n"
        "   showpkg - Show some general information for a single package\n"
        "   showsrc - Show source records\n"
        "   gencaches - Build both the package and source cache\n"
        "   showpkg - Show some general information for a single package\n"
        "   showsrc - Show source records\n"
@@ -1786,40 +1782,12 @@ bool ShowHelp(CommandLine &Cmd)
     return true;
  }
                                                                         /*}}}*/
     return true;
  }
                                                                         /*}}}*/
-// CacheInitialize - Initialize things for apt-cache                   /*{{{*/
-// ---------------------------------------------------------------------
-/* */
-void CacheInitialize()
-{
-   _config->Set("quiet",0);
-   _config->Set("help",false);
-}
-                                                                       /*}}}*/
  int main(int argc,const char *argv[])                                  /*{{{*/
  {
  int main(int argc,const char *argv[])                                  /*{{{*/
  {
-   CommandLine::Args Args[] = {
-      {'h',"help","help",0},
-      {'v',"version","version",0},
-      {'p',"pkg-cache","Dir::Cache::pkgcache",CommandLine::HasArg},
-      {'s',"src-cache","Dir::Cache::srcpkgcache",CommandLine::HasArg},
-      {'q',"quiet","quiet",CommandLine::IntLevel},
-      {'i',"important","APT::Cache::Important",0},
-      {'f',"full","APT::Cache::ShowFull",0},
-      {'g',"generate","APT::Cache::Generate",0},
-      {'a',"all-versions","APT::Cache::AllVersions",0},
-      {'n',"names-only","APT::Cache::NamesOnly",0},
-      {0,"all-names","APT::Cache::AllNames",0},
-      {0,"recurse","APT::Cache::RecurseDepends",0},
-      {'c',"config-file",0,CommandLine::ConfigFile},
-      {'o',"option",0,CommandLine::ArbItem},
-      {0,"installed","APT::Cache::Installed",0},
-      {0,0,0,0}};
-   CommandLine::Dispatch CmdsA[] = {{"help",&ShowHelp},
-                                    {"add",&DoAdd},
+   CommandLine::Dispatch Cmds[] =  {{"help",&ShowHelp},
                                      {"gencaches",&GenCaches},
                                      {"showsrc",&ShowSrcPackage},
                                      {"gencaches",&GenCaches},
                                      {"showsrc",&ShowSrcPackage},
-                                    {0,0}};
-   CommandLine::Dispatch CmdsB[] = {{"showpkg",&DumpPackage},
+                                    {"showpkg",&DumpPackage},
                                      {"stats",&Stats},
                                      {"dump",&Dump},
                                      {"dumpavail",&DumpAvail},
                                      {"stats",&Stats},
                                      {"dump",&Dump},
                                      {"dumpavail",&DumpAvail},
@@ -1831,18 +1799,19 @@ int main(int argc,const char *argv[])                                   /*{{{*/
                                      {"xvcg",&XVcg},
                                      {"show",&ShowPackage},
                                      {"pkgnames",&ShowPkgNames},
                                      {"xvcg",&XVcg},
                                      {"show",&ShowPackage},
                                      {"pkgnames",&ShowPkgNames},
+                                    {"showauto",&ShowAuto},
                                      {"policy",&Policy},
                                      {"madison",&Madison},
                                      {0,0}};
  
                                      {"policy",&Policy},
                                      {"madison",&Madison},
                                      {0,0}};
  
-   CacheInitialize();
+   std::vector<CommandLine::Args> Args = getCommandArgs("apt-cache", CommandLine::GetCommand(Cmds, argc, argv));
  
     // Set up gettext support
     setlocale(LC_ALL,"");
     textdomain(PACKAGE);
  
     // Parse the command line and initialize the package library
  
     // Set up gettext support
     setlocale(LC_ALL,"");
     textdomain(PACKAGE);
  
     // Parse the command line and initialize the package library
-   CommandLine CmdL(Args,_config);
+   CommandLine CmdL(Args.data(),_config);
     if (pkgInitConfig(*_config) == false ||
         CmdL.Parse(argc,argv) == false ||
         pkgInitSystem(*_config,_system) == false)
     if (pkgInitConfig(*_config) == false ||
         CmdL.Parse(argc,argv) == false ||
         pkgInitSystem(*_config,_system) == false)
@@ -1860,46 +1829,21 @@ int main(int argc,const char *argv[])                                   /*{{{*/
     }
     
     // Deal with stdout not being a tty
     }
     
     // Deal with stdout not being a tty
-   if (isatty(STDOUT_FILENO) && _config->FindI("quiet",0) < 1)
+   if (!isatty(STDOUT_FILENO) && _config->FindI("quiet", -1) == -1)
        _config->Set("quiet","1");
  
        _config->Set("quiet","1");
  
-   if (CmdL.DispatchArg(CmdsA,false) == false && _error->PendingError() == false)
-   { 
-      MMap *Map = 0;
-      if (_config->FindB("APT::Cache::Generate",true) == false)
-      {
-        Map = new MMap(*new FileFd(_config->FindFile("Dir::Cache::pkgcache"),
-                                   FileFd::ReadOnly),MMap::Public|MMap::ReadOnly);
-      }
-      else
-      {
-        // Open the cache file
-        SrcList = new pkgSourceList;
-        SrcList->ReadMainList();
+   if (_config->Exists("APT::Cache::Generate") == true)
+      _config->Set("pkgCacheFile::Generate", _config->FindB("APT::Cache::Generate", true));
+
+   // Match the operation
+   CmdL.DispatchArg(Cmds);
  
  
-        // Generate it and map it
-        OpProgress Prog;
-        pkgMakeStatusCache(*SrcList,Prog,&Map,true);
-      }
-      
-      if (_error->PendingError() == false)
-      {
-        pkgCache Cache(Map);   
-        GCache = &Cache;
-        if (_error->PendingError() == false)
-           CmdL.DispatchArg(CmdsB);
-      }
-      delete Map;
-   }
-   
     // Print any errors or warnings found during parsing
     // Print any errors or warnings found during parsing
-   if (_error->empty() == false)
-   {
-      bool Errors = _error->PendingError();
+   bool const Errors = _error->PendingError();
+   if (_config->FindI("quiet",0) > 0)
        _error->DumpErrors();
        _error->DumpErrors();
-      return Errors == true?100:0;
-   }
-          
-   return 0;
+   else
+      _error->DumpErrors(GlobalError::DEBUG);
+   return Errors == true ? 100 : 0;
  }
                                                                         /*}}}*/
  }
                                                                         /*}}}*/