Improved message for pre-depends/depend loops

Author: jgg
Date: 2002-04-02 06:01:49 GMT
Improved message for pre-depends/depend loops

diff --git a/apt-pkg/packagemanager.cc b/apt-pkg/packagemanager.cc
index fc5f475a19f6f14a67da664a8ed19170b9deb46f..b0ebcdb590b75fb6746e2cc8eb53ea2ae3150f8b 100644 (file)
--- a/apt-pkg/packagemanager.cc
+++ b/apt-pkg/packagemanager.cc
@@ -1,6 +1,6 @@
 // -*- mode: cpp; mode: fold -*-
 // Description                                                         /*{{{*/
 // -*- mode: cpp; mode: fold -*-
 // Description                                                         /*{{{*/

-// $Id: packagemanager.cc,v 1.28 2001/05/27 05:36:04 jgg Exp $
+// $Id: packagemanager.cc,v 1.29 2002/04/02 06:01:49 jgg Exp $

 /* ######################################################################
 
    Package Manager - Abstacts the package manager
 /* ######################################################################
 
    Package Manager - Abstacts the package manager


@@ -483,13 +483,16 @@ bool pkgPackageManager::SmartUnPack(PkgIterator Pkg)
 
            Bad = !SmartConfigure(Pkg);
         }
 
            Bad = !SmartConfigure(Pkg);
         }

-        
+

         /* If this or element did not match then continue on to the
         /* If this or element did not match then continue on to the

-           next or element until a matching element is found*/
+           next or element until a matching element is found */

         if (Bad == true)
         if (Bad == true)

-        {          
+        {
+           // This triggers if someone make a pre-depends/depend loop.

            if (Start == End)
            if (Start == End)

-              return _error->Error("Internal Error, Couldn't configure a pre-depend");
+              return _error->Error("Couldn't configure pre-depend %s for %s, "
+                                   "probably a dependency cycle.",
+                                   End.TargetPkg().Name(),Pkg.Name());

            Start++;
         }
         else
            Start++;
         }
         else