* Matthias Andree <ma(_at_)dt(_dot_)e-technik(_dot_)uni-dortmund(_dot_)de>
[2004-11-16 01:11 +0100]:
Joshua Crawford <mortarn_lists(_at_)yahoo(_dot_)com(_dot_)au> writes:
I tried sending this patch 4 days ago, before I'd subscribed to the list. I
received an automated message saying it was being held for review by the
moderator. As it hasn't appeared here yet, and 4 days is close enough to
forever in the online world :), perhaps it was lost in a spam filter or
something. My apologies to the moderator if he's just busy or
something.
G'day Joshua,
G'day Matthias.
long time no see! I'd indeed think ESR is "busy"...
Heh.
I take it that you've debugged the new PopDel.py because you weren't
satisfied with the prior version?
# Out of range error fixed
The prior version didn't work. The choice variable was initialised to the
wrong value, so the user was never prompted for the message to delete, it
just tried to delete the last one straight away. But, luckily, there was
another error that meant it actually tried to delete the message after the
last one, which obviously can't exist.
# Allow for all caps SUBJECT:
Many viruses have this, and they're the ones we want to delete.
# Display email address
Makes things clearer when checking multiple accounts.
# Don't prompt for save if no changes
# Don't clear the screen until we're displaying a menu
Cosmetic.
# Check for invalid choice
# Check all arguments exist
# Check for errors in POP
Error-checking.
# Return 1 on errors, 0 otherwise
Again for checking multiple accounts. E.g, I run
python PopDel.py pop.mail.yahoo.com.au joshua_gordon_crawford <password> && \
python PopDel.py pop.mail.yahoo.com.au mortarn_cash <password> && \
python PopDel.py pop.mail.yahoo.com.au mortarn_lists <password> && \
python PopDel.py pop.mail.yahoo.com.au mortarn_usenet <password>
whenever fetchmail warns me there's oversize messages on yahoo. If yahoo's
inaccessible (maybe I went offline in the meantime), it only trys to connect
once.
I'm asking because I wouldn't want to commit untested code to the SVN
repository.
I wouldn't submit untested code.
I figured one typo at superficial inspection, "n" missing in username:
+ if (len(sys.argv) < 4):
+ print 'Usage: ' + sys.argv[0] + ' pop3.host.name
userame password'
Oops.
Please post further code snippets to the fetchmail-devel list,
subscription info at http://developer.berlios.de/mail/?group_id=1824
Okay, will do. That one isn't linked from the catb.org webpage. It says to
"discuss fixes and improvements in fetchmail and help co-develop it" here.
--
Joshua 'bruce' Crawford ... http://www.geocities.com/mortarn
Don't worry if you're a kleptomaniac, you can always take something for it.
pgpdlBg3Cvh6W.pgp
Description: PGP signature