[Yum-devel] [PATCH][RFC] Skip-broken support again

[Tim Lauridsen]

Florian Festi wrote:
> Shouldn't
> 
> depTree = self._buildDepTree()
> 
> be done in the while loop as the tsInfo changes with each loop?
> 

No need to build the depTree again, it is just used to find the relation
between packages original in the transaction and which packages they 
drag in as dependcies.
So if foo is needed to be skipped then foobar & foogui dragged in as 
deps to foo is skipped too.
If we recalc the deptree in each loop, then we lose some relations, 
because some packages is removed, and we can get looping in special cases.

Tim