]>
Commit | Line | Data |
---|---|---|
6c139d6e AL |
1 | // -*- mode: cpp; mode: fold -*- |
2 | // Description /*{{{*/ | |
b8c0f9b7 | 3 | // $Id: algorithms.cc,v 1.44 2002/11/28 18:49:16 jgg Exp $ |
6c139d6e AL |
4 | /* ###################################################################### |
5 | ||
6 | Algorithms - A set of misc algorithms | |
7 | ||
0a8e3465 AL |
8 | The pkgProblemResolver class has become insanely complex and |
9 | very sophisticated, it handles every test case I have thrown at it | |
10 | to my satisfaction. Understanding exactly why all the steps the class | |
11 | does are required is difficult and changing though not very risky | |
12 | may result in other cases not working. | |
13 | ||
6c139d6e AL |
14 | ##################################################################### */ |
15 | /*}}}*/ | |
16 | // Include Files /*{{{*/ | |
ea542140 DK |
17 | #include <config.h> |
18 | ||
094a497d AL |
19 | #include <apt-pkg/algorithms.h> |
20 | #include <apt-pkg/error.h> | |
0a8e3465 | 21 | #include <apt-pkg/configuration.h> |
0a57c0f0 | 22 | #include <apt-pkg/version.h> |
b2e465d6 | 23 | #include <apt-pkg/sptr.h> |
760d4968 | 24 | #include <apt-pkg/acquire-item.h> |
c3b85126 | 25 | #include <apt-pkg/edsp.h> |
472ff00e DK |
26 | #include <apt-pkg/sourcelist.h> |
27 | #include <apt-pkg/fileutl.h> | |
28 | #include <apt-pkg/progress.h> | |
6d38011b | 29 | |
22dcc318 | 30 | #include <sys/types.h> |
152ab79e MV |
31 | #include <cstdlib> |
32 | #include <algorithm> | |
90f057fd | 33 | #include <iostream> |
6d38011b | 34 | #include <stdio.h> |
ea542140 DK |
35 | |
36 | #include <apti18n.h> | |
6c139d6e | 37 | /*}}}*/ |
584e4558 | 38 | using namespace std; |
6c139d6e AL |
39 | |
40 | pkgProblemResolver *pkgProblemResolver::This = 0; | |
41 | ||
42 | // Simulate::Simulate - Constructor /*{{{*/ | |
43 | // --------------------------------------------------------------------- | |
b2e465d6 AL |
44 | /* The legacy translations here of input Pkg iterators is obsolete, |
45 | this is not necessary since the pkgCaches are fully shared now. */ | |
46 | pkgSimulate::pkgSimulate(pkgDepCache *Cache) : pkgPackageManager(Cache), | |
47 | iPolicy(Cache), | |
496d5c70 MV |
48 | Sim(&Cache->GetCache(),&iPolicy), |
49 | group(Sim) | |
6c139d6e | 50 | { |
b2e465d6 AL |
51 | Sim.Init(0); |
52 | Flags = new unsigned char[Cache->Head().PackageCount]; | |
53 | memset(Flags,0,sizeof(*Flags)*Cache->Head().PackageCount); | |
281daf46 AL |
54 | |
55 | // Fake a filename so as not to activate the media swapping | |
56 | string Jnk = "SIMULATE"; | |
b2e465d6 | 57 | for (unsigned int I = 0; I != Cache->Head().PackageCount; I++) |
281daf46 | 58 | FileNames[I] = Jnk; |
6c139d6e AL |
59 | } |
60 | /*}}}*/ | |
b270388b MV |
61 | // Simulate::~Simulate - Destructor /*{{{*/ |
62 | pkgSimulate::~pkgSimulate() | |
63 | { | |
64 | delete[] Flags; | |
65 | } | |
66 | /*}}}*/ | |
b2e465d6 AL |
67 | // Simulate::Describe - Describe a package /*{{{*/ |
68 | // --------------------------------------------------------------------- | |
3826564e MZ |
69 | /* Parameter Current == true displays the current package version, |
70 | Parameter Candidate == true displays the candidate package version */ | |
71 | void pkgSimulate::Describe(PkgIterator Pkg,ostream &out,bool Current,bool Candidate) | |
b2e465d6 AL |
72 | { |
73 | VerIterator Ver(Sim); | |
e59458f7 | 74 | |
47f6d1b7 | 75 | out << Pkg.FullName(true); |
e59458f7 | 76 | |
3826564e | 77 | if (Current == true) |
e59458f7 | 78 | { |
b2e465d6 | 79 | Ver = Pkg.CurrentVer(); |
e59458f7 AL |
80 | if (Ver.end() == false) |
81 | out << " [" << Ver.VerStr() << ']'; | |
82 | } | |
b2e465d6 | 83 | |
3826564e MZ |
84 | if (Candidate == true) |
85 | { | |
86 | Ver = Sim[Pkg].CandidateVerIter(Sim); | |
87 | if (Ver.end() == true) | |
88 | return; | |
b2e465d6 | 89 | |
3826564e MZ |
90 | out << " (" << Ver.VerStr() << ' ' << Ver.RelStr() << ')'; |
91 | } | |
b2e465d6 AL |
92 | } |
93 | /*}}}*/ | |
6c139d6e AL |
94 | // Simulate::Install - Simulate unpacking of a package /*{{{*/ |
95 | // --------------------------------------------------------------------- | |
96 | /* */ | |
97 | bool pkgSimulate::Install(PkgIterator iPkg,string /*File*/) | |
98 | { | |
99 | // Adapt the iterator | |
803ea2a8 | 100 | PkgIterator Pkg = Sim.FindPkg(iPkg.Name(), iPkg.Arch()); |
6c139d6e AL |
101 | Flags[Pkg->ID] = 1; |
102 | ||
b2e465d6 | 103 | cout << "Inst "; |
3826564e | 104 | Describe(Pkg,cout,true,true); |
6c139d6e | 105 | Sim.MarkInstall(Pkg,false); |
803ea2a8 | 106 | |
6c139d6e | 107 | // Look for broken conflicts+predepends. |
f7f0d6c7 | 108 | for (PkgIterator I = Sim.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
109 | { |
110 | if (Sim[I].InstallVer == 0) | |
111 | continue; | |
112 | ||
b2e465d6 AL |
113 | for (DepIterator D = Sim[I].InstVerIter(Sim).DependsList(); D.end() == false;) |
114 | { | |
115 | DepIterator Start; | |
116 | DepIterator End; | |
117 | D.GlobOr(Start,End); | |
359e46db | 118 | if (Start.IsNegative() == true || |
b2e465d6 | 119 | End->Type == pkgCache::Dep::PreDepends) |
6c139d6e | 120 | { |
b2e465d6 | 121 | if ((Sim[End] & pkgDepCache::DepGInstall) == 0) |
6c139d6e | 122 | { |
47f6d1b7 | 123 | cout << " [" << I.FullName(false) << " on " << Start.TargetPkg().FullName(false) << ']'; |
b2e465d6 | 124 | if (Start->Type == pkgCache::Dep::Conflicts) |
47f6d1b7 | 125 | _error->Error("Fatal, conflicts violated %s",I.FullName(false).c_str()); |
6c139d6e | 126 | } |
b2e465d6 AL |
127 | } |
128 | } | |
6c139d6e AL |
129 | } |
130 | ||
131 | if (Sim.BrokenCount() != 0) | |
132 | ShortBreaks(); | |
133 | else | |
04aa15a8 | 134 | cout << endl; |
6c139d6e AL |
135 | return true; |
136 | } | |
137 | /*}}}*/ | |
138 | // Simulate::Configure - Simulate configuration of a Package /*{{{*/ | |
139 | // --------------------------------------------------------------------- | |
140 | /* This is not an acurate simulation of relatity, we should really not | |
141 | install the package.. For some investigations it may be necessary | |
142 | however. */ | |
143 | bool pkgSimulate::Configure(PkgIterator iPkg) | |
144 | { | |
145 | // Adapt the iterator | |
803ea2a8 | 146 | PkgIterator Pkg = Sim.FindPkg(iPkg.Name(), iPkg.Arch()); |
6c139d6e AL |
147 | |
148 | Flags[Pkg->ID] = 2; | |
803ea2a8 | 149 | |
6c139d6e AL |
150 | if (Sim[Pkg].InstBroken() == true) |
151 | { | |
47f6d1b7 | 152 | cout << "Conf " << Pkg.FullName(false) << " broken" << endl; |
6c139d6e AL |
153 | |
154 | Sim.Update(); | |
155 | ||
156 | // Print out each package and the failed dependencies | |
f7f0d6c7 | 157 | for (pkgCache::DepIterator D = Sim[Pkg].InstVerIter(Sim).DependsList(); D.end() == false; ++D) |
6c139d6e AL |
158 | { |
159 | if (Sim.IsImportantDep(D) == false || | |
160 | (Sim[D] & pkgDepCache::DepInstall) != 0) | |
161 | continue; | |
162 | ||
b2e465d6 | 163 | if (D->Type == pkgCache::Dep::Obsoletes) |
47f6d1b7 | 164 | cout << " Obsoletes:" << D.TargetPkg().FullName(false); |
b2e465d6 | 165 | else if (D->Type == pkgCache::Dep::Conflicts) |
47f6d1b7 | 166 | cout << " Conflicts:" << D.TargetPkg().FullName(false); |
308c7d30 | 167 | else if (D->Type == pkgCache::Dep::DpkgBreaks) |
47f6d1b7 | 168 | cout << " Breaks:" << D.TargetPkg().FullName(false); |
6c139d6e | 169 | else |
47f6d1b7 | 170 | cout << " Depends:" << D.TargetPkg().FullName(false); |
6c139d6e | 171 | } |
04aa15a8 | 172 | cout << endl; |
6c139d6e | 173 | |
09a10f9c | 174 | _error->Error("Conf Broken %s",Pkg.FullName(false).c_str()); |
6c139d6e AL |
175 | } |
176 | else | |
b2e465d6 AL |
177 | { |
178 | cout << "Conf "; | |
3826564e | 179 | Describe(Pkg,cout,false,true); |
b2e465d6 | 180 | } |
6c139d6e AL |
181 | |
182 | if (Sim.BrokenCount() != 0) | |
183 | ShortBreaks(); | |
184 | else | |
04aa15a8 | 185 | cout << endl; |
6c139d6e AL |
186 | |
187 | return true; | |
188 | } | |
189 | /*}}}*/ | |
190 | // Simulate::Remove - Simulate the removal of a package /*{{{*/ | |
191 | // --------------------------------------------------------------------- | |
192 | /* */ | |
fc4b5c9f | 193 | bool pkgSimulate::Remove(PkgIterator iPkg,bool Purge) |
6c139d6e AL |
194 | { |
195 | // Adapt the iterator | |
803ea2a8 | 196 | PkgIterator Pkg = Sim.FindPkg(iPkg.Name(), iPkg.Arch()); |
c919ad6e DK |
197 | if (Pkg.end() == true) |
198 | { | |
199 | std::cerr << (Purge ? "Purg" : "Remv") << " invalid package " << iPkg.FullName() << std::endl; | |
200 | return false; | |
201 | } | |
6c139d6e AL |
202 | |
203 | Flags[Pkg->ID] = 3; | |
204 | Sim.MarkDelete(Pkg); | |
803ea2a8 | 205 | |
fc4b5c9f | 206 | if (Purge == true) |
b2e465d6 | 207 | cout << "Purg "; |
fc4b5c9f | 208 | else |
b2e465d6 | 209 | cout << "Remv "; |
3826564e | 210 | Describe(Pkg,cout,true,false); |
6c139d6e AL |
211 | |
212 | if (Sim.BrokenCount() != 0) | |
213 | ShortBreaks(); | |
214 | else | |
04aa15a8 | 215 | cout << endl; |
6c139d6e AL |
216 | |
217 | return true; | |
218 | } | |
219 | /*}}}*/ | |
220 | // Simulate::ShortBreaks - Print out a short line describing all breaks /*{{{*/ | |
221 | // --------------------------------------------------------------------- | |
222 | /* */ | |
223 | void pkgSimulate::ShortBreaks() | |
224 | { | |
04aa15a8 | 225 | cout << " ["; |
f7f0d6c7 | 226 | for (PkgIterator I = Sim.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
227 | { |
228 | if (Sim[I].InstBroken() == true) | |
229 | { | |
230 | if (Flags[I->ID] == 0) | |
47f6d1b7 | 231 | cout << I.FullName(false) << ' '; |
6c139d6e | 232 | /* else |
04aa15a8 | 233 | cout << I.Name() << "! ";*/ |
6c139d6e AL |
234 | } |
235 | } | |
04aa15a8 | 236 | cout << ']' << endl; |
6c139d6e AL |
237 | } |
238 | /*}}}*/ | |
239 | // ApplyStatus - Adjust for non-ok packages /*{{{*/ | |
240 | // --------------------------------------------------------------------- | |
241 | /* We attempt to change the state of the all packages that have failed | |
242 | installation toward their real state. The ordering code will perform | |
243 | the necessary calculations to deal with the problems. */ | |
244 | bool pkgApplyStatus(pkgDepCache &Cache) | |
245 | { | |
74a05226 MV |
246 | pkgDepCache::ActionGroup group(Cache); |
247 | ||
f7f0d6c7 | 248 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e | 249 | { |
e481d5b0 AL |
250 | if (I->VersionList == 0) |
251 | continue; | |
252 | ||
d38b7b3d AL |
253 | // Only choice for a ReInstReq package is to reinstall |
254 | if (I->InstState == pkgCache::State::ReInstReq || | |
255 | I->InstState == pkgCache::State::HoldReInstReq) | |
256 | { | |
5871718b | 257 | if (I->CurrentVer != 0 && I.CurrentVer().Downloadable() == true) |
74a05226 | 258 | Cache.MarkKeep(I, false, false); |
813c8eea AL |
259 | else |
260 | { | |
261 | // Is this right? Will dpkg choke on an upgrade? | |
2a3f3893 AL |
262 | if (Cache[I].CandidateVer != 0 && |
263 | Cache[I].CandidateVerIter(Cache).Downloadable() == true) | |
74a05226 | 264 | Cache.MarkInstall(I, false, 0, false); |
813c8eea | 265 | else |
b2e465d6 | 266 | return _error->Error(_("The package %s needs to be reinstalled, " |
09a10f9c | 267 | "but I can't find an archive for it."),I.FullName(true).c_str()); |
813c8eea AL |
268 | } |
269 | ||
d38b7b3d AL |
270 | continue; |
271 | } | |
272 | ||
6c139d6e AL |
273 | switch (I->CurrentState) |
274 | { | |
813c8eea AL |
275 | /* This means installation failed somehow - it does not need to be |
276 | re-unpacked (probably) */ | |
b518cca6 AL |
277 | case pkgCache::State::UnPacked: |
278 | case pkgCache::State::HalfConfigured: | |
9d06bc80 MV |
279 | case pkgCache::State::TriggersAwaited: |
280 | case pkgCache::State::TriggersPending: | |
5871718b | 281 | if ((I->CurrentVer != 0 && I.CurrentVer().Downloadable() == true) || |
813c8eea | 282 | I.State() != pkgCache::PkgIterator::NeedsUnpack) |
74a05226 | 283 | Cache.MarkKeep(I, false, false); |
813c8eea AL |
284 | else |
285 | { | |
2a3f3893 AL |
286 | if (Cache[I].CandidateVer != 0 && |
287 | Cache[I].CandidateVerIter(Cache).Downloadable() == true) | |
74a05226 | 288 | Cache.MarkInstall(I, true, 0, false); |
813c8eea | 289 | else |
b83cad32 | 290 | Cache.MarkDelete(I, false, 0, false); |
813c8eea | 291 | } |
6c139d6e AL |
292 | break; |
293 | ||
294 | // This means removal failed | |
b518cca6 | 295 | case pkgCache::State::HalfInstalled: |
b83cad32 | 296 | Cache.MarkDelete(I, false, 0, false); |
6c139d6e AL |
297 | break; |
298 | ||
299 | default: | |
b518cca6 | 300 | if (I->InstState != pkgCache::State::Ok) |
6c139d6e | 301 | return _error->Error("The package %s is not ok and I " |
09a10f9c | 302 | "don't know how to fix it!",I.FullName(false).c_str()); |
6c139d6e AL |
303 | } |
304 | } | |
305 | return true; | |
306 | } | |
307 | /*}}}*/ | |
308 | // FixBroken - Fix broken packages /*{{{*/ | |
309 | // --------------------------------------------------------------------- | |
0a8e3465 AL |
310 | /* This autoinstalls every broken package and then runs the problem resolver |
311 | on the result. */ | |
6c139d6e AL |
312 | bool pkgFixBroken(pkgDepCache &Cache) |
313 | { | |
74a05226 MV |
314 | pkgDepCache::ActionGroup group(Cache); |
315 | ||
6c139d6e | 316 | // Auto upgrade all broken packages |
f7f0d6c7 | 317 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e | 318 | if (Cache[I].NowBroken() == true) |
74a05226 | 319 | Cache.MarkInstall(I, true, 0, false); |
7e798dd7 | 320 | |
6c139d6e AL |
321 | /* Fix packages that are in a NeedArchive state but don't have a |
322 | downloadable install version */ | |
f7f0d6c7 | 323 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
324 | { |
325 | if (I.State() != pkgCache::PkgIterator::NeedsUnpack || | |
326 | Cache[I].Delete() == true) | |
327 | continue; | |
328 | ||
b518cca6 | 329 | if (Cache[I].InstVerIter(Cache).Downloadable() == false) |
6c139d6e AL |
330 | continue; |
331 | ||
74a05226 | 332 | Cache.MarkInstall(I, true, 0, false); |
6c139d6e AL |
333 | } |
334 | ||
b2e465d6 | 335 | pkgProblemResolver Fix(&Cache); |
6c139d6e AL |
336 | return Fix.Resolve(true); |
337 | } | |
338 | /*}}}*/ | |
6c139d6e AL |
339 | // ProblemResolver::pkgProblemResolver - Constructor /*{{{*/ |
340 | // --------------------------------------------------------------------- | |
341 | /* */ | |
dcaa1185 | 342 | pkgProblemResolver::pkgProblemResolver(pkgDepCache *pCache) : d(NULL), Cache(*pCache) |
6c139d6e AL |
343 | { |
344 | // Allocate memory | |
b2e465d6 | 345 | unsigned long Size = Cache.Head().PackageCount; |
d0f2c87c | 346 | Scores = new int[Size]; |
6c139d6e AL |
347 | Flags = new unsigned char[Size]; |
348 | memset(Flags,0,sizeof(*Flags)*Size); | |
349 | ||
350 | // Set debug to true to see its decision logic | |
0a8e3465 | 351 | Debug = _config->FindB("Debug::pkgProblemResolver",false); |
6c139d6e AL |
352 | } |
353 | /*}}}*/ | |
b2e465d6 AL |
354 | // ProblemResolver::~pkgProblemResolver - Destructor /*{{{*/ |
355 | // --------------------------------------------------------------------- | |
356 | /* */ | |
357 | pkgProblemResolver::~pkgProblemResolver() | |
358 | { | |
359 | delete [] Scores; | |
360 | delete [] Flags; | |
361 | } | |
362 | /*}}}*/ | |
6c139d6e AL |
363 | // ProblemResolver::ScoreSort - Sort the list by score /*{{{*/ |
364 | // --------------------------------------------------------------------- | |
365 | /* */ | |
366 | int pkgProblemResolver::ScoreSort(const void *a,const void *b) | |
367 | { | |
368 | Package const **A = (Package const **)a; | |
369 | Package const **B = (Package const **)b; | |
370 | if (This->Scores[(*A)->ID] > This->Scores[(*B)->ID]) | |
371 | return -1; | |
372 | if (This->Scores[(*A)->ID] < This->Scores[(*B)->ID]) | |
373 | return 1; | |
374 | return 0; | |
375 | } | |
376 | /*}}}*/ | |
377 | // ProblemResolver::MakeScores - Make the score table /*{{{*/ | |
378 | // --------------------------------------------------------------------- | |
379 | /* */ | |
380 | void pkgProblemResolver::MakeScores() | |
381 | { | |
b2e465d6 | 382 | unsigned long Size = Cache.Head().PackageCount; |
6c139d6e AL |
383 | memset(Scores,0,sizeof(*Scores)*Size); |
384 | ||
b9179170 MV |
385 | // maps to pkgCache::State::VerPriority: |
386 | // Required Important Standard Optional Extra | |
d0f2c87c | 387 | int PrioMap[] = { |
8b4894fe | 388 | 0, |
84de0cea | 389 | _config->FindI("pkgProblemResolver::Scores::Required",3), |
b9179170 | 390 | _config->FindI("pkgProblemResolver::Scores::Important",2), |
d0f2c87c CW |
391 | _config->FindI("pkgProblemResolver::Scores::Standard",1), |
392 | _config->FindI("pkgProblemResolver::Scores::Optional",-1), | |
393 | _config->FindI("pkgProblemResolver::Scores::Extra",-2) | |
8b4894fe | 394 | }; |
d0f2c87c CW |
395 | int PrioEssentials = _config->FindI("pkgProblemResolver::Scores::Essentials",100); |
396 | int PrioInstalledAndNotObsolete = _config->FindI("pkgProblemResolver::Scores::NotObsolete",1); | |
8daf68e3 DK |
397 | int DepMap[] = { |
398 | 0, | |
399 | _config->FindI("pkgProblemResolver::Scores::Depends",1), | |
400 | _config->FindI("pkgProblemResolver::Scores::PreDepends",1), | |
401 | _config->FindI("pkgProblemResolver::Scores::Suggests",0), | |
402 | _config->FindI("pkgProblemResolver::Scores::Recommends",1), | |
403 | _config->FindI("pkgProblemResolver::Scores::Conflicts",-1), | |
404 | _config->FindI("pkgProblemResolver::Scores::Replaces",0), | |
405 | _config->FindI("pkgProblemResolver::Scores::Obsoletes",0), | |
406 | _config->FindI("pkgProblemResolver::Scores::Breaks",-1), | |
407 | _config->FindI("pkgProblemResolver::Scores::Enhances",0) | |
408 | }; | |
d0f2c87c CW |
409 | int AddProtected = _config->FindI("pkgProblemResolver::Scores::AddProtected",10000); |
410 | int AddEssential = _config->FindI("pkgProblemResolver::Scores::AddEssential",5000); | |
8b4894fe MV |
411 | |
412 | if (_config->FindB("Debug::pkgProblemResolver::ShowScores",false) == true) | |
413 | clog << "Settings used to calculate pkgProblemResolver::Scores::" << endl | |
84de0cea MV |
414 | << " Required => " << PrioMap[pkgCache::State::Required] << endl |
415 | << " Important => " << PrioMap[pkgCache::State::Important] << endl | |
416 | << " Standard => " << PrioMap[pkgCache::State::Standard] << endl | |
417 | << " Optional => " << PrioMap[pkgCache::State::Optional] << endl | |
418 | << " Extra => " << PrioMap[pkgCache::State::Extra] << endl | |
8b4894fe MV |
419 | << " Essentials => " << PrioEssentials << endl |
420 | << " InstalledAndNotObsolete => " << PrioInstalledAndNotObsolete << endl | |
8daf68e3 DK |
421 | << " Pre-Depends => " << DepMap[pkgCache::Dep::PreDepends] << endl |
422 | << " Depends => " << DepMap[pkgCache::Dep::Depends] << endl | |
423 | << " Recommends => " << DepMap[pkgCache::Dep::Recommends] << endl | |
424 | << " Suggests => " << DepMap[pkgCache::Dep::Suggests] << endl | |
425 | << " Conflicts => " << DepMap[pkgCache::Dep::Conflicts] << endl | |
426 | << " Breaks => " << DepMap[pkgCache::Dep::DpkgBreaks] << endl | |
427 | << " Replaces => " << DepMap[pkgCache::Dep::Replaces] << endl | |
428 | << " Obsoletes => " << DepMap[pkgCache::Dep::Obsoletes] << endl | |
429 | << " Enhances => " << DepMap[pkgCache::Dep::Enhances] << endl | |
8b4894fe MV |
430 | << " AddProtected => " << AddProtected << endl |
431 | << " AddEssential => " << AddEssential << endl; | |
432 | ||
6c139d6e | 433 | // Generate the base scores for a package based on its properties |
f7f0d6c7 | 434 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
435 | { |
436 | if (Cache[I].InstallVer == 0) | |
437 | continue; | |
438 | ||
d0f2c87c | 439 | int &Score = Scores[I->ID]; |
6c139d6e | 440 | |
7365ff46 | 441 | /* This is arbitrary, it should be high enough to elevate an |
6c139d6e AL |
442 | essantial package above most other packages but low enough |
443 | to allow an obsolete essential packages to be removed by | |
1e3f4083 | 444 | a conflicts on a powerful normal package (ie libc6) */ |
c5200869 JAK |
445 | if ((I->Flags & pkgCache::Flag::Essential) == pkgCache::Flag::Essential |
446 | || (I->Flags & pkgCache::Flag::Important) == pkgCache::Flag::Important) | |
8b4894fe | 447 | Score += PrioEssentials; |
6c139d6e AL |
448 | |
449 | // We transform the priority | |
6c139d6e AL |
450 | if (Cache[I].InstVerIter(Cache)->Priority <= 5) |
451 | Score += PrioMap[Cache[I].InstVerIter(Cache)->Priority]; | |
452 | ||
453 | /* This helps to fix oddball problems with conflicting packages | |
4172c784 MV |
454 | on the same level. We enhance the score of installed packages |
455 | if those are not obsolete | |
456 | */ | |
020daa7b | 457 | if (I->CurrentVer != 0 && Cache[I].CandidateVer != 0 && Cache[I].CandidateVerIter(Cache).Downloadable()) |
8b4894fe | 458 | Score += PrioInstalledAndNotObsolete; |
8daf68e3 | 459 | |
cb4b85b0 | 460 | // propagate score points along dependencies |
f7f0d6c7 | 461 | for (pkgCache::DepIterator D = Cache[I].InstVerIter(Cache).DependsList(); D.end() == false; ++D) |
9ec748ff DK |
462 | { |
463 | if (DepMap[D->Type] == 0) | |
464 | continue; | |
465 | pkgCache::PkgIterator const T = D.TargetPkg(); | |
466 | if (D->Version != 0) | |
467 | { | |
468 | pkgCache::VerIterator const IV = Cache[T].InstVerIter(Cache); | |
469 | if (IV.end() == true || D.IsSatisfied(IV) != D.IsNegative()) | |
470 | continue; | |
471 | } | |
472 | Scores[T->ID] += DepMap[D->Type]; | |
473 | } | |
8daf68e3 DK |
474 | } |
475 | ||
6c139d6e | 476 | // Copy the scores to advoid additive looping |
d0f2c87c | 477 | SPtrArray<int> OldScores = new int[Size]; |
6c139d6e AL |
478 | memcpy(OldScores,Scores,sizeof(*Scores)*Size); |
479 | ||
480 | /* Now we cause 1 level of dependency inheritance, that is we add the | |
481 | score of the packages that depend on the target Package. This | |
482 | fortifies high scoring packages */ | |
f7f0d6c7 | 483 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
484 | { |
485 | if (Cache[I].InstallVer == 0) | |
486 | continue; | |
487 | ||
f7f0d6c7 | 488 | for (pkgCache::DepIterator D = I.RevDependsList(); D.end() == false; ++D) |
6c139d6e AL |
489 | { |
490 | // Only do it for the install version | |
491 | if ((pkgCache::Version *)D.ParentVer() != Cache[D.ParentPkg()].InstallVer || | |
3a998f6a MV |
492 | (D->Type != pkgCache::Dep::Depends && |
493 | D->Type != pkgCache::Dep::PreDepends && | |
494 | D->Type != pkgCache::Dep::Recommends)) | |
6c139d6e AL |
495 | continue; |
496 | ||
dec5b117 MV |
497 | // Do not propagate negative scores otherwise |
498 | // an extra (-2) package might score better than an optional (-1) | |
499 | if (OldScores[D.ParentPkg()->ID] > 0) | |
500 | Scores[I->ID] += OldScores[D.ParentPkg()->ID]; | |
6c139d6e AL |
501 | } |
502 | } | |
503 | ||
1e3f4083 | 504 | /* Now we propagate along provides. This makes the packages that |
6c139d6e | 505 | provide important packages extremely important */ |
f7f0d6c7 | 506 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e | 507 | { |
f7f0d6c7 | 508 | for (pkgCache::PrvIterator P = I.ProvidesList(); P.end() == false; ++P) |
6c139d6e AL |
509 | { |
510 | // Only do it once per package | |
511 | if ((pkgCache::Version *)P.OwnerVer() != Cache[P.OwnerPkg()].InstallVer) | |
512 | continue; | |
513 | Scores[P.OwnerPkg()->ID] += abs(Scores[I->ID] - OldScores[I->ID]); | |
514 | } | |
515 | } | |
516 | ||
517 | /* Protected things are pushed really high up. This number should put them | |
518 | ahead of everything */ | |
f7f0d6c7 | 519 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
d2685fd6 | 520 | { |
6c139d6e | 521 | if ((Flags[I->ID] & Protected) != 0) |
8b4894fe | 522 | Scores[I->ID] += AddProtected; |
c5200869 JAK |
523 | if ((I->Flags & pkgCache::Flag::Essential) == pkgCache::Flag::Essential || |
524 | (I->Flags & pkgCache::Flag::Important) == pkgCache::Flag::Important) | |
8b4894fe MV |
525 | Scores[I->ID] += AddEssential; |
526 | } | |
6c139d6e AL |
527 | } |
528 | /*}}}*/ | |
529 | // ProblemResolver::DoUpgrade - Attempt to upgrade this package /*{{{*/ | |
530 | // --------------------------------------------------------------------- | |
531 | /* This goes through and tries to reinstall packages to make this package | |
532 | installable */ | |
533 | bool pkgProblemResolver::DoUpgrade(pkgCache::PkgIterator Pkg) | |
534 | { | |
74a05226 MV |
535 | pkgDepCache::ActionGroup group(Cache); |
536 | ||
6c139d6e AL |
537 | if ((Flags[Pkg->ID] & Upgradable) == 0 || Cache[Pkg].Upgradable() == false) |
538 | return false; | |
3a486305 AL |
539 | if ((Flags[Pkg->ID] & Protected) == Protected) |
540 | return false; | |
0a8e3465 | 541 | |
6c139d6e AL |
542 | Flags[Pkg->ID] &= ~Upgradable; |
543 | ||
544 | bool WasKept = Cache[Pkg].Keep(); | |
74a05226 | 545 | Cache.MarkInstall(Pkg, false, 0, false); |
6c139d6e | 546 | |
0a8e3465 AL |
547 | // This must be a virtual package or something like that. |
548 | if (Cache[Pkg].InstVerIter(Cache).end() == true) | |
549 | return false; | |
550 | ||
6c139d6e AL |
551 | // Isolate the problem dependency |
552 | bool Fail = false; | |
553 | for (pkgCache::DepIterator D = Cache[Pkg].InstVerIter(Cache).DependsList(); D.end() == false;) | |
554 | { | |
555 | // Compute a single dependency element (glob or) | |
556 | pkgCache::DepIterator Start = D; | |
557 | pkgCache::DepIterator End = D; | |
4b1b89c5 | 558 | for (bool LastOR = true; D.end() == false && LastOR == true;) |
6c139d6e | 559 | { |
b518cca6 | 560 | LastOR = (D->CompareOp & pkgCache::Dep::Or) == pkgCache::Dep::Or; |
0284eee4 | 561 | ++D; |
6c139d6e AL |
562 | if (LastOR == true) |
563 | End = D; | |
564 | } | |
565 | ||
566 | // We only worry about critical deps. | |
567 | if (End.IsCritical() != true) | |
568 | continue; | |
4b1b89c5 AL |
569 | |
570 | // Iterate over all the members in the or group | |
571 | while (1) | |
0a8e3465 | 572 | { |
4b1b89c5 AL |
573 | // Dep is ok now |
574 | if ((Cache[End] & pkgDepCache::DepGInstall) == pkgDepCache::DepGInstall) | |
575 | break; | |
576 | ||
577 | // Do not change protected packages | |
578 | PkgIterator P = Start.SmartTargetPkg(); | |
579 | if ((Flags[P->ID] & Protected) == Protected) | |
580 | { | |
581 | if (Debug == true) | |
47f6d1b7 | 582 | clog << " Reinst Failed because of protected " << P.FullName(false) << endl; |
4b1b89c5 | 583 | Fail = true; |
4b1b89c5 | 584 | } |
648e3cb4 | 585 | else |
6c139d6e | 586 | { |
648e3cb4 AL |
587 | // Upgrade the package if the candidate version will fix the problem. |
588 | if ((Cache[Start] & pkgDepCache::DepCVer) == pkgDepCache::DepCVer) | |
589 | { | |
590 | if (DoUpgrade(P) == false) | |
591 | { | |
592 | if (Debug == true) | |
47f6d1b7 | 593 | clog << " Reinst Failed because of " << P.FullName(false) << endl; |
648e3cb4 AL |
594 | Fail = true; |
595 | } | |
596 | else | |
597 | { | |
598 | Fail = false; | |
599 | break; | |
600 | } | |
601 | } | |
602 | else | |
4b1b89c5 | 603 | { |
648e3cb4 AL |
604 | /* We let the algorithm deal with conflicts on its next iteration, |
605 | it is much smarter than us */ | |
359e46db | 606 | if (Start.IsNegative() == true) |
b2e465d6 | 607 | break; |
648e3cb4 | 608 | |
4b1b89c5 | 609 | if (Debug == true) |
47f6d1b7 | 610 | clog << " Reinst Failed early because of " << Start.TargetPkg().FullName(false) << endl; |
4b1b89c5 | 611 | Fail = true; |
648e3cb4 | 612 | } |
4b1b89c5 | 613 | } |
6c139d6e | 614 | |
4b1b89c5 AL |
615 | if (Start == End) |
616 | break; | |
f7f0d6c7 | 617 | ++Start; |
4b1b89c5 AL |
618 | } |
619 | if (Fail == true) | |
6c139d6e | 620 | break; |
6c139d6e AL |
621 | } |
622 | ||
623 | // Undo our operations - it might be smart to undo everything this did.. | |
624 | if (Fail == true) | |
625 | { | |
626 | if (WasKept == true) | |
74a05226 | 627 | Cache.MarkKeep(Pkg, false, false); |
6c139d6e | 628 | else |
b83cad32 | 629 | Cache.MarkDelete(Pkg, false, 0, false); |
6c139d6e AL |
630 | return false; |
631 | } | |
632 | ||
633 | if (Debug == true) | |
47f6d1b7 | 634 | clog << " Re-Instated " << Pkg.FullName(false) << endl; |
6c139d6e AL |
635 | return true; |
636 | } | |
637 | /*}}}*/ | |
6d38011b DK |
638 | // ProblemResolver::Resolve - calls a resolver to fix the situation /*{{{*/ |
639 | // --------------------------------------------------------------------- | |
640 | /* */ | |
641 | bool pkgProblemResolver::Resolve(bool BrokenFix) | |
642 | { | |
98278a81 | 643 | std::string const solver = _config->Find("APT::Solver", "internal"); |
b57c0e35 DK |
644 | if (solver != "internal") { |
645 | OpTextProgress Prog(*_config); | |
646 | return EDSP::ResolveExternal(solver.c_str(), Cache, false, false, false, &Prog); | |
647 | } | |
6d38011b DK |
648 | return ResolveInternal(BrokenFix); |
649 | } | |
650 | /*}}}*/ | |
651 | // ProblemResolver::ResolveInternal - Run the resolution pass /*{{{*/ | |
6c139d6e AL |
652 | // --------------------------------------------------------------------- |
653 | /* This routines works by calculating a score for each package. The score | |
654 | is derived by considering the package's priority and all reverse | |
655 | dependents giving an integer that reflects the amount of breakage that | |
656 | adjusting the package will inflict. | |
657 | ||
658 | It goes from highest score to lowest and corrects all of the breaks by | |
1e3f4083 | 659 | keeping or removing the dependent packages. If that fails then it removes |
6c139d6e AL |
660 | the package itself and goes on. The routine should be able to intelligently |
661 | go from any broken state to a fixed state. | |
662 | ||
663 | The BrokenFix flag enables a mode where the algorithm tries to | |
664 | upgrade packages to advoid problems. */ | |
6d38011b | 665 | bool pkgProblemResolver::ResolveInternal(bool const BrokenFix) |
6c139d6e | 666 | { |
74a05226 MV |
667 | pkgDepCache::ActionGroup group(Cache); |
668 | ||
6c139d6e AL |
669 | // Record which packages are marked for install |
670 | bool Again = false; | |
671 | do | |
672 | { | |
673 | Again = false; | |
f7f0d6c7 | 674 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
675 | { |
676 | if (Cache[I].Install() == true) | |
677 | Flags[I->ID] |= PreInstalled; | |
678 | else | |
679 | { | |
680 | if (Cache[I].InstBroken() == true && BrokenFix == true) | |
681 | { | |
74a05226 | 682 | Cache.MarkInstall(I, false, 0, false); |
6c139d6e AL |
683 | if (Cache[I].Install() == true) |
684 | Again = true; | |
685 | } | |
686 | ||
687 | Flags[I->ID] &= ~PreInstalled; | |
688 | } | |
689 | Flags[I->ID] |= Upgradable; | |
690 | } | |
691 | } | |
692 | while (Again == true); | |
693 | ||
32b5dd08 | 694 | if (Debug == true) { |
49b49018 MV |
695 | clog << "Starting pkgProblemResolver with broken count: " |
696 | << Cache.BrokenCount() << endl; | |
32b5dd08 | 697 | } |
6c139d6e AL |
698 | |
699 | MakeScores(); | |
6d38011b DK |
700 | |
701 | unsigned long const Size = Cache.Head().PackageCount; | |
702 | ||
6c139d6e AL |
703 | /* We have to order the packages so that the broken fixing pass |
704 | operates from highest score to lowest. This prevents problems when | |
705 | high score packages cause the removal of lower score packages that | |
706 | would cause the removal of even lower score packages. */ | |
b2e465d6 | 707 | SPtrArray<pkgCache::Package *> PList = new pkgCache::Package *[Size]; |
6c139d6e | 708 | pkgCache::Package **PEnd = PList; |
f7f0d6c7 | 709 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
6c139d6e AL |
710 | *PEnd++ = I; |
711 | This = this; | |
712 | qsort(PList,PEnd - PList,sizeof(*PList),&ScoreSort); | |
8b4894fe MV |
713 | |
714 | if (_config->FindB("Debug::pkgProblemResolver::ShowScores",false) == true) | |
715 | { | |
716 | clog << "Show Scores" << endl; | |
717 | for (pkgCache::Package **K = PList; K != PEnd; K++) | |
718 | if (Scores[(*K)->ID] != 0) | |
719 | { | |
720 | pkgCache::PkgIterator Pkg(Cache,*K); | |
721 | clog << Scores[(*K)->ID] << ' ' << Pkg << std::endl; | |
722 | } | |
723 | } | |
6c139d6e | 724 | |
32b5dd08 | 725 | if (Debug == true) { |
49b49018 MV |
726 | clog << "Starting 2 pkgProblemResolver with broken count: " |
727 | << Cache.BrokenCount() << endl; | |
32b5dd08 | 728 | } |
8b4894fe | 729 | |
6c139d6e AL |
730 | /* Now consider all broken packages. For each broken package we either |
731 | remove the package or fix it's problem. We do this once, it should | |
732 | not be possible for a loop to form (that is a < b < c and fixing b by | |
733 | changing a breaks c) */ | |
734 | bool Change = true; | |
09a10f9c | 735 | bool const TryFixByInstall = _config->FindB("pkgProblemResolver::FixByInstall", true); |
6c139d6e AL |
736 | for (int Counter = 0; Counter != 10 && Change == true; Counter++) |
737 | { | |
738 | Change = false; | |
739 | for (pkgCache::Package **K = PList; K != PEnd; K++) | |
740 | { | |
741 | pkgCache::PkgIterator I(Cache,*K); | |
742 | ||
743 | /* We attempt to install this and see if any breaks result, | |
744 | this takes care of some strange cases */ | |
745 | if (Cache[I].CandidateVer != Cache[I].InstallVer && | |
746 | I->CurrentVer != 0 && Cache[I].InstallVer != 0 && | |
747 | (Flags[I->ID] & PreInstalled) != 0 && | |
0a8e3465 AL |
748 | (Flags[I->ID] & Protected) == 0 && |
749 | (Flags[I->ID] & ReInstateTried) == 0) | |
6c139d6e AL |
750 | { |
751 | if (Debug == true) | |
09a10f9c | 752 | clog << " Try to Re-Instate (" << Counter << ") " << I.FullName(false) << endl; |
a6568219 | 753 | unsigned long OldBreaks = Cache.BrokenCount(); |
6c139d6e | 754 | pkgCache::Version *OldVer = Cache[I].InstallVer; |
0a8e3465 AL |
755 | Flags[I->ID] &= ReInstateTried; |
756 | ||
74a05226 | 757 | Cache.MarkInstall(I, false, 0, false); |
6c139d6e AL |
758 | if (Cache[I].InstBroken() == true || |
759 | OldBreaks < Cache.BrokenCount()) | |
760 | { | |
761 | if (OldVer == 0) | |
b83cad32 | 762 | Cache.MarkDelete(I, false, 0, false); |
6c139d6e | 763 | else |
74a05226 | 764 | Cache.MarkKeep(I, false, false); |
6c139d6e AL |
765 | } |
766 | else | |
767 | if (Debug == true) | |
47f6d1b7 | 768 | clog << "Re-Instated " << I.FullName(false) << " (" << OldBreaks << " vs " << Cache.BrokenCount() << ')' << endl; |
6c139d6e AL |
769 | } |
770 | ||
771 | if (Cache[I].InstallVer == 0 || Cache[I].InstBroken() == false) | |
772 | continue; | |
773 | ||
00b47c98 | 774 | if (Debug == true) |
09a10f9c | 775 | clog << "Investigating (" << Counter << ") " << I << endl; |
00b47c98 | 776 | |
6c139d6e AL |
777 | // Isolate the problem dependency |
778 | PackageKill KillList[100]; | |
779 | PackageKill *LEnd = KillList; | |
421c8d10 AL |
780 | bool InOr = false; |
781 | pkgCache::DepIterator Start; | |
782 | pkgCache::DepIterator End; | |
b2e465d6 | 783 | PackageKill *OldEnd = LEnd; |
648e3cb4 AL |
784 | |
785 | enum {OrRemove,OrKeep} OrOp = OrRemove; | |
421c8d10 AL |
786 | for (pkgCache::DepIterator D = Cache[I].InstVerIter(Cache).DependsList(); |
787 | D.end() == false || InOr == true;) | |
6c139d6e AL |
788 | { |
789 | // Compute a single dependency element (glob or) | |
648e3cb4 AL |
790 | if (Start == End) |
791 | { | |
792 | // Decide what to do | |
09a10f9c | 793 | if (InOr == true && OldEnd == LEnd) |
648e3cb4 | 794 | { |
09a10f9c | 795 | if (OrOp == OrRemove) |
70777d4b AL |
796 | { |
797 | if ((Flags[I->ID] & Protected) != Protected) | |
00b47c98 AL |
798 | { |
799 | if (Debug == true) | |
47f6d1b7 | 800 | clog << " Or group remove for " << I.FullName(false) << endl; |
b83cad32 | 801 | Cache.MarkDelete(I, false, 0, false); |
cd14eaf2 | 802 | Change = true; |
09a10f9c DK |
803 | } |
804 | } | |
805 | else if (OrOp == OrKeep) | |
00b47c98 AL |
806 | { |
807 | if (Debug == true) | |
47f6d1b7 | 808 | clog << " Or group keep for " << I.FullName(false) << endl; |
74a05226 | 809 | Cache.MarkKeep(I, false, false); |
cd14eaf2 | 810 | Change = true; |
b2e465d6 | 811 | } |
648e3cb4 AL |
812 | } |
813 | ||
b2e465d6 AL |
814 | /* We do an extra loop (as above) to finalize the or group |
815 | processing */ | |
816 | InOr = false; | |
648e3cb4 | 817 | OrOp = OrRemove; |
421c8d10 | 818 | D.GlobOr(Start,End); |
b2e465d6 AL |
819 | if (Start.end() == true) |
820 | break; | |
cd14eaf2 | 821 | |
b2e465d6 AL |
822 | // We only worry about critical deps. |
823 | if (End.IsCritical() != true) | |
824 | continue; | |
cd14eaf2 | 825 | |
648e3cb4 AL |
826 | InOr = Start != End; |
827 | OldEnd = LEnd; | |
cd14eaf2 | 828 | } |
421c8d10 | 829 | else |
4cc152f9 | 830 | { |
f7f0d6c7 | 831 | ++Start; |
4cc152f9 MV |
832 | // We only worry about critical deps. |
833 | if (Start.IsCritical() != true) | |
834 | continue; | |
835 | } | |
cd14eaf2 | 836 | |
6c139d6e AL |
837 | // Dep is ok |
838 | if ((Cache[End] & pkgDepCache::DepGInstall) == pkgDepCache::DepGInstall) | |
cd14eaf2 AL |
839 | { |
840 | InOr = false; | |
6c139d6e | 841 | continue; |
cd14eaf2 AL |
842 | } |
843 | ||
6c139d6e | 844 | if (Debug == true) |
47f6d1b7 | 845 | clog << "Broken " << Start << endl; |
fcf85120 AL |
846 | |
847 | /* Look across the version list. If there are no possible | |
848 | targets then we keep the package and bail. This is necessary | |
1e3f4083 | 849 | if a package has a dep on another package that can't be found */ |
b2e465d6 | 850 | SPtrArray<pkgCache::Version *> VList = Start.AllTargets(); |
fcf85120 | 851 | if (*VList == 0 && (Flags[I->ID] & Protected) != Protected && |
359e46db | 852 | Start.IsNegative() == false && |
fcf85120 | 853 | Cache[I].NowBroken() == false) |
648e3cb4 AL |
854 | { |
855 | if (InOr == true) | |
856 | { | |
857 | /* No keep choice because the keep being OK could be the | |
858 | result of another element in the OR group! */ | |
859 | continue; | |
860 | } | |
861 | ||
fcf85120 | 862 | Change = true; |
74a05226 | 863 | Cache.MarkKeep(I, false, false); |
fcf85120 AL |
864 | break; |
865 | } | |
866 | ||
6c139d6e AL |
867 | bool Done = false; |
868 | for (pkgCache::Version **V = VList; *V != 0; V++) | |
869 | { | |
870 | pkgCache::VerIterator Ver(Cache,*V); | |
871 | pkgCache::PkgIterator Pkg = Ver.ParentPkg(); | |
a6bfe583 | 872 | |
2ba99db8 MV |
873 | /* This is a conflicts, and the version we are looking |
874 | at is not the currently selected version of the | |
875 | package, which means it is not necessary to | |
876 | remove/keep */ | |
359e46db | 877 | if (Cache[Pkg].InstallVer != Ver && Start.IsNegative() == true) |
4429616b | 878 | { |
2ba99db8 MV |
879 | if (Debug) |
880 | clog << " Conflicts//Breaks against version " | |
881 | << Ver.VerStr() << " for " << Pkg.Name() | |
882 | << " but that is not InstVer, ignoring" | |
24e93662 | 883 | << endl; |
2ba99db8 | 884 | continue; |
4429616b MV |
885 | } |
886 | ||
6c139d6e | 887 | if (Debug == true) |
47f6d1b7 DK |
888 | clog << " Considering " << Pkg.FullName(false) << ' ' << (int)Scores[Pkg->ID] << |
889 | " as a solution to " << I.FullName(false) << ' ' << (int)Scores[I->ID] << endl; | |
a6bfe583 AL |
890 | |
891 | /* Try to fix the package under consideration rather than | |
892 | fiddle with the VList package */ | |
6c139d6e | 893 | if (Scores[I->ID] <= Scores[Pkg->ID] || |
421c8d10 | 894 | ((Cache[Start] & pkgDepCache::DepNow) == 0 && |
359e46db | 895 | End.IsNegative() == false)) |
6c139d6e | 896 | { |
200f8c52 | 897 | // Try a little harder to fix protected packages.. |
3b5421b4 | 898 | if ((Flags[I->ID] & Protected) == Protected) |
200f8c52 AL |
899 | { |
900 | if (DoUpgrade(Pkg) == true) | |
0296c633 | 901 | { |
b2e465d6 AL |
902 | if (Scores[Pkg->ID] > Scores[I->ID]) |
903 | Scores[Pkg->ID] = Scores[I->ID]; | |
0296c633 AL |
904 | break; |
905 | } | |
906 | ||
6c139d6e | 907 | continue; |
200f8c52 AL |
908 | } |
909 | ||
910 | /* See if a keep will do, unless the package is protected, | |
648e3cb4 AL |
911 | then installing it will be necessary */ |
912 | bool Installed = Cache[I].Install(); | |
74a05226 | 913 | Cache.MarkKeep(I, false, false); |
6c139d6e AL |
914 | if (Cache[I].InstBroken() == false) |
915 | { | |
648e3cb4 AL |
916 | // Unwind operation will be keep now |
917 | if (OrOp == OrRemove) | |
918 | OrOp = OrKeep; | |
919 | ||
920 | // Restore | |
921 | if (InOr == true && Installed == true) | |
74a05226 | 922 | Cache.MarkInstall(I, false, 0, false); |
648e3cb4 | 923 | |
6c139d6e | 924 | if (Debug == true) |
47f6d1b7 | 925 | clog << " Holding Back " << I.FullName(false) << " rather than change " << Start.TargetPkg().FullName(false) << endl; |
6c139d6e AL |
926 | } |
927 | else | |
421c8d10 | 928 | { |
6c139d6e AL |
929 | if (BrokenFix == false || DoUpgrade(I) == false) |
930 | { | |
421c8d10 | 931 | // Consider other options |
87da7451 | 932 | if (InOr == false || Cache[I].Garbage == true) |
421c8d10 | 933 | { |
6910a2ac | 934 | if (Debug == true) |
47f6d1b7 | 935 | clog << " Removing " << I.FullName(false) << " rather than change " << Start.TargetPkg().FullName(false) << endl; |
b83cad32 | 936 | Cache.MarkDelete(I, false, 0, false); |
6910a2ac DK |
937 | if (Counter > 1 && Scores[Pkg->ID] > Scores[I->ID]) |
938 | Scores[I->ID] = Scores[Pkg->ID]; | |
d6ebeb21 | 939 | } |
09a10f9c DK |
940 | else if (TryFixByInstall == true && |
941 | Start.TargetPkg()->CurrentVer == 0 && | |
942 | Cache[Start.TargetPkg()].Delete() == false && | |
a3f1a6cc | 943 | (Flags[Start.TargetPkg()->ID] & ToRemove) != ToRemove && |
09a10f9c DK |
944 | Cache.GetCandidateVer(Start.TargetPkg()).end() == false) |
945 | { | |
946 | /* Before removing or keeping the package with the broken dependency | |
947 | try instead to install the first not previously installed package | |
948 | solving this dependency. This helps every time a previous solver | |
949 | is removed by the resolver because of a conflict or alike but it is | |
950 | dangerous as it could trigger new breaks/conflicts… */ | |
443266ef DK |
951 | if (Debug == true) |
952 | clog << " Try Installing " << Start.TargetPkg() << " before changing " << I.FullName(false) << std::endl; | |
09a10f9c DK |
953 | unsigned long const OldBroken = Cache.BrokenCount(); |
954 | Cache.MarkInstall(Start.TargetPkg(), true, 1, false); | |
955 | // FIXME: we should undo the complete MarkInstall process here | |
956 | if (Cache[Start.TargetPkg()].InstBroken() == true || Cache.BrokenCount() > OldBroken) | |
957 | Cache.MarkDelete(Start.TargetPkg(), false, 1, false); | |
958 | } | |
0a8e3465 | 959 | } |
6c139d6e | 960 | } |
b5dc9785 | 961 | |
6c139d6e AL |
962 | Change = true; |
963 | Done = true; | |
964 | break; | |
965 | } | |
966 | else | |
967 | { | |
308c7d30 IJ |
968 | if (Start->Type == pkgCache::Dep::DpkgBreaks) |
969 | { | |
76264cb7 MV |
970 | // first, try upgradring the package, if that |
971 | // does not help, the breaks goes onto the | |
972 | // kill list | |
2ba99db8 | 973 | // |
76264cb7 | 974 | // FIXME: use DoUpgrade(Pkg) instead? |
2ba99db8 | 975 | if (Cache[End] & pkgDepCache::DepGCVer) |
76264cb7 | 976 | { |
308c7d30 | 977 | if (Debug) |
47f6d1b7 | 978 | clog << " Upgrading " << Pkg.FullName(false) << " due to Breaks field in " << I.FullName(false) << endl; |
308c7d30 IJ |
979 | Cache.MarkInstall(Pkg, false, 0, false); |
980 | continue; | |
981 | } | |
308c7d30 IJ |
982 | } |
983 | ||
648e3cb4 | 984 | // Skip adding to the kill list if it is protected |
6c139d6e AL |
985 | if ((Flags[Pkg->ID] & Protected) != 0) |
986 | continue; | |
a6bfe583 AL |
987 | |
988 | if (Debug == true) | |
47f6d1b7 | 989 | clog << " Added " << Pkg.FullName(false) << " to the remove list" << endl; |
6c139d6e AL |
990 | |
991 | LEnd->Pkg = Pkg; | |
992 | LEnd->Dep = End; | |
993 | LEnd++; | |
0a8e3465 | 994 | |
b47053bd | 995 | if (Start.IsNegative() == false) |
6c139d6e AL |
996 | break; |
997 | } | |
998 | } | |
999 | ||
1000 | // Hm, nothing can possibly satisify this dep. Nuke it. | |
359e46db DK |
1001 | if (VList[0] == 0 && |
1002 | Start.IsNegative() == false && | |
648e3cb4 | 1003 | (Flags[I->ID] & Protected) != Protected) |
6c139d6e | 1004 | { |
648e3cb4 | 1005 | bool Installed = Cache[I].Install(); |
6c139d6e AL |
1006 | Cache.MarkKeep(I); |
1007 | if (Cache[I].InstBroken() == false) | |
1008 | { | |
648e3cb4 AL |
1009 | // Unwind operation will be keep now |
1010 | if (OrOp == OrRemove) | |
1011 | OrOp = OrKeep; | |
1012 | ||
1013 | // Restore | |
1014 | if (InOr == true && Installed == true) | |
74a05226 | 1015 | Cache.MarkInstall(I, false, 0, false); |
648e3cb4 | 1016 | |
6c139d6e | 1017 | if (Debug == true) |
47f6d1b7 | 1018 | clog << " Holding Back " << I.FullName(false) << " because I can't find " << Start.TargetPkg().FullName(false) << endl; |
6c139d6e AL |
1019 | } |
1020 | else | |
1021 | { | |
1022 | if (Debug == true) | |
47f6d1b7 | 1023 | clog << " Removing " << I.FullName(false) << " because I can't find " << Start.TargetPkg().FullName(false) << endl; |
648e3cb4 | 1024 | if (InOr == false) |
b83cad32 | 1025 | Cache.MarkDelete(I, false, 0, false); |
6c139d6e AL |
1026 | } |
1027 | ||
1028 | Change = true; | |
1029 | Done = true; | |
1030 | } | |
1031 | ||
421c8d10 AL |
1032 | // Try some more |
1033 | if (InOr == true) | |
1034 | continue; | |
1035 | ||
6c139d6e AL |
1036 | if (Done == true) |
1037 | break; | |
1038 | } | |
1039 | ||
1040 | // Apply the kill list now | |
1041 | if (Cache[I].InstallVer != 0) | |
648e3cb4 | 1042 | { |
6c139d6e | 1043 | for (PackageKill *J = KillList; J != LEnd; J++) |
6c139d6e | 1044 | { |
648e3cb4 AL |
1045 | Change = true; |
1046 | if ((Cache[J->Dep] & pkgDepCache::DepGNow) == 0) | |
1047 | { | |
359e46db | 1048 | if (J->Dep.IsNegative() == true) |
648e3cb4 AL |
1049 | { |
1050 | if (Debug == true) | |
47f6d1b7 | 1051 | clog << " Fixing " << I.FullName(false) << " via remove of " << J->Pkg.FullName(false) << endl; |
b83cad32 | 1052 | Cache.MarkDelete(J->Pkg, false, 0, false); |
648e3cb4 AL |
1053 | } |
1054 | } | |
1055 | else | |
6c139d6e AL |
1056 | { |
1057 | if (Debug == true) | |
47f6d1b7 | 1058 | clog << " Fixing " << I.FullName(false) << " via keep of " << J->Pkg.FullName(false) << endl; |
74a05226 | 1059 | Cache.MarkKeep(J->Pkg, false, false); |
6c139d6e | 1060 | } |
b2e465d6 | 1061 | |
648e3cb4 | 1062 | if (Counter > 1) |
b2e465d6 AL |
1063 | { |
1064 | if (Scores[I->ID] > Scores[J->Pkg->ID]) | |
1065 | Scores[J->Pkg->ID] = Scores[I->ID]; | |
1066 | } | |
648e3cb4 AL |
1067 | } |
1068 | } | |
1069 | } | |
6c139d6e AL |
1070 | } |
1071 | ||
1072 | if (Debug == true) | |
0a8e3465 | 1073 | clog << "Done" << endl; |
b2e465d6 | 1074 | |
6c139d6e | 1075 | if (Cache.BrokenCount() != 0) |
b5dc9785 AL |
1076 | { |
1077 | // See if this is the result of a hold | |
1078 | pkgCache::PkgIterator I = Cache.PkgBegin(); | |
f7f0d6c7 | 1079 | for (;I.end() != true; ++I) |
b5dc9785 AL |
1080 | { |
1081 | if (Cache[I].InstBroken() == false) | |
1082 | continue; | |
1083 | if ((Flags[I->ID] & Protected) != Protected) | |
b2e465d6 | 1084 | return _error->Error(_("Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.")); |
b5dc9785 | 1085 | } |
b2e465d6 | 1086 | return _error->Error(_("Unable to correct problems, you have held broken packages.")); |
b5dc9785 AL |
1087 | } |
1088 | ||
fce72602 | 1089 | // set the auto-flags (mvo: I'm not sure if we _really_ need this) |
80fa0d8a | 1090 | pkgCache::PkgIterator I = Cache.PkgBegin(); |
f7f0d6c7 | 1091 | for (;I.end() != true; ++I) { |
80fa0d8a | 1092 | if (Cache[I].NewInstall() && !(Flags[I->ID] & PreInstalled)) { |
120365ce | 1093 | if(_config->FindI("Debug::pkgAutoRemove",false)) { |
47f6d1b7 | 1094 | std::clog << "Resolve installed new pkg: " << I.FullName(false) |
120365ce MV |
1095 | << " (now marking it as auto)" << std::endl; |
1096 | } | |
80fa0d8a MV |
1097 | Cache[I].Flags |= pkgCache::Flag::Auto; |
1098 | } | |
1099 | } | |
1100 | ||
1101 | ||
0a8e3465 AL |
1102 | return true; |
1103 | } | |
1104 | /*}}}*/ | |
953b348c MV |
1105 | // ProblemResolver::BreaksInstOrPolicy - Check if the given pkg is broken/*{{{*/ |
1106 | // --------------------------------------------------------------------- | |
1107 | /* This checks if the given package is broken either by a hard dependency | |
1108 | (InstBroken()) or by introducing a new policy breakage e.g. new | |
1109 | unsatisfied recommends for a package that was in "policy-good" state | |
1110 | ||
1111 | Note that this is not perfect as it will ignore further breakage | |
1112 | for already broken policy (recommends) | |
1113 | */ | |
1114 | bool pkgProblemResolver::InstOrNewPolicyBroken(pkgCache::PkgIterator I) | |
1115 | { | |
953b348c MV |
1116 | // a broken install is always a problem |
1117 | if (Cache[I].InstBroken() == true) | |
deec6474 DK |
1118 | { |
1119 | if (Debug == true) | |
1120 | std::clog << " Dependencies are not satisfied for " << I << std::endl; | |
953b348c | 1121 | return true; |
deec6474 | 1122 | } |
953b348c MV |
1123 | |
1124 | // a newly broken policy (recommends/suggests) is a problem | |
1125 | if (Cache[I].NowPolicyBroken() == false && | |
1126 | Cache[I].InstPolicyBroken() == true) | |
deec6474 DK |
1127 | { |
1128 | if (Debug == true) | |
1129 | std::clog << " Policy breaks with upgrade of " << I << std::endl; | |
953b348c | 1130 | return true; |
deec6474 DK |
1131 | } |
1132 | ||
953b348c MV |
1133 | return false; |
1134 | } | |
36a171e1 | 1135 | /*}}}*/ |
0a8e3465 AL |
1136 | // ProblemResolver::ResolveByKeep - Resolve problems using keep /*{{{*/ |
1137 | // --------------------------------------------------------------------- | |
1138 | /* This is the work horse of the soft upgrade routine. It is very gental | |
1139 | in that it does not install or remove any packages. It is assumed that the | |
1140 | system was non-broken previously. */ | |
1141 | bool pkgProblemResolver::ResolveByKeep() | |
741b7da9 | 1142 | { |
98278a81 | 1143 | std::string const solver = _config->Find("APT::Solver", "internal"); |
b57c0e35 DK |
1144 | if (solver != "internal") { |
1145 | OpTextProgress Prog(*_config); | |
1146 | return EDSP::ResolveExternal(solver.c_str(), Cache, true, false, false, &Prog); | |
1147 | } | |
741b7da9 DK |
1148 | return ResolveByKeepInternal(); |
1149 | } | |
1150 | /*}}}*/ | |
1151 | // ProblemResolver::ResolveByKeepInternal - Resolve problems using keep /*{{{*/ | |
1152 | // --------------------------------------------------------------------- | |
1153 | /* This is the work horse of the soft upgrade routine. It is very gental | |
1154 | in that it does not install or remove any packages. It is assumed that the | |
1155 | system was non-broken previously. */ | |
1156 | bool pkgProblemResolver::ResolveByKeepInternal() | |
0a8e3465 | 1157 | { |
74a05226 MV |
1158 | pkgDepCache::ActionGroup group(Cache); |
1159 | ||
b2e465d6 | 1160 | unsigned long Size = Cache.Head().PackageCount; |
0a8e3465 | 1161 | |
0a8e3465 AL |
1162 | MakeScores(); |
1163 | ||
1164 | /* We have to order the packages so that the broken fixing pass | |
1165 | operates from highest score to lowest. This prevents problems when | |
1166 | high score packages cause the removal of lower score packages that | |
1167 | would cause the removal of even lower score packages. */ | |
1168 | pkgCache::Package **PList = new pkgCache::Package *[Size]; | |
1169 | pkgCache::Package **PEnd = PList; | |
f7f0d6c7 | 1170 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
0a8e3465 AL |
1171 | *PEnd++ = I; |
1172 | This = this; | |
1173 | qsort(PList,PEnd - PList,sizeof(*PList),&ScoreSort); | |
8b4894fe MV |
1174 | |
1175 | if (_config->FindB("Debug::pkgProblemResolver::ShowScores",false) == true) | |
1176 | { | |
1177 | clog << "Show Scores" << endl; | |
1178 | for (pkgCache::Package **K = PList; K != PEnd; K++) | |
1179 | if (Scores[(*K)->ID] != 0) | |
1180 | { | |
1181 | pkgCache::PkgIterator Pkg(Cache,*K); | |
1182 | clog << Scores[(*K)->ID] << ' ' << Pkg << std::endl; | |
1183 | } | |
1184 | } | |
1185 | ||
1186 | if (Debug == true) | |
1187 | clog << "Entering ResolveByKeep" << endl; | |
1188 | ||
0a8e3465 AL |
1189 | // Consider each broken package |
1190 | pkgCache::Package **LastStop = 0; | |
1191 | for (pkgCache::Package **K = PList; K != PEnd; K++) | |
1192 | { | |
1193 | pkgCache::PkgIterator I(Cache,*K); | |
1194 | ||
953b348c | 1195 | if (Cache[I].InstallVer == 0) |
0a8e3465 AL |
1196 | continue; |
1197 | ||
953b348c MV |
1198 | if (InstOrNewPolicyBroken(I) == false) |
1199 | continue; | |
1200 | ||
0a8e3465 | 1201 | /* Keep the package. If this works then great, otherwise we have |
1e3f4083 | 1202 | to be significantly more aggressive and manipulate its dependencies */ |
0a8e3465 AL |
1203 | if ((Flags[I->ID] & Protected) == 0) |
1204 | { | |
1205 | if (Debug == true) | |
47f6d1b7 | 1206 | clog << "Keeping package " << I.FullName(false) << endl; |
74a05226 | 1207 | Cache.MarkKeep(I, false, false); |
953b348c | 1208 | if (InstOrNewPolicyBroken(I) == false) |
0a8e3465 | 1209 | { |
b2e465d6 | 1210 | K = PList - 1; |
0a8e3465 AL |
1211 | continue; |
1212 | } | |
1213 | } | |
1214 | ||
1215 | // Isolate the problem dependencies | |
1216 | for (pkgCache::DepIterator D = Cache[I].InstVerIter(Cache).DependsList(); D.end() == false;) | |
1217 | { | |
c5532863 AL |
1218 | DepIterator Start; |
1219 | DepIterator End; | |
1220 | D.GlobOr(Start,End); | |
1221 | ||
0a8e3465 AL |
1222 | // We only worry about critical deps. |
1223 | if (End.IsCritical() != true) | |
1224 | continue; | |
1225 | ||
1226 | // Dep is ok | |
1227 | if ((Cache[End] & pkgDepCache::DepGInstall) == pkgDepCache::DepGInstall) | |
1228 | continue; | |
c5532863 AL |
1229 | |
1230 | /* Hm, the group is broken.. I suppose the best thing to do is to | |
1231 | is to try every combination of keep/not-keep for the set, but thats | |
1232 | slow, and this never happens, just be conservative and assume the | |
1233 | list of ors is in preference and keep till it starts to work. */ | |
1234 | while (true) | |
0a8e3465 | 1235 | { |
c5532863 | 1236 | if (Debug == true) |
47f6d1b7 | 1237 | clog << "Package " << I.FullName(false) << " " << Start << endl; |
8b4894fe | 1238 | |
c5532863 AL |
1239 | // Look at all the possible provides on this package |
1240 | SPtrArray<pkgCache::Version *> VList = Start.AllTargets(); | |
1241 | for (pkgCache::Version **V = VList; *V != 0; V++) | |
0a8e3465 | 1242 | { |
c5532863 AL |
1243 | pkgCache::VerIterator Ver(Cache,*V); |
1244 | pkgCache::PkgIterator Pkg = Ver.ParentPkg(); | |
1245 | ||
1246 | // It is not keepable | |
1247 | if (Cache[Pkg].InstallVer == 0 || | |
1248 | Pkg->CurrentVer == 0) | |
1249 | continue; | |
1250 | ||
1251 | if ((Flags[I->ID] & Protected) == 0) | |
1252 | { | |
1253 | if (Debug == true) | |
47f6d1b7 | 1254 | clog << " Keeping Package " << Pkg.FullName(false) << " due to " << Start.DepType() << endl; |
74a05226 | 1255 | Cache.MarkKeep(Pkg, false, false); |
c5532863 AL |
1256 | } |
1257 | ||
953b348c | 1258 | if (InstOrNewPolicyBroken(I) == false) |
c5532863 | 1259 | break; |
0a8e3465 AL |
1260 | } |
1261 | ||
953b348c | 1262 | if (InstOrNewPolicyBroken(I) == false) |
0a8e3465 | 1263 | break; |
0a8e3465 | 1264 | |
c5532863 AL |
1265 | if (Start == End) |
1266 | break; | |
f7f0d6c7 | 1267 | ++Start; |
c5532863 AL |
1268 | } |
1269 | ||
953b348c | 1270 | if (InstOrNewPolicyBroken(I) == false) |
0a8e3465 AL |
1271 | break; |
1272 | } | |
1273 | ||
953b348c | 1274 | if (InstOrNewPolicyBroken(I) == true) |
0a8e3465 AL |
1275 | continue; |
1276 | ||
1277 | // Restart again. | |
0291f645 JT |
1278 | if (K == LastStop) { |
1279 | // I is an iterator based off our temporary package list, | |
1280 | // so copy the name we need before deleting the temporary list | |
1281 | std::string const LoopingPackage = I.FullName(false); | |
1282 | delete[] PList; | |
1283 | return _error->Error("Internal Error, pkgProblemResolver::ResolveByKeep is looping on package %s.", LoopingPackage.c_str()); | |
1284 | } | |
0a8e3465 | 1285 | LastStop = K; |
b2e465d6 | 1286 | K = PList - 1; |
0291f645 | 1287 | } |
6c139d6e | 1288 | |
0291f645 | 1289 | delete[] PList; |
6c139d6e AL |
1290 | return true; |
1291 | } | |
1292 | /*}}}*/ | |
f3c736f9 | 1293 | // ProblemResolver::InstallProtect - deprecated cpu-eating no-op /*{{{*/ |
3b5421b4 | 1294 | // --------------------------------------------------------------------- |
f3c736f9 DK |
1295 | /* Actions issued with FromUser bit set are protected from further |
1296 | modification (expect by other calls with FromUser set) nowadays , so we | |
1297 | don't need to reissue actions here, they are already set in stone. */ | |
3b5421b4 AL |
1298 | void pkgProblemResolver::InstallProtect() |
1299 | { | |
74a05226 MV |
1300 | pkgDepCache::ActionGroup group(Cache); |
1301 | ||
f7f0d6c7 | 1302 | for (pkgCache::PkgIterator I = Cache.PkgBegin(); I.end() == false; ++I) |
3b5421b4 AL |
1303 | { |
1304 | if ((Flags[I->ID] & Protected) == Protected) | |
1305 | { | |
1306 | if ((Flags[I->ID] & ToRemove) == ToRemove) | |
1307 | Cache.MarkDelete(I); | |
c15f5690 MV |
1308 | else |
1309 | { | |
da543ed8 OS |
1310 | // preserve the information whether the package was auto |
1311 | // or manually installed | |
c15f5690 MV |
1312 | bool autoInst = (Cache[I].Flags & pkgCache::Flag::Auto); |
1313 | Cache.MarkInstall(I, false, 0, !autoInst); | |
1314 | } | |
3b5421b4 AL |
1315 | } |
1316 | } | |
1317 | } | |
1318 | /*}}}*/ | |
b2e465d6 AL |
1319 | // PrioSortList - Sort a list of versions by priority /*{{{*/ |
1320 | // --------------------------------------------------------------------- | |
1321 | /* This is ment to be used in conjunction with AllTargets to get a list | |
1322 | of versions ordered by preference. */ | |
1323 | static pkgCache *PrioCache; | |
1324 | static int PrioComp(const void *A,const void *B) | |
1325 | { | |
1326 | pkgCache::VerIterator L(*PrioCache,*(pkgCache::Version **)A); | |
1327 | pkgCache::VerIterator R(*PrioCache,*(pkgCache::Version **)B); | |
1328 | ||
1329 | if ((L.ParentPkg()->Flags & pkgCache::Flag::Essential) == pkgCache::Flag::Essential && | |
b8c0f9b7 AL |
1330 | (R.ParentPkg()->Flags & pkgCache::Flag::Essential) != pkgCache::Flag::Essential) |
1331 | return 1; | |
b2e465d6 | 1332 | if ((L.ParentPkg()->Flags & pkgCache::Flag::Essential) != pkgCache::Flag::Essential && |
b8c0f9b7 AL |
1333 | (R.ParentPkg()->Flags & pkgCache::Flag::Essential) == pkgCache::Flag::Essential) |
1334 | return -1; | |
c5200869 JAK |
1335 | |
1336 | if ((L.ParentPkg()->Flags & pkgCache::Flag::Important) == pkgCache::Flag::Important && | |
1337 | (R.ParentPkg()->Flags & pkgCache::Flag::Important) != pkgCache::Flag::Important) | |
1338 | return 1; | |
1339 | if ((L.ParentPkg()->Flags & pkgCache::Flag::Important) != pkgCache::Flag::Important && | |
1340 | (R.ParentPkg()->Flags & pkgCache::Flag::Important) == pkgCache::Flag::Important) | |
1341 | return -1; | |
b2e465d6 AL |
1342 | |
1343 | if (L->Priority != R->Priority) | |
b8c0f9b7 | 1344 | return R->Priority - L->Priority; |
b2e465d6 AL |
1345 | return strcmp(L.ParentPkg().Name(),R.ParentPkg().Name()); |
1346 | } | |
1347 | void pkgPrioSortList(pkgCache &Cache,pkgCache::Version **List) | |
1348 | { | |
1349 | unsigned long Count = 0; | |
1350 | PrioCache = &Cache; | |
1351 | for (pkgCache::Version **I = List; *I != 0; I++) | |
1352 | Count++; | |
1353 | qsort(List,Count,sizeof(*List),PrioComp); | |
1354 | } | |
1355 | /*}}}*/ |