2008/7/14 Karl Hasselström <kha@xxxxxxxxxxx>:
> On 2008-07-13 12:40:48 +0100, Catalin Marinas wrote:
>
>> @@ -67,8 +64,8 @@ def func(parser, options, args):
>> rev = strip_suffix('/', rev)
>> if rev.endswith('/'):
>> rev = strip_suffix('/', rev)
>> - rev1 = rev + '//bottom'
>> - rev2 = rev + '//top'
>> + rev1 = rev + 'HEAD^'
>> + rev2 = rev + 'HEAD'
>
> This looks bogus. You probably want to s/HEAD// here.
Actually, the full "if" block here is useless. This was to specify
diff -r patch/ to show a patch but we now have the "show" command.
> I guess the test suite doesn't cover this? :-(
And it shouldn't, I removed it :-)
--
Catalin
--
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
