[Libvir] PATCH: 0/16: Storage management APIs
Jim Meyering
jim at meyering.net
Thu Feb 14 12:27:18 UTC 2008
Daniel Veillard <veillard at redhat.com> wrote:
...
> In general I like the approach 'let's put what we consider fine first
> and discuss other parts later' . I guess it's the third round of review
> and I really find easier to comment based on existing stuff than big patches
> that gets iterated over time. So let's isolate what we consider okay
> and not needing further discussion, push them now, than in an upcoming
> fourth iteration,
Yes, monolithic patches are hard to manage, and it's not
effective to re-review from scratch a 10,000-line diff when
the incremental diff is say just 1/10 the size.
For that reason, I've been pulling from Dan's hg patch queue
on an irregular basis, and importing the result into a
git repository, putting each "pulled batch" on a new branch.
Then, I can compare one batch to another with a simple
git diff t7..t8
If only I had pulled a little more frequently over the
last week or two. My final incremental is summarized like this:
$ git diff --shortstat t8..t9-public
74 files changed, 2156 insertions(+), 9861 deletions(-)
More information about the libvir-list
mailing list