On Wed, May 07, 2014 at 10:59:56PM +0200, Per Cederqvist wrote:
> On Tue, May 6, 2014 at 9:40 PM, Jeff Sipek <[email protected]> wrote:
> > On Fri, Mar 21, 2014 at 08:31:45AM +0100, Per Cederqvist wrote:
> >> Test that we can combine any combination of patches with empty and
> >> non-empty messages, both with and without guilt.diffstat. (All
> >> patches are empty.)
> >
> > I feel like we should have *some* content there - most of the time, I care
> > more about the diffs getting folded than the commit message :)
>
> I added these tests for a reason: the reproduce a bug in guilt that I found.
>
> I'm afraid that having some content might hide the bug I found. (I'm also
> equally afraid that it might uncover other bugs in guilt, which would delay
> integration of this patch series. So adding more test cases with content
> is a good thing to do, but maybe not in this patch series.)
Fair enough. I use guilt-fold all the time and it hasn't lost any of my
diffs, so I'm happy to defer this until some point in the future.
...
> > for using_diffstat in true false ; do
> > for patcha in empty nonempty ; do
> > for patchb in empty nonempty ; do
> > echo "%% $patcha + $patchb
> > (diffstat=$using_diffstat)"
> > ${patcha}_patch $patcha
> > ${patchb}_patch $patchb
> > cmd guilt pop $patchb
> > cmd guilt fold $patchb
> > fixup_time_info $patcha
> > cmd list_files
> > cleanup $patcha
> > cmd list_files
> > done
> > done
> > done
> >
> > Aha! That's better, IMO.
>
> I'll try that and post a version 2 of the series. It might take a few
> days, though.
No problem. I'm still the slower one of the two of us. :/
Jeff.
--
Keyboard not found!
Press F1 to enter Setup
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
