cvs-syncmail-talk — User & developer discussions for the "syncmail" CVS checkin notification tool

RE: [Cvs-syncmail] Broke cvssyncmail [WAS: RE: CVS: xdoclet2/lib/rngconv/jars rngconv-20030225.jar,NONE,1.1]

[<asl...@ne...>]

Done.

http://sourceforge.net/tracker/index.php?func=detail&aid=717082&group_id=476
11&atid=450019

Thanks for a great product btw.

> -----Original Message-----
> From: Fred L. Drake, Jr. [mailto:fd...@ac...]
> Sent: 7. april 2003 19:18
> To: Aslak Hellesoy
> Cc: Marcus Brito; cvs...@li...
> Subject: Re: [Cvs-syncmail] Broke cvssyncmail [WAS: RE: CVS:
> xdoclet2/lib/rngconv/jars rngconv-20030225.jar,NONE,1.1]
>
>
>
> Aslak Hellesoy writes:
>  > cvs-syncmail team: I just added a jar file (it's really a zip
> file in case
>  > you don't know java). syncmail.py apparently thinks it's an
> ascii file. Is
>  > this a known bug? (Couldn't find a trace for it in your bug tracker).
>
> Please file a bug report for this.
>
> syncmail currently uses a heuristic approach to determine whether a
> file is binary as an optimization.  I expect this won't be hard to
> fix, but it means every check-in email that gets generated will be
> just a bit more resource-intensive on the CVS server.
>
>
>   -Fred
>
> --
> Fred L. Drake, Jr.  <fdrake at acm.org>
> PythonLabs at Zope Corporation

[Cvs-syncmail] [ cvs-syncmail-Bugs-717082 ] jar files incorrectly treated as ascii

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

Bugs item #717082, was opened at 2003-04-08 00:57
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=717082&group_id=47611

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Aslak Hellesøy (rinkrank)
Assigned to: Nobody/Anonymous (nobody)
Summary: jar files incorrectly treated as ascii

Initial Comment:
When adding/committing a new jar file to cvs, cvs-
syncmail incorrectly sends a message like this:
------------
Update of /cvsroot/xdoclet/xdoclet2/lib/rngconv/jars
In directory sc8-pr-
cvs1:/tmp/cvsserv14703/lib/rngconv/jars

Added Files:
	rngconv-20030225.jar 
Log Message:
Initial support for XML Schema in XGG (using Sun's 
rngconv)

--- NEW FILE: rngconv-20030225.jar ---
PK
------------
This leads people to think the file was incorrectly added 
as ascii, while it's actually syncmail that incorrectly 
assumes it's ascii.

(jar files are in zip format).


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

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

Re: [Cvs-syncmail] Broke cvssyncmail [WAS: RE: CVS: xdoclet2/lib/rngconv/jars rngconv-20030225.jar,NONE,1.1]

[Fred L. D. Jr. <fd...@ac...>]

Aslak Hellesoy writes:
 > cvs-syncmail team: I just added a jar file (it's really a zip file in case
 > you don't know java). syncmail.py apparently thinks it's an ascii file. Is
 > this a known bug? (Couldn't find a trace for it in your bug tracker).

Please file a bug report for this.

syncmail currently uses a heuristic approach to determine whether a
file is binary as an optimization.  I expect this won't be hard to
fix, but it means every check-in email that gets generated will be
just a bit more resource-intensive on the CVS server.


  -Fred

-- 
Fred L. Drake, Jr.  <fdrake at acm.org>
PythonLabs at Zope Corporation

[Cvs-syncmail] Broke cvssyncmail [WAS: RE: CVS: xdoclet2/lib/rngconv/jars rngconv-20030225.jar,NONE,1.1]

[<asl...@ne...>]

There's nothing wrong with the jar file (opens ok in winzip, and java can
read it). My CVS client ain't broke. I just deleted the file locally and
updated, and my WinCVS says it's binary.

I'm cc-ing the cvs-syncmail team, as I believe this is due to a bug in the
syncmail.py script (the script that notifies our list).

cvs-syncmail team: I just added a jar file (it's really a zip file in case
you don't know java). syncmail.py apparently thinks it's an ascii file. Is
this a known bug? (Couldn't find a trace for it in your bug tracker).

Cheers,
Aslak

> -----Original Message-----
> From: Marcus Brito [mailto:ma...@ca...]
> Sent: 7. april 2003 14:42
> To: Aslak Hellesoy
> Subject: Re: [Xdoclet-devel] CVS: xdoclet2/lib/rngconv/jars
> rngconv-20030225.jar,NONE,1.1
>
>
>
> > Update of /cvsroot/xdoclet/xdoclet2/lib/rngconv/jars
> > In directory sc8-pr-cvs1:/tmp/cvs-serv14703/lib/rngconv/jars
> >
> > Added Files:
> > 	rngconv-20030225.jar
> > Log Message:
> > Initial support for XML Schema in XGG (using Sun's rngconv)
> >
> > --- NEW FILE: rngconv-20030225.jar ---
> > PK
>
> Oh, oh, I've seen this before. Binary files added as text again :) Your
> CVS client is sure broke, Aslak.
>
> --
> Marcus, the CVS cop <ma...@ca...>
>

[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-04-01 17:26

Message:
Logged In: YES 
user_id=535753

Here is a new, bug-fixed version of my patch (again relative
to CVS revision 1.29.  In this version, it now generates
correct CVSWeb references for deleted files (aka files in
the Attic).

(Let's see if my patch is attached to my comment this time;
if not, I'll send it in separate mail to the mailing list.)

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

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

[Cvs-syncmail] Man page bugs

[Kevin R. <ke...@ro...>]

Here's a bug report forwarded from a Debian user:

> Package: cvs-syncmail
> Version: 1.2+cvs.2003.03.20-1
>
> The -m, -S and -R options at least are missing from the manpage.
> Also, the manpage's description of -h is odd to say the least.

Thanks!

--=20
       Kevin Rosenberg        |  .''`.  ** Debian GNU/Linux **
  http://b9.com/debian.html   | : :' :      The  universal
  GPG signed and encrypted    | `. `'      Operating System
     messages accepted.       |   `-    http://www.debian.org/

[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

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

[Michael H. <ho...@sa...>]

(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.

Kind regards,
Michael
-- 
ho...@sa..., ho...@in...
http://home.pages.de/~hohmuth/

[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-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

[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

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

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

[Cvs-syncmail] [ cvs-syncmail-Bugs-583345 ] ValueError: unpack list of wrong size

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

Bugs item #583345, was opened at 2002-07-18 09:26
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=583345&group_id=47611

Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Mike Noyes (mhnoyes)
>Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: ValueError: unpack list of wrong size

Initial Comment:
Imports that contain empty directories cause a ValueError.

Traceback (innermost last):
  File "/cvsroot/leaf/CVSROOT/syncmail", line 322, in ?
    main()
  File "/cvsroot/leaf/CVSROOT/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:],
contextlines, fromhost)
  File "/cvsroot/leaf/CVSROOT/syncmail", line 240, in
blast_mail
    print calculate_diff(file, contextlines)
  File "/cvsroot/leaf/CVSROOT/syncmail", line 139, in
calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size

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

>Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-03-18 10:40

Message:
Logged In: YES 
user_id=3066

Ok, I think this is fixed in CVS now.  I checked in a
modified version of the patch that should with both the %s
replacement in the command line and with files that contain
commas in the filename.
Try using syncmail 1.29 from the cvs-syncmail CVS.

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

Comment By: Michael Hohmuth (rotofoil)
Date: 2003-03-18 10:00

Message:
Logged In: YES 
user_id=535753

Here's a simple patch that avoids the error with a loginfo
specification of "%s" (as opposed to "${sVv}").  (For file
names containing commas (","), the error is avoided, too,
but no diff is generated.)

Michael

Index: syncmail
===================================================================
RCS file: /cvsroot/opencxx/CVSROOT/syncmail,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -p -r1.1 -r1.2
--- syncmail    18 Mar 2003 14:31:01 -0000      1.1
+++ syncmail    18 Mar 2003 14:32:30 -0000      1.2
@@ -136,7 +136,12 @@ def usage(code, msg=''):
 
 

 def calculate_diff(filespec, contextlines):
-    file, oldrev, newrev = string.split(filespec, ',')
+    fspecrev = string.split(filespec, ',')
+    if len(fspecrev) <> 3:
+        return ''
+
+    file, oldrev, newrev = fspecrev
+
     # Make sure we can find a CVS version number
     if not REVCRE.match(oldrev):
         return NOVERSION % filespec


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

Comment By: Patrik Fimml (fipa)
Date: 2002-12-31 07:19

Message:
Logged In: YES 
user_id=655162

I've got the same problem always when I want to create a 
module.

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

Comment By: Nobody/Anonymous (nobody)
Date: 2002-12-03 08:53

Message:
Logged In: NO 

Imports that contain a comma "," in the file name also cause
the ValueError. It would be nice to see this fixed!


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

Comment By: Johan Andersson (mrphonig)
Date: 2002-08-18 09:09

Message:
Logged In: YES 
user_id=579869

I got this error when I imported about 500 files at the same 
time.

---- 

Traceback (most recent call last):
  File "/cvsroot/bin/syncmail", line 322, in ?
    main()
  File "/cvsroot/bin/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:], contextlines, 
fromhost)
  File "/cvsroot/bin/syncmail", line 240, in blast_mail
    print calculate_diff(file, contextlines)
  File "/cvsroot/bin/syncmail", line 139, in calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size


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

Comment By: Mike Noyes (mhnoyes)
Date: 2002-08-14 09:23

Message:
Logged In: YES 
user_id=39521

Ernst,
I've always used "%{sVv}". It doesn't correct the problem I
reported above.

ref.
http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/leaf/CVSROOT/

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

Comment By: Ernst de Haan (znerd)
Date: 2002-08-14 07:35

Message:
Logged In: YES 
user_id=11053

I solved the problem by using %{sVv} instead of %s only. For
example:

test   $CVSROOT/CVSROOT/syncmail %{sVv} cv...@cv...

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

Comment By: Ernst de Haan (znerd)
Date: 2002-08-14 07:32

Message:
Logged In: YES 
user_id=11053

I have the same problem with revision 1.20.

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

Comment By: Chris Boot (bootc)
Date: 2002-08-12 14:47

Message:
Logged In: YES 
user_id=151802

This also seems to be related to a problem when checking files in 
directories that have spaces in their names.

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

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

[Cvs-syncmail] syncmail syncmail,1.28,1.29

[Fred L. D. <fd...@us...>]

Update of /cvsroot/cvs-syncmail/syncmail
In directory sc8-pr-cvs1:/tmp/cvs-serv14017

Modified Files:
	syncmail 
Log Message:
Fix error when the loginfo contains a replacement of %s instead of %{sVv}:
just don't produce a diff.  This also allows commas in filenames when
%{sVv} is used.


Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/syncmail/syncmail,v
retrieving revision 1.28
retrieving revision 1.29
diff -u -d -r1.28 -r1.29
--- syncmail	14 Mar 2003 21:32:15 -0000	1.28
+++ syncmail	18 Mar 2003 15:13:35 -0000	1.29
@@ -155,7 +155,17 @@
 
 
 def calculate_diff(filespec, contextlines):
-    file, oldrev, newrev = string.split(filespec, ',')
+    spec = string.split(filespec, ',')
+    if len(spec) < 3:
+        # Too few parts; command line probable used a replacement
+        # other than "%{sVv}"; don't fail, but don't produce a diff
+        # since we can't be sure what diff to generate.
+        return ''
+    # This allows filenames that contain commas:
+    file = string.join(spec[:-2], ",")
+    oldrev = spec[-2]
+    newrev = spec[-1]
+
     # Make sure we can find a CVS version number
     if not REVCRE.match(oldrev):
         return NOVERSION % filespec

[Cvs-syncmail] [ cvs-syncmail-Bugs-583345 ] ValueError: unpack list of wrong size

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

Bugs item #583345, was opened at 2002-07-18 15:26
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=583345&group_id=47611

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Mike Noyes (mhnoyes)
Assigned to: Nobody/Anonymous (nobody)
Summary: ValueError: unpack list of wrong size

Initial Comment:
Imports that contain empty directories cause a ValueError.

Traceback (innermost last):
  File "/cvsroot/leaf/CVSROOT/syncmail", line 322, in ?
    main()
  File "/cvsroot/leaf/CVSROOT/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:],
contextlines, fromhost)
  File "/cvsroot/leaf/CVSROOT/syncmail", line 240, in
blast_mail
    print calculate_diff(file, contextlines)
  File "/cvsroot/leaf/CVSROOT/syncmail", line 139, in
calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size

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

Comment By: Michael Hohmuth (rotofoil)
Date: 2003-03-18 16:00

Message:
Logged In: YES 
user_id=535753

Here's a simple patch that avoids the error with a loginfo
specification of "%s" (as opposed to "${sVv}").  (For file
names containing commas (","), the error is avoided, too,
but no diff is generated.)

Michael

Index: syncmail
===================================================================
RCS file: /cvsroot/opencxx/CVSROOT/syncmail,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -p -r1.1 -r1.2
--- syncmail    18 Mar 2003 14:31:01 -0000      1.1
+++ syncmail    18 Mar 2003 14:32:30 -0000      1.2
@@ -136,7 +136,12 @@ def usage(code, msg=''):
 
 

 def calculate_diff(filespec, contextlines):
-    file, oldrev, newrev = string.split(filespec, ',')
+    fspecrev = string.split(filespec, ',')
+    if len(fspecrev) <> 3:
+        return ''
+
+    file, oldrev, newrev = fspecrev
+
     # Make sure we can find a CVS version number
     if not REVCRE.match(oldrev):
         return NOVERSION % filespec


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

Comment By: Patrik Fimml (fipa)
Date: 2002-12-31 13:19

Message:
Logged In: YES 
user_id=655162

I've got the same problem always when I want to create a 
module.

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

Comment By: Nobody/Anonymous (nobody)
Date: 2002-12-03 14:53

Message:
Logged In: NO 

Imports that contain a comma "," in the file name also cause
the ValueError. It would be nice to see this fixed!


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

Comment By: Johan Andersson (mrphonig)
Date: 2002-08-18 15:09

Message:
Logged In: YES 
user_id=579869

I got this error when I imported about 500 files at the same 
time.

---- 

Traceback (most recent call last):
  File "/cvsroot/bin/syncmail", line 322, in ?
    main()
  File "/cvsroot/bin/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:], contextlines, 
fromhost)
  File "/cvsroot/bin/syncmail", line 240, in blast_mail
    print calculate_diff(file, contextlines)
  File "/cvsroot/bin/syncmail", line 139, in calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size


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

Comment By: Mike Noyes (mhnoyes)
Date: 2002-08-14 15:23

Message:
Logged In: YES 
user_id=39521

Ernst,
I've always used "%{sVv}". It doesn't correct the problem I
reported above.

ref.
http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/leaf/CVSROOT/

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

Comment By: Ernst de Haan (znerd)
Date: 2002-08-14 13:35

Message:
Logged In: YES 
user_id=11053

I solved the problem by using %{sVv} instead of %s only. For
example:

test   $CVSROOT/CVSROOT/syncmail %{sVv} cv...@cv...

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

Comment By: Ernst de Haan (znerd)
Date: 2002-08-14 13:32

Message:
Logged In: YES 
user_id=11053

I have the same problem with revision 1.20.

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

Comment By: Chris Boot (bootc)
Date: 2002-08-12 20:47

Message:
Logged In: YES 
user_id=151802

This also seems to be related to a problem when checking files in 
directories that have spaces in their names.

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

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

[Cvs-syncmail] Re: syncmail alternative

[Klaus J. R. <Kla...@at...>]

In <200...@ba...>, Joost van Baal <j.e...@uv...> writes:
> In the syncmail manpages, you ask to get informed about tools that serve
> similar purposes.  I've been using log_accum before, a Perl script.  You
> can get it from
> 
> http://savannah.gnu.org/cgi-bin/viewcvs/savannah/savannah/cvsadmin/log_accum

Please also add

  cvs-syncmail (Perl version)
  http://cvs-syncmail-pl.sourceforge.net/

to the list of related tools.


Thanks

-- 
Klaus Johannes Rusch
Kla...@at...
http://www.atmedia.net/KlausRusch/

[Cvs-syncmail] [ cvs-syncmail-Bugs-626140 ] Every import syncmail it fails

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

Bugs item #626140, was opened at 2002-10-20 20:26
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=626140&group_id=47611

Category: None
Group: None
>Status: Closed
>Resolution: Duplicate
Priority: 5
Submitted By: Serguei A. Mokhov (mokhov)
Assigned to: Nobody/Anonymous (nobody)
Summary: Every import syncmail it fails

Initial Comment:
No message is sent because syncmail fails as below:

....
No conflicts created by this import

Mailing mar...@li......
Generating notification message...
Generating notification message... done.
Mailing mar...@li......
Generating notification message...
Traceback (innermost last):
  File "/cvsroot/marf/CVSROOT/syncmail", line 322, in ?
    main()
  File "/cvsroot/marf/CVSROOT/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:],
contextlines, fromhost)
  File "/cvsroot/marf/CVSROOT/syncmail", line 240, in
blast_mail
    print calculate_diff(file, contextlines)
  File "/cvsroot/marf/CVSROOT/syncmail", line 139, in
calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size


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

>Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-03-14 16:48

Message:
Logged In: YES 
user_id=3066

This is a duplicate of SF bug #583345.

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

Comment By: Nobody/Anonymous (nobody)
Date: 2003-01-15 22:52

Message:
Logged In: NO 

I am seeing this as well.  I am using WinCVS (actually got 
same error using Eclipse as well) as the client.  I am entering 
the module into a subdirectory of the cvsroot.

In other words, the command looks like:

cvs -z9 import -m "comment" subdir/module vendor tag

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

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

[Cvs-syncmail] [ cvs-syncmail-Bugs-696462 ] syncmail not compatible with Python 1.5.2.

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

Bugs item #696462, was opened at 2003-03-03 05:22
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=696462&group_id=47611

Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Sjoerd Mullender (sjoerd)
>Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: syncmail not compatible with Python 1.5.2.

Initial Comment:
Revision 1.24 introduced an incompatibility with Python
1.5.2.
The function quotename uses a string method which was
not yet present in Python 1.5.2.

Replace name.replace('"', '\"') with
string.replace(name, '"', '\"').

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

>Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2003-03-14 16:45

Message:
Logged In: YES 
user_id=3066

Fixed in syncmail/syncmail 1.28.

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

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

[Cvs-syncmail] syncmail syncmail,1.27,1.28

[Fred L. D. <fd...@us...>]

Update of /cvsroot/cvs-syncmail/syncmail
In directory sc8-pr-cvs1:/tmp/cvs-serv7100

Modified Files:
	syncmail 
Log Message:
Fix Python 1.5.2 compatibility: avoid string methods.
Closes SF bug #696462.


Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/syncmail/syncmail,v
retrieving revision 1.27
retrieving revision 1.28
diff -u -d -r1.27 -r1.28
--- syncmail	16 Jan 2003 07:11:19 -0000	1.27
+++ syncmail	14 Mar 2003 21:32:15 -0000	1.28
@@ -234,7 +234,7 @@
 
 def quotename(name):
     if name and rfc822_specials_re.search(name):
-        return '"%s"' % name.replace('"', '\\"')
+        return '"%s"' % string.replace(name, '"', '\\"')
     else:
         return name
 

[Cvs-syncmail] [ cvs-syncmail-Patches-702373 ] Added Date: header

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

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

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Stefan Puiu (stefanpuiu)
Assigned to: Nobody/Anonymous (nobody)
Summary: Added Date: header

Initial Comment:
syncmail doesn't put a Date: field in mail headers, and
qmail doesn't add missing headers to emails it
receives, so if you used an MUA behind qmail (like
ezmlm and colobus, as I do), the date header would be
missing from the received message. I get a date of "1st
of January 1970", which of course isn't accurate.
Taking into consideration that qmail isn't likely to
change, it would be a good idea to have syncmail cope
with these differences. 

So I made this patch to add a Date: field; it could be
fixed to add the offset from GMT also. (besides what it
prints now). 

I've written the few lines there by reading python docs
and examples, I don't know any python (or didn't know
until now) so feel free to criticise :D. 

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

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

[Cvs-syncmail] Patch for date field

[Kevin R. <ke...@ro...>]

Hello cvs-syncmail folks,

The below explanation and patch were submitted by a Debian user:

-- 
       Kevin Rosenberg        |  .''`.  ** Debian GNU/Linux **
  http://b9.com/debian.html   | : :' :      The  universal
  GPG signed and encrypted    | `. `'      Operating System
     messages accepted.       |   `-    http://www.debian.org/


> When cvs-syncmail sends an E-mail, the date header is not included in
> the E-mail.  This causes some mail readers to sort the new E-mail
> incorrectly.  This patch fixes the problem.
>
> -Tim Winter


--- /usr/bin/syncmail	Sat Jan 18 11:35:16 2003
+++ syncmail	Tue Mar 11 10:18:04 2003
@@ -262,6 +262,7 @@
                     'people'  : string.join(people, COMMASPACE),
                     'subject' : subject,
                     'version' : __version__,
+                    'date'    : time.strftime('%a, %d %b %Y %H:%M:%S +0000', time.gmtime())
                     }
             print '''\
 From: %(name)s <%(address)s>
@@ -271,6 +272,7 @@
             print '''\
 Subject: %(subject)s
 X-Mailer: Python syncmail %(version)s <http://sf.net/projects/cvs-syncmail>
+Date: %(date)s
 ''' % vars
             s.write(sys.stdin.read())
             # append the diffs if available

[Cvs-syncmail] Filtering actions

[Sergio B. <ser...@ex...>]

Hi there

Is it possible to filter messages in order to trig only some CVS actions
like Branch or tag creation?

thanks

Sergio

[Cvs-syncmail] (no subject)

[Sergio B. <ser...@ex...>]

Hi there

Is it possible to filter messages in order to trig only some CVS actions
like Branch or tag creation?

thanks

Sergio

[Cvs-syncmail] (no subject)

[Sergio B. <ser...@ex...>]

Hi there
Is it possible to filter messages to trig only some af the CVS activities
like Branch or tag creation?

thanks

Sergio

[Cvs-syncmail] [ cvs-syncmail-Bugs-696462 ] syncmail not compatible with Python 1.5.2.

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

Bugs item #696462, was opened at 2003-03-03 11:22
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=450019&aid=696462&group_id=47611

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Sjoerd Mullender (sjoerd)
Assigned to: Nobody/Anonymous (nobody)
Summary: syncmail not compatible with Python 1.5.2.

Initial Comment:
Revision 1.24 introduced an incompatibility with Python
1.5.2.
The function quotename uses a string method which was
not yet present in Python 1.5.2.

Replace name.replace('"', '\"') with
string.replace(name, '"', '\"').

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

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

Re: [Cvs-syncmail] "Permission denied" for syncmail

[Mike N. <mh...@us...>]

Tom,
Read the SF Site Doc below. If that doesn't help, you'll need to open a
support request with the SF staff.

# F3. Enhancing Project CVS Services Using Scripts
https://sourceforge.net/docman/display_doc.php?docid=772&group_id=1

SourceForge.net: Support Request
https://sourceforge.net/tracker/?func=add&group_id=1&atid=200001

On Fri, 2003-02-14 at 17:05, [TOM] wrote:
> Howdy,
> 
> I tried to get syncmail running on SF for our project phpcms. But I only 
> get a
> 
>      sh: /cvsroot/phpcms/CVSROOT/syncmail: Permission denied
> 
> 
> What I've done (I used Debian):
> 
> 1. I checkout CVSROOT to a local directory:
> 
> 2. I add the line "syncmail" to checkoutlist
> 
> 3. I add
> 
>    DEFAULT $CVSROOT/CVSROOT/syncmail -u %{sVv} nz...@us...
> 
>    to loginfo
> 
> 4. I commit checkoutlist and loginfo
> 
> 5. Copy syncmail to my local CVSROOT directory
> 
> 6. $ chmod a+x ./syncmail
> 
> 7. $ cvs add ./syncmail
> 
> 8. $ cvs commit -m "Syncmail added" ./syncmail
> 
> I got a "cvs server: Rebuilding administrative database". But when I 
> commit an other file to test syncmail, I got the error "sh: 
> /cvsroot/phpcms/CVSROOT/syncmail: Permission denied"
> 
> I used the actual syncmail from CVS so I tested the new -m switch to use 
> an other mail host.
> 
> I don't know the user CVS use to execute the script. The files described 
> above are available at 
> http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/phpcms/CVSROOT/.
> 
> Maybe somebody knows this problem and can give me a hint. I didn't find 
> anything in the mail archive.

-- 
Mike Noyes <mhnoyes @ users.sourceforge.net>
http://sourceforge.net/users/mhnoyes/
http://leaf-project.org/  http://sitedocs.sf.net/  http://ffl.sf.net/

[Cvs-syncmail] "Permission denied" for syncmail

[[TOM] <to...@xs...>]

Howdy,

I tried to get syncmail running on SF for our project phpcms. But I only 
get a

     sh: /cvsroot/phpcms/CVSROOT/syncmail: Permission denied


What I've done (I used Debian):

1. I checkout CVSROOT to a local directory:

2. I add the line "syncmail" to checkoutlist

3. I add

   DEFAULT $CVSROOT/CVSROOT/syncmail -u %{sVv} nz...@us...

   to loginfo

4. I commit checkoutlist and loginfo

5. Copy syncmail to my local CVSROOT directory

6. $ chmod a+x ./syncmail

7. $ cvs add ./syncmail

8. $ cvs commit -m "Syncmail added" ./syncmail

I got a "cvs server: Rebuilding administrative database". But when I 
commit an other file to test syncmail, I got the error "sh: 
/cvsroot/phpcms/CVSROOT/syncmail: Permission denied"

I used the actual syncmail from CVS so I tested the new -m switch to use 
an other mail host.

I don't know the user CVS use to execute the script. The files described 
above are available at 
http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/phpcms/CVSROOT/.

Maybe somebody knows this problem and can give me a hint. I didn't find 
anything in the mail archive.

thx

[TOM]


-- 
Fingerprint: 7AFA 35A2 3136 4365 E7AF 8D1D F5C2 00E9 C051 201E
Key:         mailto:pgp...@df...?subject=GET%200xC051201E
Updated:     2002-11-08 (yyyy-mm-dd)