Code review comment for lp:~maxb/bzr-cvsps-import/fix-test

Revision history for this message
Vincent Ladeuil (vila) wrote :

<vila> maxb: this fix looks and feels correct, yet... I should miss something. Why did it fail ?
<jelmer> it also looks good here, but my CVS foo is weak.
<maxb> Because of paths containing /./ which were not expected
<vila> Haaaa

Please mention that in a comment, otherwise: good to land !

review: Approve

« Back to merge proposal