sajingeo / rietveld

Automatically exported from code.google.com/p/rietveld
Apache License 2.0
0 stars 0 forks source link

support CVS #117

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
do you have any plan to support CVS.

Original issue reported on code.google.com by leei...@gmail.com on 18 May 2009 at 3:25

GoogleCodeExporter commented 9 years ago
No.

Original comment by techtonik@gmail.com on 27 Aug 2010 at 9:56

GoogleCodeExporter commented 9 years ago
thanks

Original comment by leei...@gmail.com on 27 Aug 2010 at 11:05

GoogleCodeExporter commented 9 years ago
huh? Everyone is invited to write a backend for CVS! There's no "no" here IMHO!

Original comment by albrecht.andi on 27 Aug 2010 at 12:34

GoogleCodeExporter commented 9 years ago
Of course nobody will object against CVS support if somebody sends a patch, but 
I do not have any plans to work on this. I guess the same is true for other 
active contributors here.

Original comment by techtonik@gmail.com on 27 Aug 2010 at 12:38

GoogleCodeExporter commented 9 years ago
This's a patch for CVS feature, hope you can merge it.
thanks a lot

Original comment by leei...@gmail.com on 29 Aug 2010 at 2:16

Attachments:

GoogleCodeExporter commented 9 years ago
Could you upload your patch to codereview.appspot.com?

Original comment by albrecht.andi on 30 Aug 2010 at 9:20

GoogleCodeExporter commented 9 years ago
done. url is http://codereview.appspot.com/2058043

Original comment by leei...@gmail.com on 30 Aug 2010 at 3:46

GoogleCodeExporter commented 9 years ago
Fixed in r680.

Original comment by techtonik@gmail.com on 8 Apr 2011 at 9:21