On 19/11/2010 13:13, Corinna Vinschen wrote: > The code in question was supposed to make sure that the order is always > "base-cygwin base-passwd [...]" and that was the case so far. Of > course, given the obvious mishandling of the casecompare return value > it's not clear why this ever worked. Even more mysterious is the fact > that the bugfix *breaks* this order. Well, time to debug...
Well, the bugfix breaks the order because visit() on "base-cygwin" inserts it and all it's dependencies depth-first (ignoring loops). I guess what's needed is a separate function to insert something explicitly into the ordered list and mark that as visited, and use that for "base-cygwin" and "base-passwd".
