Yedidyah Bar David has posted comments on this change. Change subject: core: rewrite buildSequence using toposort ......................................................................
Patch Set 3: I think I already explained before the intention of this patch. Following your request I started here a discussion about that, which didn't advance much. If you feel some other medium is better for that, please propose. I am not currently pushing you (or anyone) to merge it. Just maintain it so that it's usable. If you do want it merged, I'd be happy to attend to all of your comments. Currently we (Sandro, Simone and me) are using it as a backup for the existing implementation when the latter is hard to use, mainly for solving loops, and also as some kind of "extra verification". If you do not want to merge it for now, feel free to mark -1 so that you do not merge it by accident. I don't mind doing that myself if you want me (when I push updates). -- To view, visit http://gerrit.ovirt.org/28775 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9a7e9a26ac68543331dc869ab151883ac8bd5b30 Gerrit-PatchSet: 3 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Lev Veyde <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Simone Tiraboschi <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
