Attempt to skip and resync any seriously broken semantics in package sections.

[apt-legacy.git] / apt-pkg / pkgcachegen.cc

diff --git a/apt-pkg/pkgcachegen.cc b/apt-pkg/pkgcachegen.cc
index 48a79768e01d372b9f465a1374848c6728b3b3af..f20c3e579c0bc602cbb7eb5e00d293e4c4af0cb8 100644 (file)
--- a/apt-pkg/pkgcachegen.cc
+++ b/apt-pkg/pkgcachegen.cc
@@ -108,6 +108,7 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
    debListParser *debian(dynamic_cast<debListParser *>(&List));
 
    unsigned int Counter = 0;
+  step:
    while (List.Step() == true)
    {
       // Get a pointer to the package structure
@@ -120,8 +121,11 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
         return false;
       
       pkgCache::PkgIterator Pkg;
-      if (NewPackage(Pkg,PackageName) == false)
-        return _error->Error(_("Error occurred while processing %s (NewPackage)"),std::string(PackageName).c_str());
+      if (NewPackage(Pkg,PackageName) == false) {
+        _error->Warning(_("Error occurred while processing %s (NewPackage)"),std::string(PackageName).c_str());
+         goto step;
+      }
+
       Counter++;
       if (Counter % 100 == 0 && Progress != 0)
         Progress->Progress(List.Offset());
@@ -141,9 +145,11 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
         // we first process the package, then the descriptions
         // (this has the bonus that we get MMap error when we run out
         //  of MMap space)
-        if (List.UsePackage(Pkg,pkgCache::VerIterator(Cache)) == false)
-           return _error->Error(_("Error occurred while processing %s (UsePackage1)"),
+        if (List.UsePackage(Pkg,pkgCache::VerIterator(Cache)) == false) {
+           _error->Warning(_("Error occurred while processing %s (UsePackage1)"),
                                 std::string(PackageName).c_str());
+            goto step;
+         }
 
         // Find the right version to write the description
         MD5SumValue CurMd5 = List.Description_md5();
@@ -175,8 +181,10 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
                  *LastDesc = NewDescription(Desc, language, CurMd5, *LastDesc);
                  Desc->ParentPkg = Pkg.Index();
                  
-                 if ((*LastDesc == 0 && _error->PendingError()) || NewFileDesc(Desc,List) == false)
-                    return _error->Error(_("Error occurred while processing %s (NewFileDesc1)"),std::string(PackageName).c_str());
+                 if ((*LastDesc == 0 && _error->PendingError()) || NewFileDesc(Desc,List) == false) {
+                    _error->Warning(_("Error occurred while processing %s (NewFileDesc1)"),std::string(PackageName).c_str());
+                     goto step;
+                  }
                  break;
               }
            }
@@ -200,13 +208,17 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
       unsigned long Hash = List.VersionHash();
       if (Res == 0 && Ver->Hash == Hash)
       {
-        if (List.UsePackage(Pkg,Ver) == false)
-           return _error->Error(_("Error occurred while processing %s (UsePackage2)"),
+        if (List.UsePackage(Pkg,Ver) == false) {
+           _error->Warning(_("Error occurred while processing %s (UsePackage2)"),
                                 std::string(PackageName).c_str());
+            goto step;
+         }
 
-        if (NewFileVer(Ver,List) == false)
-           return _error->Error(_("Error occurred while processing %s (NewFileVer1)"),
+        if (NewFileVer(Ver,List) == false) {
+           _error->Warning(_("Error occurred while processing %s (NewFileVer1)"),
                                 std::string(PackageName).c_str());
+            goto step;
+         }
         
         // Read only a single record and return
         if (OutVer != 0)
@@ -235,17 +247,23 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
       Ver->ParentPkg = Pkg.Index();
       Ver->Hash = Hash;
 
-      if ((*LastVer == 0 && _error->PendingError()) || List.NewVersion(Ver) == false)
-        return _error->Error(_("Error occurred while processing %s (NewVersion1)"),
+      if ((*LastVer == 0 && _error->PendingError()) || List.NewVersion(Ver) == false) {
+        _error->Warning(_("Error occurred while processing %s (NewVersion1)"),
                              std::string(PackageName).c_str());
+         goto step;
+      }
 
-      if (List.UsePackage(Pkg,Ver) == false)
-        return _error->Error(_("Error occurred while processing %s (UsePackage3)"),
+      if (List.UsePackage(Pkg,Ver) == false) {
+        _error->Warning(_("Error occurred while processing %s (UsePackage3)"),
                              std::string(PackageName).c_str());
+         goto step;
+      }
       
-      if (NewFileVer(Ver,List) == false)
-        return _error->Error(_("Error occurred while processing %s (NewVersion2)"),
+      if (NewFileVer(Ver,List) == false) {
+        _error->Warning(_("Error occurred while processing %s (NewVersion2)"),
                              std::string(PackageName).c_str());
+         goto step;
+      }
 
       // Read only a single record and return
       if (OutVer != 0)
@@ -267,8 +285,10 @@ bool pkgCacheGenerator::MergeList(ListParser &List,
       *LastDesc = NewDescription(Desc, language, List.Description_md5(), *LastDesc);
       Desc->ParentPkg = Pkg.Index();
 
-      if ((*LastDesc == 0 && _error->PendingError()) || NewFileDesc(Desc,List) == false)
-        return _error->Error(_("Error occurred while processing %s (NewFileDesc2)"),std::string(PackageName).c_str());
+      if ((*LastDesc == 0 && _error->PendingError()) || NewFileDesc(Desc,List) == false) {
+        _error->Warning(_("Error occurred while processing %s (NewFileDesc2)"),std::string(PackageName).c_str());
+         goto step;
+      }
    }
 
    FoundFileDeps |= List.HasFileDeps();