From: Simon J. <si...@jo...> - 2009-06-02 12:11:04
|
Hi! Thanks for releasing a new version of statcvs, the output looks much nicer! I'm using it on http://statcvs.josefsson.org/ for some projects. Two minor nits: 1) There is a typo for the -disable-twitter-button parameter, see attached patch. 2) The viewcvs urls doesn't seem to work. You can see this on your own pages, see: http://statcvs.sourceforge.net/statcvs/2009-05.html and try clicking on the filenames. The first URL is: http://statcvs.sourceforge.net/statcvs/http%3A%2F%2Fstatcvs.cvs.sourceforge.net%2Fstatcvs%2Fstatcvs%2Fsrc%2Fnet%2Fsf%2Fstatcvs%2Fpages%2FDirectoryPageMaker.java%3Fr1%3D1.9%26amp%3Br2%3D1.10 This used to work in 0.4.0, compare: http://statcvs.josefsson.org/libssh2/2009-05.html and the links on it. Btw, when fixing this, it would be nice to support ? in the URLs, I'm invoking statcvs like this: -viewcvs http://cvs.savannah.gnu.org/viewvc/inetutils/?root=inetutils Thanks, /Simon Index: src/net/sf/statcvs/Main.java =================================================================== RCS file: /cvsroot/statcvs/statcvs/src/net/sf/statcvs/Main.java,v retrieving revision 1.66 diff -u -p -r1.66 Main.java --- src/net/sf/statcvs/Main.java 27 May 2009 18:19:14 -0000 1.66 +++ src/net/sf/statcvs/Main.java 2 Jun 2009 12:06:58 -0000 @@ -110,7 +110,7 @@ public class Main { + " -trac <url> integrate with Trac at <url>\n" + " -xml generate XML instead of HTML\n" + " -charset <charset> specify the charset to use for html/xdoc\n" + " -verbose print extra progress information\n" + " -viewcvs/viewvc/cvsweb/chora/jcvsweb/bugzilla/mantis <url>\n" + " add links to installation at <url>\n" - + " -disable-twitter-buttons\n" + "\n" + "Full options list: http://statcvs.sf.net/manual"); + + " -disable-twitter-button\n" + "\n" + "Full options list: http://statcvs.sf.net/manual"); System.exit(1); } |