Great, thanks for checking. I'll have a look at the updated merge proposal some time later this week.

milki <notifications@github.com> wrote:
>> @@ -315,14 +316,55 @@ def _split_proto_line(line, allowed):
>>      try:
>>          if len(fields) == 1 and command in ('done', None):
>>              return (command, None)
>> -        elif len(fields) == 2 and command in ('want', 'have'):
>> -            hex_to_sha(fields[1])
>> -            return tuple(fields)
>> -    except (TypeError, AssertionError), e:
>> +        elif len(fields) == 2:
>> +            if command in ('want', 'have', 'shallow', 'unshallow'):
>> +                hex_to_sha(fields[1])
>> +                return tuple(fields)
>> +            elif command == 'deepen':
>> +                return command, int(fields[1])
>> +    except (TypeError, AssertionError, ValueError), e:
>
>You are right. There's nothing new that would need to be caught.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/jelmer/dulwich/pull/137/files#r9025130