Nicely done.
Acked-by: Karl Hasselström <kha@xxxxxxxxxxx>
On 2009-03-12 12:09:13 +0000, Catalin Marinas wrote:
> + applied = applied[:insert_idx] + patches + applied[insert_idx:]
> +
> + unapplied = [p for p in stack.patchorder.unapplied if p not in patches]
> + hidden = list(stack.patchorder.hidden)
> +
> + iw = stack.repository.default_iw
> + clean_iw = not options.keep and iw or None
> + trans = transaction.StackTransaction(stack, 'sink',
> + check_clean_iw = clean_iw)
> +
> + try:
> + trans.reorder_patches(applied, unapplied, hidden, iw)
Hmm. We should maybe have a default value for hidden: the current list
of patches. Not changing the hidden patches is a common operation.
--
Karl Hasselström, kha@xxxxxxxxxxx
www.treskal.com/kalle
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
