Traceback on checkdbs

Bug #265867 reported by Stdivjak
2
Affects Status Importance Assigned to Milestone
GNU Mailman
Fix Released
Medium
Unassigned

Bug Description

The checkdbs script of mailman 2.1.1 quits with a
traceback:

[mailman@styx cron]$ ./checkdbs
Traceback (most recent call last):
  File "./checkdbs", line 136, in ?
    main()
  File "./checkdbs", line 80, in main
    text += '\n' + pending_requests(mlist)
  File "./checkdbs", line 123, in pending_requests
    text = NL.join(pending)
UnicodeError: ASCII decoding error: ordinal not in
range(128)

I tried reverting to mailman 2.1, and everything works
fine now.

[http://sourceforge.net/tracker/index.php?func=detail&aid=683833&group_id=103&atid=100103]

Revision history for this message
Tokio Kikuchi (tkikuchi) wrote :

Here is a patch. Please try and feedback.

Revision history for this message
Tokio Kikuchi (tkikuchi) wrote :

increment priority because someone else asked this bug.
until merged into CVS. ;-)

Revision history for this message
Stdivjak (stdivjak) wrote :

[mailman@styx cron]# ./checkdbs
Traceback (most recent call last):
  File "./checkdbs", line 138, in ?
    main()
  File "./checkdbs", line 80, in main
    text += '\n' + pending_requests(mlist)
  File "./checkdbs", line 105, in pending_requests
    fullname = fullname.encode(lcset, 'replace')
NameError: global name 'lcset' is not defined

Revision history for this message
Tokio Kikuchi (tkikuchi) wrote :

stdivjak> [mailman@styx cron]# ./checkdbs
you should type
python -S /path/to/checkdbs

Revision history for this message
Stdivjak (stdivjak) wrote :

Unfortunately, there's no difference :-( ...

[mailman@styx mailman]$ python -
S /home/mailman/cron/checkdbs
Traceback (most recent call last):
  File "/home/mailman/cron/checkdbs", line 138, in ?
    main()
  File "/home/mailman/cron/checkdbs", line 80, in main
    text += '\n' + pending_requests(mlist)
  File "/home/mailman/cron/checkdbs", line 105, in
pending_requests
    fullname = fullname.encode(lcset, 'replace')
NameError: global name 'lcset' is not defined

Revision history for this message
Tokio Kikuchi (tkikuchi) wrote :

OK, somehow ^L has disappeared during genrating diff from
cvs. I am uploading new patch using non-cvs diff.
(You might be able to report error in patch command execution)

Revision history for this message
Stdivjak (stdivjak) wrote :

Patching worked without any errors, and so does the
checkdbs script now. Well done! :-)

Revision history for this message
Barry Warsaw (barry) wrote :

I believe this has been fixed in a slightly different way
for 2.1.1 or 2.1.2 (cvs).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.