Thread (28 messages) flat view 28 messages, 3 authors, 2016-06-15

Re: [StGit PATCH 00/13] Eliminate 'top' and 'bottom' files

From: Karl Hasselström <hidden>
Date: 2016-06-15 22:43:35

On 2007-09-16 08:28:53 +0100, Catalin Marinas wrote:
We should get rid of top.old and bottom.old as well.
Yeah, I guess that data could be computed from the patch log?
My question - does this conflict with the DAG patches in any way? I
intend to include the them at some point, once I get a chance to
test the performance penalty with a big tree like the Linux kernel.
I haven't been able to get rid of all the expensive DAG walking, so
I've been considering a different approach: continue using the current
applied and unapplied files if the existing HEAD == top patch check
passes, and letting the assimilate command do a full DAG walk to
regenerate those files. (And by "full DAG walk", I mean walking from
HEAD down to the first commit with parents != 1; the patches we see
are applied (in the order we see them), and the rest are unapplied.)
Is there any patch which consists of more than one commit? Maybe
only uncommit could generate one but I think we put some tests in
place.
Uncommit does not generate such patches, unless I've made a thinko; I
don't approve of them. But I always assumed they could exist, since
the code (at least in places) seems careful to not assume anything
about the number of commits between top and bottom.

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help