summaryrefslogtreecommitdiff
path: root/apt-pkg/packagemanager.cc
diff options
context:
space:
mode:
Diffstat (limited to 'apt-pkg/packagemanager.cc')
-rw-r--r--apt-pkg/packagemanager.cc116
1 files changed, 113 insertions, 3 deletions
diff --git a/apt-pkg/packagemanager.cc b/apt-pkg/packagemanager.cc
index fe9f6eb68..7fcaa8d41 100644
--- a/apt-pkg/packagemanager.cc
+++ b/apt-pkg/packagemanager.cc
@@ -278,7 +278,7 @@ bool pkgPackageManager::ConfigureAll()
{
PkgIterator Pkg(Cache,*I);
- if (ConfigurePkgs == true && Configure(Pkg) == false)
+ if (ConfigurePkgs == true && VerifyAndConfigure(Pkg,OList) == false)
return false;
List->Flag(Pkg,pkgOrderList::Configured,pkgOrderList::States);
@@ -313,7 +313,7 @@ bool pkgPackageManager::SmartConfigure(PkgIterator Pkg)
{
PkgIterator Pkg(Cache,*I);
- if (ConfigurePkgs == true && Configure(Pkg) == false)
+ if (ConfigurePkgs == true && VerifyAndConfigure(Pkg,OList) == false)
return false;
List->Flag(Pkg,pkgOrderList::Configured,pkgOrderList::States);
@@ -337,6 +337,91 @@ bool pkgPackageManager::SmartConfigure(PkgIterator Pkg)
return true;
}
+
+// PM::VerifyConfigure - Check configuration of dependancies /*{{{*/
+// ---------------------------------------------------------------------
+/* This routine checks that all a packages dependancies have been
+ configured, before it is going to be configured. If this gives a warning
+ on a virtual package, it means that the package thats providing it is not
+ configured*/
+bool pkgPackageManager::VerifyConfigure(PkgIterator Pkg, pkgOrderList &OList)
+{
+ // If this is true at the end, then the package should not be configured
+ bool error=true;
+ // This holds the the OR status of the previous dependancy
+ bool previousOr=false;
+
+ // First iterate through the dependancies of Pkg
+ for (DepIterator D = Cache[Pkg].InstVerIter(Cache).DependsList(); D.end() == false; D++)
+ {
+
+ /* If the dependancy is of type Depends or PreDepends, we need to check it, but only if it is going to be
+ configured at some point */
+ if (D->Type == pkgCache::Dep::Depends || D->Type == pkgCache::Dep::PreDepends) {
+
+ /* If the previous package and this package are OR dependancies, and the previous package satisfied the dependancy
+ then skip this dependancy as it is not relevent, this will repeat for the next package if the situation is the
+ same */
+ if (previousOr && !error) { // As error has not been reset, this refers to the previous dependancy
+ previousOr = (D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or;
+ continue;
+ }
+
+ // Reset error
+ error = true;
+
+ // Check thorugh all possible versions of this dependancy (D)
+ SPtrArray<Version *> VList = D.AllTargets();
+ for (Version **I = VList; *I != 0; I++)
+ {
+ VerIterator DepVer(Cache,*I);
+ PkgIterator DepPkg = DepVer.ParentPkg();
+ VerIterator DepInstallVer(Cache,Cache[DepPkg].InstallVer);
+
+ if (DepPkg.CurrentVer() == DepVer && !List->IsFlag(DepPkg,pkgOrderList::UnPacked)) {
+ error=false;
+ break;
+ }
+
+ if (Cache[DepPkg].InstallVer == DepVer &&
+ (List->IsFlag(DepPkg,pkgOrderList::Configured) || OList.IsFlag(DepPkg,pkgOrderList::InList))) {
+ error=false;
+ break;
+ }
+ }
+
+ /* Only worry here if this package is a OR with the next, as even though this package does not satisfy the OR
+ the next one might */
+ if (error && !((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or)) {
+ _error->Error("Package %s should not be configured because package %s is not configured",Pkg.Name(),D.TargetPkg().Name());
+ return false;
+ /* If the previous package is a OR but not this package, but there is still an error then fail as it will not
+ be satisfied */
+ } else if (error && previousOr && !((D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or)) {
+ _error->Error("Package %s should not be configured because package %s (or any alternatives) are not configured",Pkg.Name(),D.TargetPkg().Name());
+ return false;
+ }
+
+ previousOr = (D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or;
+ } else {
+ previousOr=false;
+ }
+ }
+ return true;
+}
+
+// PM::VerifyAndConfigure - Check configuration of dependancies /*{{{*/
+// ---------------------------------------------------------------------
+/* This routine verifies if a package can be configured and if so
+ configures it */
+bool pkgPackageManager::VerifyAndConfigure(PkgIterator Pkg, pkgOrderList &OList)
+{
+ if (VerifyConfigure(Pkg, OList))
+ return Configure(Pkg);
+ else
+ return false;
+
+}
/*}}}*/
// PM::DepAdd - Add all dependents to the oder list /*{{{*/
// ---------------------------------------------------------------------
@@ -391,6 +476,13 @@ bool pkgPackageManager::DepAdd(pkgOrderList &OList,PkgIterator Pkg,int Depth)
continue;
}
+ // Check if this package is being re-installed
+ if ((Cache[Pkg].iFlags & pkgDepCache::ReInstall) == pkgDepCache::ReInstall && Cache[Pkg].InstallVer != *I &&
+ List->IsNow(Pkg) == true && Pkg.State() == PkgIterator::NeedsNothing) {
+ Bad = false;
+ continue;
+ }
+
// Not the install version
if (Cache[Pkg].InstallVer != *I ||
(Cache[Pkg].Keep() == true && Pkg.State() == PkgIterator::NeedsNothing))
@@ -490,6 +582,9 @@ bool pkgPackageManager::SmartUnPack(PkgIterator Pkg)
}
bool pkgPackageManager::SmartUnPack(PkgIterator Pkg, bool const Immediate)
{
+ if (Debug == true)
+ clog << "SmartUnPack " << Pkg.Name() << endl;
+
// Check if it is already unpacked
if (Pkg.State() == pkgCache::PkgIterator::NeedsConfigure &&
Cache[Pkg].Keep() == true)
@@ -589,6 +684,21 @@ bool pkgPackageManager::SmartUnPack(PkgIterator Pkg, bool const Immediate)
}
}
}
+
+ // Check for breaks
+ if (End->Type == pkgCache::Dep::DpkgBreaks) {
+ SPtrArray<Version *> VList = End.AllTargets();
+ for (Version **I = VList; *I != 0; I++)
+ {
+ VerIterator Ver(Cache,*I);
+ PkgIterator Pkg = Ver.ParentPkg();
+ // Check if it needs to be unpacked
+ if (List->IsFlag(Pkg,pkgOrderList::InList) && Cache[Pkg].Delete() == false) {
+ // Found a break, so unpack the package
+ SmartUnPack(Pkg, false);
+ }
+ }
+ }
}
// Check for reverse conflicts.
@@ -741,4 +851,4 @@ pkgPackageManager::OrderResult pkgPackageManager::DoInstall(int statusFd)
return DoInstallPostFork(statusFd);
}
- /*}}}*/
+ /*}}}*/