Re: [Monotone-devel] Fw: [bug #21706] automate packet IO broken on 0.37

[Zack Weinberg]

On Dec 3, 2007 12:42 PM, Zbigniew Zagórski <address@hidden> wrote:
> >     mtn: error: malformed packet: too many arguments in header
>
> Failing packet is:
> [rcert 57fb649ce6809a7f806dd481554f7edbcc95f141
>        changelog
>        address@hidden
>        
> KiBpbml0aWFsIGNvbW1pdDogYnJhbmNoIGFuZCBsYWJlbHMgbGlzdGVyCiogc2ltcGxlIHRl
> c3QgYWRkZWQK]
> RSc+EaqFfb/FQjDSw3KPOxqWEjc3Bi87ylTAbwVxjFKF570Cwyu/FU6JrVavw/w9Wm+Ys17q
> AThy6Z/RyLc+HFolzGg+OtppE7U3WFBnrMmn+zwSiJGMfovKM+4JbA/LsHIpSOJkbhdIpfZf
> zzBmUC+Bgb63kPIiXS+RWgCCN/c=
> [end]
>
> anyone knows if this kind of packet is correct (0.36 reads it)?
>
> I've got it. I've "unbroken" last parameter (the base6t(cert_value))
> to fit one line and mtn accepts it :)

If 0.36 read this packet, then subsequent versions should too.  This
looks like an easy fix.

The packet reading code doesn't really have comprehensive tests; would
you be interested in writing some?

zw