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

[Cvs-syncmail] Patch: fix bug #592836

[Greg W. <gw...@py...>]

While I'm in the mood, here's another patch for syncmail.  This one
fixes bug #592836, which complains about syncmail generating "From:"
headers like

  From: "" <us...@so...main>

-- apparently some mail clients don't like that.  And besides, it's
ugly.  This patch fixes syncmail to only quote names when they actually
need to be quoted, ie. when they contain RFC 822 "special" characters.
This patch is relative to the "reply-to" patch I just posted.

--- syncmail.reply-to	2002-10-29 14:26:44.000000000 -0500
+++ syncmail	2002-10-29 14:36:43.000000000 -0500
@@ -219,6 +219,16 @@
 
 
 
+rfc822_specials_re = re.compile(r'[\(\)\<\>\@\,\;\:\\\"\.\[\]]')
+
+def quotename(name):
+    if name and rfc822_specials_re.search(name):
+        return '"%s"' % name.replace('"', '\\"')
+    else:
+        return name
+
+
+
 def blast_mail(subject, people, filestodiff, contextlines, fromhost, replyto):
     # cannot wait for child process or that will cause parent to retain cvs
     # lock for too long.  Urg!
@@ -236,7 +246,10 @@
         s = StringIO()
         sys.stdout = s
         try:
-            print 'From: "%s" <%s>' % (name, address)
+            if name:
+                print 'From: %s <%s>' % (quotename(name), address)
+            else:
+                print 'From: %s' % address
             print 'To: %s' % string.join(people, COMMASPACE)
             print 'Subject: %s' % subject
             if replyto:


-- 
Greg Ward <gw...@py...>                         http://www.gerg.ca/
Eschew obfuscation!

[Cvs-syncmail] Patch: add Reply-To header

[Greg W. <gw...@py...>]

Here's a patch to syncmail that adds a -R/--reply-to option, which makes
syncmail add a "Reply-To" header to the email message.  IMHO this is
long overdue -- it makes perfect sense for replies to foo-checkins to be
directed at foo-devel, and this is the obvious way to do it.

Anyone object?  Or shall I go ahead and check this in?

--- syncmail	19 Sep 2002 19:30:15 -0000	1.21
+++ syncmail	29 Oct 2002 19:26:48 -0000
@@ -51,6 +51,10 @@
     --subject-prefix=TEXT
         Prepend TEXT to the email subject line.
 
+    -R ADDR
+    --reply-to=ADDR
+      Add a "Reply-To: ADDR" header to the email message.
+
     --quiet / -q
         Don't print as much status to stdout.
 
@@ -215,7 +219,7 @@
 
 
 
-def blast_mail(subject, people, filestodiff, contextlines, fromhost):
+def blast_mail(subject, people, filestodiff, contextlines, fromhost, replyto):
     # cannot wait for child process or that will cause parent to retain cvs
     # lock for too long.  Urg!
     if not os.fork():
@@ -232,17 +236,15 @@
         s = StringIO()
         sys.stdout = s
         try:
-            print '''\
-From: "%(name)s" <%(address)s>
-To: %(people)s
-Subject: %(subject)s
-X-Mailer: Python syncmail %(version)s <http://sf.net/projects/cvs-syncmail>
-''' % {'address' : address,
-       'name'    : name,
-       'people'  : string.join(people, COMMASPACE),
-       'subject' : subject,
-       'version' : __version__,
-       }
+            print 'From: "%s" <%s>' % (name, address)
+            print 'To: %s' % string.join(people, COMMASPACE)
+            print 'Subject: %s' % subject
+            if replyto:
+                print 'Reply-To: %s' % replyto
+            print ('X-Mailer: Python syncmail %s '
+                   '<http://sf.net/projects/cvs-syncmail>'
+                   % __version__)
+
             s.write(sys.stdin.read())
             # append the diffs if available
             print
@@ -260,8 +262,9 @@
 def main():
     try:
         opts, args = getopt.getopt(
-            sys.argv[1:], 'hC:cuS:qf:',
-            ['fromhost=', 'context=', 'cvsroot=', 'subject-prefix=',
+            sys.argv[1:], 'hC:cuS:R:qf:',
+            ['fromhost=', 'context=', 'cvsroot=',
+             'subject-prefix=', 'reply-to=',
              'help', 'quiet'])
     except getopt.error, msg:
         usage(1, msg)
@@ -270,6 +273,7 @@
     contextlines = 2
     verbose = 1
     subject_prefix = ""
+    replyto = None
     fromhost = None
     for opt, arg in opts:
         if opt in ('-h', '--help'):
@@ -285,6 +289,8 @@
             contextlines = 0
         elif opt in ('-S', '--subject-prefix'):
             subject_prefix = arg
+        elif opt in ('-R', '--reply-to'):
+            replyto = arg
         elif opt in ('-q', '--quiet'):
             verbose = 0
         elif opt in ('-f', '--fromhost'):
@@ -326,7 +332,7 @@
 
     if verbose:
         print 'Generating notification message...'
-    blast_mail(subject, people, specs[1:], contextlines, fromhost)
+    blast_mail(subject, people, specs[1:], contextlines, fromhost, replyto)
     if verbose:
         print 'Generating notification message... done.'
 


-- 
Greg Ward <gw...@py...>                         http://www.gerg.ca/
Never put off till tomorrow what you can avoid all together.

RE: [Cvs-syncmail] ValueError: unpack list of wrong size

[Wim K. <wi...@ny...>]

> On Fri, 2002-10-04 at 09:12, Wim Kerkhoff wrote:
> > I noticed your message while searching for the exact same 
> error. After a
> > whole bunch of poking around, I discovered that the 
> documention at the
> > top of the syncmail script is not correct. CVSROOT/loginfo expects
> > something like this:
> > 
> > module /path/to/syncmail %{sVv} yo...@ad... an...@so...
> > 
> > If you put %%s as described in syncmail, then the string 
> "s" will get
> > sent to the script and not the module and files that were changed.
> 
> Wim,
> I have always used %{sVv}, so I doubt that's the problem.

This worked for me: %{sVv}
This did not work for me: %%s

Wim.

Re: [Cvs-syncmail] ValueError: unpack list of wrong size

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

On Fri, 2002-10-04 at 09:12, Wim Kerkhoff wrote:
> I noticed your message while searching for the exact same error. After a
> whole bunch of poking around, I discovered that the documention at the
> top of the syncmail script is not correct. CVSROOT/loginfo expects
> something like this:
> 
> module /path/to/syncmail %{sVv} yo...@ad... an...@so...
> 
> If you put %%s as described in syncmail, then the string "s" will get
> sent to the script and not the module and files that were changed.

Wim,
I have always used %{sVv}, so I doubt that's the problem.

583345  ValueError: unpack list of wrong size
https://sourceforge.net/tracker/?func=detail&aid=583345&group_id=47611&atid=450019

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

-- 
Mike Noyes <mh...@us...>
http://sourceforge.net/users/mhnoyes/
http://leaf-project.org/

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

[Wim K. <wi...@ny...>]

Murat,

I noticed your message while searching for the exact same error. After a
whole bunch of poking around, I discovered that the documention at the
top of the syncmail script is not correct. CVSROOT/loginfo expects
something like this:

module /path/to/syncmail %{sVv} yo...@ad... an...@so...

If you put %%s as described in syncmail, then the string "s" will get
sent to the script and not the module and files that were changed.

Wim

[Cvs-syncmail] syncmail syncmail,1.20,1.21

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

Update of /cvsroot/cvs-syncmail/syncmail
In directory usw-pr-cvs1:/tmp/cvs-serv26547

Modified Files:
	syncmail 
Log Message:
Fix typo in docstring.


Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/syncmail/syncmail,v
retrieving revision 1.20
retrieving revision 1.21
diff -u -d -r1.20 -r1.21
--- syncmail	5 Aug 2002 22:31:42 -0000	1.20
+++ syncmail	19 Sep 2002 19:30:15 -0000	1.21
@@ -49,7 +49,7 @@
 
     -S TEXT
     --subject-prefix=TEXT
-        Preprend TEXT to the email subject line.
+        Prepend TEXT to the email subject line.
 
     --quiet / -q
         Don't print as much status to stdout.

Re: [Cvs-syncmail] syncmail 1.20 -- cvs 1.11 --- pyton 1.5.2 --- ValueError: unpack list of wrong size

[Murat B. <mu...@ki...>]

MIAGCSqGSIb3DQEHA6CAMIACAQAxggJsMIIBMgIBADCBmjCBkjELMAkGA1UEBhMCWkExFTAT
BgNVBAgTDFdlc3Rlcm4gQ2FwZTESMBAGA1UEBxMJQ2FwZSBUb3duMQ8wDQYDVQQKEwZUaGF3
dGUxHTAbBgNVBAsTFENlcnRpZmljYXRlIFNlcnZpY2VzMSgwJgYDVQQDEx9QZXJzb25hbCBG
cmVlbWFpbCBSU0EgMjAwMC44LjMwAgMIDhEwDQYJKoZIhvcNAQEBBQAEgYAGVwlVv+uzXi8I
VyBQS9zGNrjUudwfLwcuiODASrm20/cqeCZv8rtEohiglLcSVkqnJGawLSDM1Of4vhswIRBs
6H0eqbkAzu2oM39UinO8ZGm5jxudkGxzbKmnrhi/VAqSbM9owOu+E0bbelqYDJeT7P0FcDRD
dP4Pf1O5G+6qbTCCATICAQAwgZowgZIxCzAJBgNVBAYTAlpBMRUwEwYDVQQIEwxXZXN0ZXJu
IENhcGUxEjAQBgNVBAcTCUNhcGUgVG93bjEPMA0GA1UEChMGVGhhd3RlMR0wGwYDVQQLExRD
ZXJ0aWZpY2F0ZSBTZXJ2aWNlczEoMCYGA1UEAxMfUGVyc29uYWwgRnJlZW1haWwgUlNBIDIw
MDAuOC4zMAIDCA4RMA0GCSqGSIb3DQEBAQUABIGAOdf3Tm8UyOazE4VJ7AKstwlauXq/PKpx
gTdgCC+XERoUPtrSzp4L0QCZ2H8yOdFB84oDsvvzQGsSDKgEHKR6DtTPP/tLngOJSFnK0nfw
G03Nnz7gn8cBP8PVp61YTQt96d5f0xw3LavsGCmWe3uI8KO8jB51KpgOptC7n/NTvNgwgAYJ
KoZIhvcNAQcBMBQGCCqGSIb3DQMHBAhUySMddQ7H2qCABIGw4NdsjkExW2n91VzidDuHyesx
65YF4tWp4v8Q/twZlLza9UuSGobwk4OdrIYee7ytZxMG1jsFOR+TYoXWUhO4MjqxpbK9ROpd
ZD1U+oM3fugv629TWsVuiW/Z9OuYLVWed0IBAT42XD5onz84FosQMQGh1csc6L1QKqBJoNzG
0FXWZl6xsO37EA7e8a4h15DIlh6VK+pbIj3SQ3pBPHmjKN84TBdjKqduBsKRY8sBOnEEQBXP
f6YrgizqJAi6Xf6W4rXpB1TGMBjmkg6DlZD4wPSTimC26jkrPgM9WarBlAdew10i6ZzUxxIg
jJlvPKgxiNIEIEcdiNp1VM9w4YrnHvgiUSk2RURNpKzHGgMQUZSIyi5lBDjtCqmpPdlW804a
jAMD93kBxw9g/JaiacSVrUjD5cR3EDuqJmYiE3c+YDXoNO18faw5NNvA7hBq7wQIXBoGt47E
CyUEEPd99tiMHqe++rVA50uMJlsEKLwSwwLFBip/HmzIQjG3wPWTcvZdcoTioCNZL8KqXTix
dfxd2QJv8vkEMDa4SguP6dW48ELChHiSSNLvE//jjbNRLqm59VszMCsTePhBrDXQMTgbYcBK
ga6VtQQgQ0HiY/H0M+1bdnzxqqIwuBC16hBvocW7YwUaL0AQDrkEIKgTNEenOuMdLRupqWlE
uERxt0BOi55PbZnRlaNtNzMIBAjqbB1RbimZQgQIEJ+84nv0kakECBcbMXXqdMhyBCDzkeEe
IPNDyOAQZAbj1gzXwIOO4U+HbmHK6jGQWe4Q2AQYvC+NAdBr8Iayr3QUM3GN+h23yN9zvmDw
BDDCh0jt1/rfXT3MQdauBB2hzP14fNAo0H24NFoD0yRYnoCCwE1hzdreaG59bE/ZrssEMO2N
4NO82oNybthryrC04/zAOffYKKXTmhyvpESxidQ3eS79122ge3r/Gk5/6mVOTQQInGDDNp8c
SxYEIGAP6q9l0TikdJGRNAhx5Zzh2ObBuHRyJaWmekFdkBf0BCiSwNj0iWETy5V1TVfwJgeU
U5vU8rjI24Il7ZXYlpM8VVeHdX4k5+0nBCh2ie7IF5gWZo/BaAr6FXxxvFrmSuA+vxYJ0lff
AxM3L1RlaDse0HVzBCCm4lidhw4LIhRcGdO67PmQXOsh2K5LdZvP/pqhkvILFAQoJTXYQmME
TerYoUhLvu9Z+ua/339QmPJOV0SaIIoJHJF5k5O4BGs61AQg9XKOyJska0RIRNIX6jROIPYu
GOb0E8ypS/U2KIXxcHcEMAn+RKrWs99wu4mZrSaixo9bYWgdbFFHdUQydsHYESQ7uVDC54F8
xggoiV6TSuXV+QQIfq9+Wa40oDQEMNLPF1hhPNITTevqyQYgisoRs4xebxgk4/dvo9Pzd3Fd
OBgz+w39VDn6ALxuOhW0SgRIL/YZU+H+WLyiWGPxFn2cwo2l5ujLFtyz8HHNMyrxQbj6U7Zy
D05MVd9WUSYALDmtPX1XEaVJtoD7YuLYctBGSHJ8xOwrgPKmBDguaELyDL/mqBflQ83c5LtG
62cJcfHuCLVgJmOyaUA0byU/D8IftIU0b3nys2v5PEGZMeBTgSA/5QQofd+l29pnROfY9YMa
HMHFkWJ8J7eapAUT0mj96DYvNTsj3XzkXD4/lARAin/fvkD9IE6OlyUkwREvFCIa1ELCOiV2
TRn0Yrb6g3Mgr410OFT5cl8hrH+1daSzgHlLZ5ldtljitniFGz34ngQ4KgPC3etz2Ga0FRoQ
vQwjnDP91DUu/AZ2BSqIMe/RdzDXSjm2knWs3GkSU932dTwkoyvVOhWseJgEKEF76vtHSJw9
/bRuvGGKplWXZjVyiqCpDZlnQJj08sGKBSQXNxLzP1gECFNIvlKJ73JtBIH4i/kNxIkiQs1W
kmEkXDGMWWDcxNaWguzpu/CcCAxsiluyOlhgtGJU+s/lZ1NVxvHdvwZXYKrUxkGchhvjvutJ
HjxvIhXM+V5CYqKMRnGAmaKNBQxb+l6J/p2ZQjaC9WnsBAPWxwTH1LQVn1Lr74CYmuRjMVzw
ENwbZgkXALw5+O40Ley4K75bmB7TNkAAOzIlFAC13YJ27FTHOOjCQa/gs7tiBJwekxddm+3c
H5e2x42Lvtbz8X2X+EPhWk7Q0fBTZd0WiWsxxhJmNw2qZs3sUNSKyLc05yeWr5yQ0zEpsz90
4PiigEMzVNb+5WOKTvrPlkL/CNHKW5EECMHY9zLkir0uBAhYM5vTO/PnrAQIxZvlO+iLNp0E
CMY/mWLAOXBRBAhKwnW4KRe82AQIfRfy4iAIf7cEEBcJ+PEzwhgldkpxzWb5EfAECDwY1L3H
xr4RBFCTZFprqvvnUFrFUiF870X9L2VnEDjWU0LFdD0PRLFOlUUtmaXJ7cAg9l6GXPyxQaME
m7GO+fYZi8FOoLMcCzoUmGVRKrEtGwzeL2Qc9fDu6ARIFLAY+IIbVCMSO91onsKKO6OYtQSw
9FeCgkFR+aKxdrb4qFrBOlxdvWkqFRZEKfzp3z6PfFV46w8YouJUO+D7dLRN+jiIwlm+BEih
kVuNktZS9K2zXx59NOXvzIhXiHQ9MLXZ5AE13O35b1AllKztF89J2GYfntqD5nqZG87vtPkj
dkmvGMe6wO0s/BT6wDst9+gESCIwkXElTqvumqM1mdba9wK+3lNZuENlf08gwCF1w6pzqTdc
riTWgvMqP0Iv8SVf44gkI2i6WLokVEP5gsk+L1q2KYt4GeesTgRQNFAGyeH76z3M8E3ORah4
qXDOrcFwYyJUEmqme8TL2ehP7isG2XfpZaIQYU4nwZ7npSGy0zjH3xrn3z19jufRNifBo8lB
QDNwDfJI10z9RAcESO7blc6rBMHMmhC0AfkFRzmxRU9GTY/VUTHrojwdvNxK18K16hetyHz5
kooLsm5/gF/vgoMBj81f3QjhoJ2JPvChzpcul4xONwRI+PnS7J8QVvs1t8PX78Aci33LB/87
huwSLUwAjp9LoiZmgItDhb2NgdPVrL/vjIxtJQLWRe2wJ2Tpehu6M1J/MhqpjvGHMY5yBEhC
werK1gEZ2z/yBRfFHkOpvEW44Z+6lReuyGyYdsF3B29ovORx2hi+ebiWFVG24Bdsi+8/6CXk
n0yUvqt7UI8lPjnzYjDGSKUEUKKq4+aJJn/jo68SIyRK2WKXl+BjYceoTpk63xb9YdLFy9lV
/jCRC8eDZlO1oD5l4A5VGlnPizpHfuQsJmaQ4RHzQPo6I6lJyn3txOqmud6WBEiTKYPP+4bt
O9oaS5ASg4FMLzICDxw8P7sAvJAfqG+tP5kAwXV4uUc/ck8Mifh1/z+I8vgz1nBHXTkEJ+T4
XRHsWsTObYQuzNMESEXtW8edoL3u9jbcbeDOZBLRJ+fdbjun5C3HrhHfTetGyEvi/mxbDVKU
WEBxTI587ZGDfMfuzK3YZvp4BAXn68IQPYn/yWKnKwQ4Ren9Aaju+FCBkQTWrcHkS0ywqX/i
VQtGtOh930Q1DZTUvfkjX2OlaRJ/h4XGPz926iEhXkhVgy0EEB8t69LI1KvOR4LqZm8uvXUE
UGdsSz+xi9ENw9I4+PFcGfcHnkJgZtKigLOZ7/VgWP6OaD515SmcriQeoLM93F8Tyb8Dn8GV
DYOBiqfH1SzRkKvJrA2nWR9b0lG6gMoNyZtmBEin+8OQinx+VTX0jzpkrVnux5tiJrtZahel
mmVjqCIkMxnY294qi81Yakt5VG66rrLo2C8vKAmLpz+OfpDoDNk4MsQDwkJssaEESH+RkuIf
BEoluRf/bVJP1xrTeLGFnEfBTHAHgLW65dot+ut6PGtjThWg8BdL1h2ZdDB2mD2WqZJ5OvZc
ACRuojAiu8d+fyIZVwRIsXM6GCar+CArA9njaA4dAV5m3kLMzyR4PjchLB10DAMOqa+/U7Vl
jsLhmWqBBXuzXsZyI6F31ZS7y9ZfjJTuwFqMtN99Nlw1BFBOO+bbYFSUYtB87T5m0AZ1Omu7
v/BYvQf7k5TQitcnLBWD+3Y7aFKwef6VbShe7kM0sw+5QcchC860pZvNdsvMrHRc2UuKRQ2T
ezb7MfjQ0gRIV4QHzUel5fBVFtD4oN6/IzED+v8dW2Y+r/KbHeZnmKPIWC0BB+VVmdBvSpam
jEB2h8EYLbBhfAVMTIDFpDdZdbLPFGqlL/hCBEgI3BZIRQW+zT+5cFZf7jJI3MFyNdUcgqb0
laEIee/VEnk6QUqWUOSVOwqDSDvuX5WtnkzgOxbyM+wuyb7oC8GcUVozwbzT62cESMfVsuph
i5qh+9hUW387KGJf7zQk75Lxblr4ig/zgHs8EKYBeqh7f4wQzaGiY4tzyo0wt9pwaVi7tJXf
hpsK49YvAVMTDI/eMwRQ6pG0xAEUE81g64GmaXrbW0kozVcdZlFIE7BZhxpva9SNQdsx5Udz
7TRN+6qPEpIEr7HQ2ozgWAwrvH3sEmaWopJWgykU9p25hHfkEZdqYfMESIoLmkbTFWxIk34I
UgdjuNiub0edTc5ec+teZukmhWJO3XEaqXDnQCzbgtWtL6CrBMJkPkkPH5ZbsqMDKFfQy+O7
4RFb8cFW/gRIwp1qQ1tJXgsFo7Dizqb8lX5CJAwv/2H//T3WBSQabcHxKizeA6ANKbncM2Zg
2NM0VkWvBOSbjn18eCWyzz59jrzRrgC95mZIBCjzlnTi7/nxK+jZpFIjVrBNYhF6E6sftWvW
SDgW9UXMGEMCJU7spQkvBCAZ8OobNdicTBCWJ/8DygaC/CsXMvJU2WelPUkGLQZvzgRQQHyp
/O3woPBB8MoJciUtD8I6gJqCJS4YiOnkIUXb1h8rulxJe+tMxqSuukFxsX1ty2hrcZ5YigML
SOG3XKeW0v55i3IVgXJ/QQ7mUm5VPw4ESKN2TNyb9GE0wgYd5VsRVzVZ4Iy3qwymPR09ypYj
9O4BTTEfPl4mJA1k/vVOSYkPwQNzM3jUg9S5FO0nQh0ycfPS1guuoqkDJARITQHimoKFB8ro
ekNs3Xqy4ztqq5cxjCadWD07ZT0muQGn054Yfpb3D6CJrzEWXqd1eo+0uykGo5Cay/wh8c/N
Xhfq661bWEHTBEhP3RM40OP0mgyU9/uWwbh2NiqMbjisabF1Fnqo6XAcFCtqQpA3hgwzP09w
1CARy+ADjxLayviFIH6mU095APYV3SKz+tRT0gMEUP+RASWPxjZqxrnXRmUE2QQAZXU6Z3kl
wEqUpAi4g4DB/yUm+Tu4/MjzJlHWPVfhEASc1mbAYJKeE50qBDn/0IXjAJZMaOFcyiVvd6Bv
uZjZBEhJFDINo4IwjyDsrjp/JsILYqn5mA+pOSzSoXyFl/aI/jlalRCamm0Gl5SYZ5oCplzH
EOWfvGLsUkL85euWUb9RA+qeO8d7PR8ESAjBr/uaIIebzujtgzCtlHSHP0AhIO6x82hDhxeU
w5J6qtE261di9OmlvXkny/csK+m9Tst1JZn8cqolj7oPTdNZW2+vE77F9gRI/oMBSIYv8XRS
kORDTAfDtw/vPIcemjQZIWQ4kfmBF+pYbv8oinTe/LFknx49PeSC14MZAZOCqM5HrbDmEYqp
ESrGoq63ehLsBFDQ79zyVZX/ARMIF9dcOR85jsMbMeUfT5e7i5aCTobn7dG9PUK7U1KJDZbw
kLRKH9zxpXOGq2Tqs+v/7jZHX+2UCV+Fi49y4fToXoTZBGjduQRISdaWb9sZuWCU1TSeFvfy
wfE9HhMjdIgby3HBzRmt0JhEqU+mGk9OahvJeZo5DM3C3oCfVhMGdiOC+4jx+w3oclUxzDHf
c3NHBEgfab/hoYK6wsyV9a0ah4ExDY0tU0oEUlgQvKyM1KlQBr/gg2CUKaIdI5SPnKnlwvOO
Cv0KeBGatt3mGDzMn4v9UeBYfbAi9BYESFU8xaljG8ywl6xtYYc/1ElAFlphykw7KdQqKy1z
aDNPV9LLPztDcfLcmakgp2L6/qteFlx0JcQp5zR54TFQlBkwvZKvdZZaygRQlfDp9/IKHOQY
OHXWkL3ygHNQK2GBTEVb5r8VeU4n4Kn7kjPeMwVjhK6j7XanF2dUfbQGaJuy2J3peHkeZYWZ
KbvU3V9MbuFLeDRALnHj9RoESLCztoL+bvGj3Nkh2IRuSk1QBypCtjwrzPySmZ1VgAmJQwyw
srHGF+EwS69VpU9S4/FPF2jk6M8GyIWvdXr+kwEwIbpJg+UzsQRAXTki4AzVVpjGHjSMB5fI
N+mwsiJtbTUhEL1cSIN2xrphVu39Mx0t80v8jQmpfOzwAyz8Ydn5PjP36W41gx/3vgQIYUD7
1f65Qy4ECFUS80HQ76idBAiNycLDcXBRSgQI17vcKUcVOlQECJxWFfIlCxMgBAiCz6xzCuwI
1QQIA58jZKVMAMoEEFoxFLZWzOop0sZ7J4ntH0kECHbSG23ZJ3qmBAiD/CXUdtbGPQQQrzcD
t5AOpaD/pmEnHhco+wQI5nFuIXyhbPoECNCvPFc72f+zBAifW6POunibTQQI9fYETXIR8eUE
GEbHNVpHssWgVHYmg8iYluTq6BiKlrcDEgQImOhwpoxurDUECN7HnaGkjRfCBAgNwsc9C1hx
aAQoNQZpVcbkN8bE6L/urETMieWLUtts2W4SYJAlkX+Lcjkw1IiZpTM2swQI3Cmp7CQYqr4E
CNgrfOjW3hD9BAhS++99zb2vvwQIwRQRelhvx/cEEC5oGkXI66aOA70ueiJbDTEEEE54fjhP
eDiNxGq5OMkwN9sEENXhVg+8E9tzwQ2vzgZL/YoECNywO8rM/cd1BBDq+wN3fgkjB1vTMHwF
HyR1BAi8XY/RE5iirwQI8wg2FpQ0pxsECH5LzZu04zQyBBhbql8QnhaztaW+EU2ER9XPv3e9
sfjAmL8ECPPRaYZfdLVQBBBbC0+8dhNmGoDbOtvKnY2BBDg8fK/VpjGe1yLbwCIiltUhDQ6p
Ie/L5f/JtEoG/pVTpWc1WLKmXrMWuJfvpMF0rrPu9HdJYCeFBwQo4ZszGPHw/JOxa4hmnw8T
yIXs3E98WfSEY5zZ7l9A7ZHKEm9zmutSGQQIc/FmKs6IpJQEEOFXFSeCeHOa39B5IErLfcsE
CNuf9bPyrGgJBEh4Z1b2OnGKMCN/dEXATXW5ZGamIympkcZd9MgDLSmstvxzcuJvClsopV/r
X/+45uXKjAmbY2OWIek8u/w4u7ZMZ3NZkZ/OkEgESET2IKSQqQi9zPMyetmH9RUfJWb9lS0O
nd6WSbqFD2bjoGttLhQcaAz/NUQuqIBl6rwD/c46X9wDbQjetJJLMliKK0to+dekWwRA5RXh
MSHSUHdoAF0zOQau519G7qQgX+NjXmhaeEKDwhPFvVbwV5fXUfBHDfSA2Y31ZcGPiEDMlFHt
i9nxijHO+gQQEVQYv9fwFezWUhdQQ5RdiQQIjXHCYZ88WSgEQN3sCYefFpmW34P1EnvzBVRj
gChM8oDOOi4iq7Qhu2KUUQgBEInvsP+gvsyLDlm9wY6GKJrNp6t5lxN1rlIbuJYESGDFnW+O
Pp7CX4O92XSdNw7l5+K+QJvwSb8xQCUpAy7uJ4+38XxWA/MGmJw5Wu3qsYT/yv6F7kRpEmxo
Ff5W1iZpx2zIEqbNHwQo8IimgUIcrxwSinEcgZ0wn8dXbYa5rMSQtuV11LS47gDymhl5wvA2
3QQI3NdUWhcvwBoEMJfGO0rZ8wXVTruUNNX6Xfk1LMVTGG8ipuFRsmtGzsPAHPf6XpFrZl0V
2NyqsxTd0QQIrDZww49oy/oAAAAAAAAAAAAA

[Cvs-syncmail] syncmail 1.20 -- cvs 1.11 --- pyton 1.5.2 --- ValueError: unpack list of wrong size

[Murat B. <mu...@ki...>]

Guys I get the following error.
I saw someone reported the same error as a bug.
Am I missing something ?
How are you guys running it ?

Thanks

Error:

cvs commit: Examining scripts
Checking in test;
/home/cvs/deployment/sms_mail/test,v  <--  test
new revision: 1.12; previous revision: 1.11
done
Mailing mu...@ki......
Generating notification message...
Generating notification message... done.
Mailing mu...@ki......
Generating notification message...
Traceback (innermost last):
  File "/usr/bin/syncmail", line 336, in ?
    main()
  File "/usr/bin/syncmail", line 329, in main
    blast_mail(subject, people, specs[1:], contextlines, fromhost)
  File "/usr/bin/syncmail", line 250, in blast_mail
    print calculate_diff(file, contextlines)
  File "/usr/bin/syncmail", line 145, in calculate_diff
    file, oldrev, newrev = string.split(filespec, ',')
ValueError: unpack list of wrong size

[Cvs-syncmail] syncmail syncmail,1.19,1.20

[Greg W. <gw...@us...>]

Update of /cvsroot/cvs-syncmail/syncmail
In directory usw-pr-cvs1:/tmp/cvs-serv18915

Modified Files:
	syncmail 
Log Message:
Change default value of MAILHOST to 'localhost'.  This seems to
be necessary to work with Python 2.2, because smtplib in 2.2 uses
the new socket.getaddrinfo() function, and that function doesn't
handle the empty string (at least not on Linux or FreeBSD).
Closes SF bug #571061.


Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/syncmail/syncmail,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -d -r1.19 -r1.20
--- syncmail	31 Jul 2002 11:45:31 -0000	1.19
+++ syncmail	5 Aug 2002 22:31:42 -0000	1.20
@@ -107,8 +107,8 @@
 
 from cStringIO import StringIO
 
-# Which SMTP server to do we connect to?  Empty string means localhost.
-MAILHOST = ''
+# Which SMTP server to do we connect to?
+MAILHOST = 'localhost'
 MAILPORT = 25
 
 # Diff trimming stuff

[Cvs-syncmail] CVSROOT syncmail,1.15,1.16

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

Update of /cvsroot/cvs-syncmail/CVSROOT
In directory usw-pr-cvs1:/tmp/cvs-serv10722

Modified Files:
	syncmail 
Log Message:
Test commit following update to the latest version.

Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/CVSROOT/syncmail,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -d -r1.15 -r1.16

[Cvs-syncmail] syncmail syncmail,1.16,1.17

[Greg W. <gw...@us...>]

Update of /cvsroot/cvs-syncmail/syncmail
In directory usw-pr-cvs1:/tmp/cvs-serv28544

Modified Files:
	syncmail 
Log Message:
Added -S/--subject-prefix option: lets caller supply a string that
is prepended to the subject line of the generated email message.


Index: syncmail
===================================================================
RCS file: /cvsroot/cvs-syncmail/syncmail/syncmail,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -d -r1.16 -r1.17
--- syncmail	23 May 2002 20:21:04 -0000	1.16
+++ syncmail	8 Jul 2002 19:28:52 -0000	1.17
@@ -47,6 +47,10 @@
     -u
         Produce a unified diff (smaller).
 
+    -S TEXT
+    --subject-prefix=TEXT
+        Preprend TEXT to the email subject line.
+
     --quiet / -q
         Don't print as much status to stdout.
 
@@ -256,14 +260,16 @@
 def main():
     try:
         opts, args = getopt.getopt(
-            sys.argv[1:], 'hC:cuqf:',
-            ['fromhost=', 'context=', 'cvsroot=', 'help', 'quiet'])
+            sys.argv[1:], 'hC:cuS:qf:',
+            ['fromhost=', 'context=', 'cvsroot=', 'subject-prefix=',
+             'help', 'quiet'])
     except getopt.error, msg:
         usage(1, msg)
 
     # parse the options
     contextlines = 2
     verbose = 1
+    subject_prefix = ""
     fromhost = None
     for opt, arg in opts:
         if opt in ('-h', '--help'):
@@ -277,6 +283,8 @@
                 contextlines = 2
         elif opt == '-u':
             contextlines = 0
+        elif opt in ('-S', '--subject-prefix'):
+            subject_prefix = arg
         elif opt in ('-q', '--quiet'):
             verbose = 0
         elif opt in ('-f', '--fromhost'):
@@ -288,7 +296,7 @@
     # $CVSROOT, followed by the list of files that are changing.
     if not args:
         usage(1, 'No CVS module specified')
-    subject = args[0]
+    subject = subject_prefix + args[0]
     specs = string.split(args[0])
     del args[0]
 

RE: [Cvs-syncmail] cvs commit error

[Sebastian U. <ud...@ha...>]

On Sat, 6 Jul 2002, S.D...@gm... (Stephan Diederich) wrote:
> Date: Sat, 6 Jul 2002 00:23:22 +0200
> To: <cvs...@li...>
> From: S.D...@gm... (Stephan Diederich)
> Subject: RE: [Cvs-syncmail] cvs commit error

[...]

> But i get that message, allthough i only send my mails to the
> sourceforge mailinglists

Why don't you just file a support request then ?


- Sebastian

RE: [Cvs-syncmail] cvs commit error

[Stephan D. <S.D...@gm...>]

Sebastian Ude wrote:

> On Sat, 6 Jul 2002, ja...@tu... (Alexander Zynevich) wrote:
> > Date: Sat, 6 Jul 2002 00:24:42 +0300
> > To: <ud...@ha...>, <cvs...@li...>
> > From: ja...@tu... (Alexander Zynevich)
> > Reply-To: ja...@tu... (Alexander Zynevich)
> > Subject: Re: [Cvs-syncmail] cvs commit error
> >
> > I recieve the same error .... it seems that source forge
> closes SMTP
> > service. May be they prohibit SMTP to external mail lists and
> > recipients like my mail ja...@tu... ... who knows :)
>
> They announced a long time ago that they plan to prohibit
> mail being send
> >from the CVS servers to non-sourceforge addresses (as they
> are already
> doing with the shell servers). It seems as if they finally
> made excactly that change.

But i get that message, allthough i only send my mails to the
sourceforge mailinglists

Stephan

Re: [Cvs-syncmail] cvs commit error

[Sebastian U. <ud...@ha...>]

On Sat, 6 Jul 2002, ja...@tu... (Alexander Zynevich) wrote:
> Date: Sat, 6 Jul 2002 00:24:42 +0300
> To: <ud...@ha...>, <cvs...@li...>
> From: ja...@tu... (Alexander Zynevich)
> Reply-To: ja...@tu... (Alexander Zynevich)
> Subject: Re: [Cvs-syncmail] cvs commit error
>
> I recieve the same error .... it seems that source forge closes SMTP
> service. May be they prohibit SMTP to external mail lists and recipients
> like my mail ja...@tu... ... who knows :)

They announced a long time ago that they plan to prohibit mail being send
from the CVS servers to non-sourceforge addresses (as they are already
doing with the shell servers). It seems as if they finally made excactly
that change.


- Sebastian

Re: [Cvs-syncmail] cvs commit error

[Alexander Z. <ja...@tu...>]

I recieve the same error .... it seems that source forge closes SMTP
service. May be they prohibit SMTP to external mail lists and recipients
like my mail ja...@tu... ... who knows :)
What for me I am trying to create mail list and send commit logs to it.
----- Original Message -----
From: "Sebastian Ude" <ud...@ha...>
To: <cvs...@li...>
Sent: Saturday, July 06, 2002 12:01 AM
Subject: Re: [Cvs-syncmail] cvs commit error


>
>
> On Fri, 5 Jul 2002, S.D...@gm... (Stephan Diederich) wrote:
> > Date: Fri, 5 Jul 2002 22:11:43 +0200
> > To: <cvs...@li...>
> > From: S.D...@gm... (Stephan Diederich)
> > Subject: [Cvs-syncmail] cvs commit error
> >
> > Hi all,
> >
> > since probably 4 days i get only an error message:
> >
> > Traceback (innermost last):
> >   File "/cvsroot/jais/CVSROOT/syncmail", line 322, in ?
> >     main()
> >   File "/cvsroot/jais/CVSROOT/syncmail", line 315, in main
> >     blast_mail(subject, people, specs[1:], contextlines, fromhost)
> >   File "/cvsroot/jais/CVSROOT/syncmail", line 221, in blast_mail
> >     conn.connect(MAILHOST, MAILPORT)
> >   File "/usr/lib/python1.5/smtplib.py", line 216, in connect
> >     self.sock.connect(host, port)
> > socket.error: (111, 'Connection refused')
> >
> > I really dont know what this means.
>
> It means excactly that - the STMP server you are using refuses the
> connection.
>
>
> - Sebastian
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Bringing you mounds of caffeinated joy.
> http://thinkgeek.com/sf
> _______________________________________________
> Cvs-syncmail-talk mailing list
> Cvs...@li...
> https://lists.sourceforge.net/lists/listinfo/cvs-syncmail-talk

Re: [Cvs-syncmail] cvs commit error

[Sebastian U. <ud...@ha...>]

On Fri, 5 Jul 2002, S.D...@gm... (Stephan Diederich) wrote:
> Date: Fri, 5 Jul 2002 22:11:43 +0200
> To: <cvs...@li...>
> From: S.D...@gm... (Stephan Diederich)
> Subject: [Cvs-syncmail] cvs commit error
>
> Hi all,
>
> since probably 4 days i get only an error message:
>
> Traceback (innermost last):
>   File "/cvsroot/jais/CVSROOT/syncmail", line 322, in ?
>     main()
>   File "/cvsroot/jais/CVSROOT/syncmail", line 315, in main
>     blast_mail(subject, people, specs[1:], contextlines, fromhost)
>   File "/cvsroot/jais/CVSROOT/syncmail", line 221, in blast_mail
>     conn.connect(MAILHOST, MAILPORT)
>   File "/usr/lib/python1.5/smtplib.py", line 216, in connect
>     self.sock.connect(host, port)
> socket.error: (111, 'Connection refused')
>
> I really dont know what this means.

It means excactly that - the STMP server you are using refuses the
connection.


- Sebastian

[Cvs-syncmail] cvs commit error

[Stephan D. <S.D...@gm...>]

Hi all,

since probably 4 days i get only an error message:

Traceback (innermost last):
  File "/cvsroot/jais/CVSROOT/syncmail", line 322, in ?
    main()
  File "/cvsroot/jais/CVSROOT/syncmail", line 315, in main
    blast_mail(subject, people, specs[1:], contextlines, fromhost)
  File "/cvsroot/jais/CVSROOT/syncmail", line 221, in blast_mail
    conn.connect(MAILHOST, MAILPORT)
  File "/usr/lib/python1.5/smtplib.py", line 216, in connect
    self.sock.connect(host, port)
socket.error: (111, 'Connection refused')

I really dont know what this means.
It worked fine since 4 months but now that.
I dont really know if your are the right ones to ask to, as it seems
more like a sourceforge problem...

Thx for your help

Stephan Diederich

Re: [Cvs-syncmail] cvs co syncmail failed.

[Sebastian U. <ud...@ha...>]

On Fri, 5 Jul 2002, ja...@tu... (Alexander Zynevich) wrote:
> Date: Fri, 5 Jul 2002 16:29:24 +0300
> To: <cvs...@li...>
> From: ja...@tu... (Alexander Zynevich)
> Subject: [Cvs-syncmail] cvs co syncmail failed.
>
> Hello I have enoggh expirience with CVS, but when I tried to checkout
> module syncmail, I failed, while module CVSROOT is checkouted well.
>
> [root@zynux mailman]# cvs co syncmail
> cvs server: cannot find module `syncmail' - ignored
> cvs [checkout aborted]: cannot expand modules

Hell - why are you doing things like that while being root ?

Concerning your problem, I bet you don't have set the CVSROOT environment
variable appropiately
(:pserver:ano...@cv...:/
cvsroot/cvs-syncmail). Alternatively, use the -d commandline option of CVS.

But I am sure you know that, because you have enough experience with CVS
..


- Sebastian

[Cvs-syncmail] cvs co syncmail failed.

[Alexander Z. <ja...@tu...>]

Hello I have enoggh expirience with CVS, but when I tried to checkout module
syncmail, I failed, while module CVSROOT is checkouted well.

[root@zynux mailman]# cvs co syncmail
cvs server: cannot find module `syncmail' - ignored
cvs [checkout aborted]: cannot expand modules


Any suggestions?

[Cvs-syncmail] (no subject)

[Alexander Z. <ja...@tu...>]

Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

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

Barry's patch included:
 > +From: %(name)s <%(address)s>

I commented:
 > This doesn't look right.  It doesn't deal with users who have commas
 > in their names (like me).

Sebastian Ude writes:
 > Do you have a better suggestion ?

How about:

 From: "%(name)s" <%(address)s>

(indented to hide from overly-cautious mail systems)

That form of quoting seems to work for just about anything in
practice.


  -Fred

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

Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

[Sebastian U. <ud...@ha...>]

On Fri, 24 May 2002, fd...@ac... (Fred L. Drake, Jr.) wrote:
> Date: Fri, 24 May 2002 12:11:20 -0400
> To: bw...@us...
> From: fd...@ac... (Fred L. Drake, Jr.)
> CC: cvs...@li...
> Subject: Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11
>
>
> bw...@us... writes:
>  > Sebastian Ude's patch to include the realname of the developer
> ....
>  > +        name = pwd.getpwuid(os.getuid())[4]
> ....
>  > +From: %(name)s <%(address)s>
>
> This doesn't look right.  It doesn't deal with users who have commas
> in their names (like me).

Mhm ...

Do you have a better suggestion ?


- Sebastian

Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

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

bw...@us... writes:
 > Sebastian Ude's patch to include the realname of the developer
...
 > +        name = pwd.getpwuid(os.getuid())[4]
...
 > +From: %(name)s <%(address)s>

This doesn't look right.  It doesn't deal with users who have commas
in their names (like me).


  -Fred

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

Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

[Sebastian U. <ud...@ha...>]

On Thu, 23 May 2002, ba...@zo... (Barry A. Warsaw) wrote:
> Date: Thu, 23 May 2002 16:27:35 -0400
> To: ud...@ha...
> From: ba...@zo... (Barry A. Warsaw)
> CC: cvs...@li...
> Subject: Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

[...]

> BTW, next time, please use the cvs-syncmail project's patch manager!
> It'll make sure stuff like this doesn't get buried in our inboxes.

Okay - no problem.


- Sebastian

Re: [Cvs-syncmail] CVSROOT syncmail,1.10,1.11

[<ba...@zo...>]

>>>>> "SU" == Sebastian Ude <ud...@ha...> writes:

    SU> I can guess why :).

Heh, I finally just installed it on my local cvs repository and
debugged it there (by spewing to a /tmp log file).

    >> Boy, debugging this thing in-situ /really/ sucks.

    SU> Yep.

Did I mention that debugging syncmail in-situ /really/ sucks?

:)

Anyway, what's in cvs now should be working.  Thanks for the patch.

-Barry

BTW, next time, please use the cvs-syncmail project's patch manager!
It'll make sure stuff like this doesn't get buried in our inboxes.