[Cvs-syncmail] [ cvs-syncmail-Patches-705853 ] New features: Prevent diffs, generate CVSWeb URL references

[SourceForge.net <no...@so...>]

Patches item #705853, was opened at 2003-03-18 21:39
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450021&aid=705853&group_id=47611

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Michael Hohmuth (rotofoil)
Assigned to: Nobody/Anonymous (nobody)
Summary: New features: Prevent diffs, generate CVSWeb URL references

Initial Comment:
(Crossposted between mailing list and the patch tracker
on the SF project website)

This patch adds two new features:

1. Options -n / --nodiffs prevents diff output even if
the loginfo format string is %{sVv}.

2. Options -l / --links generate URL references to a
CVSWeb service (such as Sourceforge's).  Options -w /
--cvsweb and -r / --repo control which CVSWeb service
is to be referenced.

The patch is relative to CVS version 1.29

----------------------------------------------------------------------

>Comment By: Michael Hohmuth (rotofoil)
Date: 2003-03-19 13:03

Message:
Logged In: YES 
user_id=535753

In retrospect, it was a bad idea to use "<>" braces around
the URLs syncmail generates with this patch: SF's
mailing-list archiving software doesn't grok this syntax and
includes the closing ">" in the URL is recovers from emails.
 I therefore suggest to remove the braces:

Index: syncmail
===================================================================
RCS file: /cvsroot/opencxx/CVSROOT/syncmail,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -p -r1.7 -r1.8
--- syncmail    18 Mar 2003 20:37:45 -0000      1.7
+++ syncmail    19 Mar 2003 11:45:15 -0000      1.8
@@ -144,9 +144,9 @@ MAILPORT = 25
 CVSWEB_HOST = "http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi"
 CVSWEB_REPO = "opencxx"
 
-CVSWEB_NEW = "%s (New):
<%s/%s/%s?rev=%s&content-type=text/vnd.viewcvs-markup>"
-CVSWEB_DEL = "%s (Deleted):
<%s/%s/%s?rev=%s&content-type=text/vnd.viewcvs-markup>"
-CVSWEB_CHG = "%s: <%s/%s/%s.diff?r1=%s&r2=%s>"
+CVSWEB_NEW = "%s (New):
%s/%s/%s?rev=%s&content-type=text/vnd.viewcvs-markup"
+CVSWEB_DEL = "%s (Deleted):
%s/%s/%s?rev=%s&content-type=text/vnd.viewcvs-markup"
+CVSWEB_CHG = "%s: %s/%s/%s.diff?r1=%s&r2=%s"
 
 # Diff trimming stuff
 DIFF_HEAD_LINES = 20


----------------------------------------------------------------------

Comment By: Michael Hohmuth (rotofoil)
Date: 2003-03-18 21:40

Message:
Logged In: YES 
user_id=535753

(Too bad the file upload didn't work.  I'll post the patch
to the mailing list.)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450021&aid=705853&group_id=47611