patch -p1 <mailer.diff

cat mailer.diff | patch -p1

id0 = {'background_color': '1E1E1E',
 'background_gradient_cutpoint': 40,
 'background_opacity': 70,
 'caption': 'jayjech@gmail.com',
 'caption_color': 'FFFFFF',
 'caption_font': 'Vera',
 'caption_font_size': 10,
 'delay': 300,
 'host': 'pop.gmail.com',
 'icon_height': 48,
 'icon_width': 48,
 'icons': ['error.png', 'noemail.png', 'email.png'],
 'imap': False,
 'message_color': None,
 'message_font': None,
 'message_font_size': None,
 'messages': ['connection error',
              'no new message',
              '1 new message',
              'new messages'],
 'messages_prepend': True,
 'program_on_click': 'mozilla http://gmail.google.com',
 'program_on_new': 'snd.sh',
 'ssl': True,
 'text_padding': 10,
 'user_name': 'jayjech',
 'user_password': 'XXX_XXX_XXX'}

print self.config['imap'],self.config['ssl'],self.config['user_name'],self.config['host']
if self.config['imap']:
...

killall -9 adesklets python
ADESKLETS_ID=0 python mailer.py

adesklets

./mailer.py
Traceback (most recent call last):
  File "./mailer.py", line 391, in ?
    Events(dirname(__file__)).pause()
  File "./mailer.py", line 199, in __init__
    adesklets.Events_handler.__init__(self)
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 157, in __init__
    self.ready()
  File "./mailer.py", line 209, in ready
    join(self.basedir,'config.txt'))
  File "./mailer.py", line 143, in __init__
    adesklets.ConfigFile.__init__(self,id,filename)
  File "/usr/lib/python2.4/site-packages/adesklets/configfile.py", line 159, in __init__
    self._load_and_save()
  File "/usr/lib/python2.4/site-packages/adesklets/configfile.py", line 202, in _load_and_save
    all= ConfigImport.load(buf[:])
  File "/usr/lib/python2.4/site-packages/adesklets/configfile.py", line 41, in __call__
    return dict(self._group(
  File "/usr/lib/python2.4/compiler/transformer.py", line 52, in parse
    return Transformer().parsesuite(buf)
  File "/usr/lib/python2.4/compiler/transformer.py", line 129, in parsesuite
    return self.transform(parser.suite(text))
  File "<string>", line 71
    'icon_height': 48,
                 ^

mv config.txt{,.bak}

Traceback (most recent call last):
  File "./mailer.py", line 391, in ?
    Events(dirname(__file__)).pause()
  File "./mailer.py", line 199, in __init__
    adesklets.Events_handler.__init__(self)
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 159, in __init__
    self._alarm()
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 296, in _alarm
    timeout=self.alarm()
  File "./mailer.py", line 240, in alarm
    self._display()
  File "./mailer.py", line 304, in _display
    adesklets.context_set_font(self._caption_font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands.py", line 383, in context_set_font
    print >> comm,   'context_set_font',   strings.String(font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands_handler.py", line 45, in write
    print >> self.__comm, self.__command
  File "/usr/lib/python2.4/site-packages/adesklets/communicator.py", line 79, in write
    raise ADESKLETSError(1)
adesklets.error_handler.ADESKLETSError: adesklets process exited -

id0 = {'background_color': '1E1E1E',
 'background_gradient_cutpoint': 40,
 'background_opacity': 100,
 'caption': '',
 'caption_color': 'FFFFFF',
 'caption_font': 'Vera',
 'caption_font_size': 10,
 'delay': 300,
 'host': 'pop.gmail.com',
 'icon_height': 48,
 'icon_width': 48,
 'icons': ['error.png', 'noemail.png', 'email.png'],
 'imap': False,
 'message_color': None,
 'message_font': None,
 'message_font_size': None,
 'messages': ['connection error',
              'no new message',
              '1 new message',
              'new messages'],
 'messages_prepend': True,
 'program_on_click': None,
 'program_on_new': 'beep.sh',
 'ssl': True,
 'text_padding': 5,
 'user_name': 'joshua.lowell@gmail.com',
 'user_password': 'xxxxxxx'}

Traceback (most recent call last):
  File "mailer.py", line 391, in ?
    Events(dirname(__file__)).pause()
  File "mailer.py", line 199, in __init__
    adesklets.Events_handler.__init__(self)
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 159, in __init__
    self._alarm()
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 296, in _alarm
    timeout=self.alarm()
  File "mailer.py", line 240, in alarm
    self._display()
  File "mailer.py", line 304, in _display
    adesklets.context_set_font(self._caption_font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands.py", line 383, in context_set_font
    print >> comm,   'context_set_font',   strings.String(font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands_handler.py", line 45, in write
    print >> self.__comm, self.__command
  File "/usr/lib/python2.4/site-packages/adesklets/communicator.py", line 79, in write
    raise ADESKLETSError(1)
adesklets.error_handler.ADESKLETSError: adesklets process exited -

Traceback (most recent call last):
  File "./mailer.py", line 391, in ?
    Events(dirname(__file__)).pause()
  File "./mailer.py", line 199, in __init__
    adesklets.Events_handler.__init__(self)
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 159, in __init__
    self._alarm()
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 296, in _alarm
    timeout=self.alarm()
  File "./mailer.py", line 240, in alarm
    self._display()
  File "./mailer.py", line 304, in _display
    adesklets.context_set_font(self._caption_font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands.py", line 383, in context_set_font
    print >> comm,   'context_set_font',   strings.String(font)
  File "/usr/lib/python2.4/site-packages/adesklets/commands_handler.py", line 45, in write
    print >> self.__comm, self.__command
  File "/usr/lib/python2.4/site-packages/adesklets/communicator.py", line 79, in write
    raise ADESKLETSError(1)
adesklets.error_handler.ADESKLETSError: adesklets process exited -

[/home/joshua/.desklets/mailer-0.0.3/mailer.py]
id=0 screen=0 x=4 y=6

[/home/joshua/.desklets/Calendar-0.5.0/Calendar.py]
id=2 screen=0 x=76 y=806

pocketfuzzies ~ $ cd desklets/mailer-0.0.4
pocketfuzzies mailer-0.0.4 $ ls
COPYING      README   config.txt       icons      mailer_error.diff
GNUmakefile  beep.sh  config.txt.save  mailer.py
pocketfuzzies mailer-0.0.4 $ patch < mailer_error.diff mailer.py
patching file mailer.py
pocketfuzzies mailer-0.0.4 $ killall -9 adesklets python
pocketfuzzies mailer-0.0.4 $ ADESKLETS_ID=0 python mailer.py 

pocketfuzzies mailer-0.0.4 $ ADESKLETS_ID=0 python mailer.py 
Traceback (most recent call last):
  File "mailer.py", line 417, in ?
    Events(dirname(__file__)).pause()
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 228, in pause
    posix_signal.pause()
KeyboardInterrupt

pocketfuzzies mailer-0.0.4 $ sleep 1 && ADESKLETS_ID=0 python mailer.py 
Traceback (most recent call last):
  File "mailer.py", line 417, in ?
    Events(dirname(__file__)).pause()
  File "mailer.py", line 225, in __init__
    adesklets.Events_handler.__init__(self)
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 159, in __init__
    self._alarm()
  File "/usr/lib/python2.4/site-packages/adesklets/events_handler.py", line 296, in _alarm
    timeout=self.alarm()
  File "mailer.py", line 265, in alarm
    mail = self.mailer()
  File "mailer.py", line 173, in __call__
    s = self.imaplib.__dict__['IMAP4'+['','_SSL']
  File "/usr/lib/python2.4/imaplib.py", line 160, in __init__
    self.open(host, port)
  File "/usr/lib/python2.4/imaplib.py", line 228, in open
    self.sock.connect((host, port))
  File "<string>", line 1, in connect
socket.gaierror: (-2, 'Name or service not known')
pocketfuzzies mailer-0.0.4 $ nano config.txt 
pocketfuzzies mailer-0.0.4 $ echo OK && killall -q -9 adesklets python
OK
pocketfuzzies mailer-0.0.4 $ sleep 1 && ADESKLETS_ID=0 python mailer.py 

ocketfuzzies mailer-0.0.4 $ patch -s < mailer_error_bis.diff mailer.py
pocketfuzzies mailer-0.0.4 $ echo OK && killall -q -9 adesklets python
OK
pocketfuzzies mailer-0.0.4 $ sleep 1 && ADESKLETS_ID=0 python mailer.py 
Get information. Method is imap4
Imap. SSL is False, host is voidvoid.org
Object creation... Ouch! socket.gaierror (-2, 'Name or service not known')
Computed size is -1, now returning.

pocketfuzzies mailer-0.0.4 $ nano config.txt 
pocketfuzzies mailer-0.0.4 $ sleep 1 && ADESKLETS_ID=0 python mailer.py 
Get information. Method is imap4
Imap. SSL is True, host is mymail.rit.edu
Object creation... Created: <imaplib.IMAP4_SSL instance at 0xb7d21c6c>
Login... logged.
Selecting... selected.
Computing size... Size is 2
Login out... ok.
Computed size is 2, now returning.

patch -p1 <mailer_error2.diff mailer.py

--- mailer.py.orig  2005-06-12 16:36:03.000000000 -0300
+++ mailer.py   2005-06-18 17:23:06.000000000 -0300
@@ -47,7 +47,7 @@
 from itertools import count
 import traceback
 import re
-from os import getenv, spawnlp, P_NOWAIT
+from os import getenv, spawnlp, P_NOWAIT, stat, utime
 from os.path import join, dirname

 #-------------------------------------------------------------------------------
@@ -144,7 +144,7 @@
# I like to use the 'beep' program from Debian package 'beep' to play a 
# customized tone. For this purpose I had to modify the line calling the
# program (see later). With this modification, 'beep.sh' won't work unless
# it is specified using full pathname.
                     'icon_height': 48,
                     'delay':300,
                     'program_on_click':None,
-                    'program_on_new':'beep.sh'}
+                    'program_on_new':None}

     def __init__(self,id,filename):
         adesklets.ConfigFile.__init__(self,id,filename)
@@ -168,6 +168,9 @@
# Declare some new variables:
         self.config=config
         self.last_size=-1
         self.size=-1
+        self.last_w = 0
+        self.mbox_last_mtime = 0
+        self.mbox_last_size = 0

     def __call__(self):
         self.last_size=self.size
@@ -198,10 +201,22 @@
# This is the mdification for the mbox method. Two features have been added:
# 1) Check for new messages only if the file has been modified, and 
# 2) Restore modification time after check, in order to not confuse mutt, and
#    maybe other mail clients.
             #
             if self.config['method'] == 'mbox':
                 size=0
-                for m in self.mailbox.PortableUnixMailbox(
-                    file(getenv('MAIL',self.config['mailspool']))):
+                mbox_path = getenv('MAIL',self.config['mailspool'])
+                # Get mbox inode properties:
+                s = stat(mbox_path)
+                # If it hasn't changed since last check, don't bother:
+                if s.st_size == self.mbox_last_size and s.st_mtime == self.mbox_last_mtime:
+                    return self.last_size
+                # It has changed: check again.
+                for m in self.mailbox.PortableUnixMailbox(file(mbox_path)):
                     if m.get('status','N').find('N') != -1:
                        size += 1
+                # Restore inode times: 
+                utime(mbox_path, (s.st_atime, s.st_mtime))
+                # Remember size & time: 
+                self.mbox_last_size = s.st_size
+                self.mbox_last_mtime = s.st_mtime
+       
         except:
             # Exception handling: output a significant printout
             #
@@ -269,10 +284,11 @@
# Use full pathname for program_on_new, ie, don't prepend basedir.
         self._display()
         if mail > self.mailer.last_size and mail>0:
             if self.config['program_on_new']:
-                self._execute(join(self.basedir,
-                                   self.config['program_on_new']))
+                self._execute(self.config['program_on_new'])
         self.unblock()
# Allow for delay < 60 for mbox method.
-        return max(self.config['delay'],60)
+        if self.config['method'] == 'mbox':
+            return self.config['delay']
+        return max(self.config['delay'], 60)

     def menu_fire(self, delayed, menu_id, item):
         if item=='Configure':@@ -282,8 +298,7 @@
# Allow to execute program_on_click even when there are no new messages.
# (Sometimes I want to open mutt to check some old mail).
                               join(self.basedir,'config.txt'))

     def button_press(self, delayed, x, y, button):
-        if self.mailer.size>0:
-            self._execute(self.config['program_on_click'])
+        self._execute(self.config['program_on_click'])

     def background_grab(self,delayed):
         if self.config['background_color']:
@@ -337,8 +352,10 @@
# Never draw using a smaller window width. This is because I don't like to 
# have my desklets changing sizes. The window width will only be changed when
# more space is needed.

         # Compute the window dimensions
         #
-        w=self.config['icon_width']+2*self.config['text_padding']+\
-           max(caption_dim[0],message_dim[0])
+        w=max(self.mailer.last_w, 
+           self.config['icon_width']+2*self.config['text_padding']+
+           max(caption_dim[0],message_dim[0]))
+        self.mailer.last_w = w
         h=max(self.config['icon_height'],caption_dim[1]+message_dim[1]+
               2*self.config['text_padding'])

--- mailer.py.orig  2005-06-12 16:36:03.000000000 -0300
+++ mailer.py   2005-06-18 17:23:06.000000000 -0300
@@ -47,7 +47,7 @@
# > The new import statements probably belongs to 
# > hte Mailer class, just for coding style consistency.
#
# Hmm... I don't get it. But yet, I am no Python programmer, 
# so I won't argue ;)
#
-from os import getenv, spawnlp, P_NOWAIT
+from os import getenv, spawnlp, P_NOWAIT, stat, utime
 #-------------------------------------------------------------------------------
@@ -144,7 +144,7 @@
# >> I like to use the 'beep' program from Debian package 'beep' to play a
# >> customized tone. 
#
# > Nothing wrong there, this is why there is a 'program_on_new' parameter.
# > But there is no need to break current behavior: look at the documentation of
# > os.path.join(). Specifying the absolute path name to Debian's beep will do the job
# > fine too.
#
# I confess! I didn't read the docs! :P
#
-                    'program_on_new':'beep.sh'}
+                    'program_on_new':None}
@@ -198,10 +201,22 @@
# >> This is the mdification for the mbox method. Two features have been added:
# >> 1) Check for new messages only if the file has been modified, and
# >> 2) Restore modification time after check, in order to not confuse mutt, and
# >>   maybe other mail clients.
#
# > Mmh. I wasn't aware current code would confuse mutt (isn't it a somewhat bad design, 
# > or is it just me? Isn't what ctime is for?). I take your word for it, but I'd like to have 
# > confirmation from someone else that this work, as I do not use mbox+mutt on a regular
# > basis (what about access race conditions?). Another small coding style issue:
# > I used a single return point for the __call__ method.
#
# Two things to comment here. The first 'new feature' is independent of
# the 'mutt confusing' problem. It just saves CPU cycles. And about this
# mutt problem, It may be a bad design, but this is the way it is :-\
# Check, for instance: http://lists.debian.org/debian-user/2000/05/msg02838.html
#
@@ -269,10 +284,11 @@
# >> Allow for delay < 60 for mbox method.
#
# > Why not, this is your CPU cycles. ;-) This was just instroduced to avoid
# > people from being banned from their online services. You could have written:
# > 
# > return (max(self.config['delay'],60),self.config['delay'])[self.config['method'] == 'mbox']
#
# Come on! Is this really the python coding style? It makes me want
# to go back to C :) .
#
-        return max(self.config['delay'],60)
+        if self.config['method'] == 'mbox':
+            return self.config['delay']
+        return max(self.config['delay'], 60)

id0 = {'background_color': 'DDDDDD',
 'background_gradient_cutpoint': 0,
 'background_opacity': 0,
 'caption': 'web.de',
 'caption_color': 'd0d0ff',
 'caption_font': 'Vera',
 'caption_font_size': 10,
 'click_anytime': False,
 'delay': 300,
 'host': 'imap.web.de',
 'icon_height': 48,
 'icon_width': 48,
 'icons': ['error.png', 'noemail.png', 'email.png'],
 'mailspool': '/dev/null',
 'message_color': False,
 'message_font': None,
 'message_font_size': None,
 'messages': ['Fehler', '0 Mails', '1 Mail', 'Mails'],
 'messages_prepend': True,
 'method': 'imap4',
 'program_on_click': '/opt/thunderbird/thunderbird',
 'program_on_new': 'beep.sh',
 'ssl': False,
 'text_padding': 5,
 'user_name': 'myuser',
 'user_password': 'mypassword',
 'window_never_shrink': False}

id1 = {'background_color': 'DDDDDD',
 'background_gradient_cutpoint': 0,
 'background_opacity': 0,
 'caption': 'compuserve.de',
 'caption_color': 'd0d0ff',
 'caption_font': 'Vera',
 'caption_font_size': 10,
 'click_anytime': False,
 'delay': 300,
 'host': 'pop.compuserve.de',
 'icon_height': 48,
 'icon_width': 48,
 'icons': ['error.png', 'noemail.png', 'email.png'],
 'mailspool': '/dev/null',
 'message_color': False,
 'message_font': None,
 'message_font_size': None,
 'messages': ['Fehler', '0 Mails', '1 Mail', 'Mails'],
 'messages_prepend': True,
 'method': 'pop3',
 'program_on_click': '/opt/thunderbird/thunderbird',
 'program_on_new': 'beep.sh',
 'ssl': False,
 'text_padding': 5,
 'user_name': 'myuser',
 'user_password': 'mypassword',
 'window_never_shrink': False}

id2 = {'background_color': 'DDDDDD',
 'background_gradient_cutpoint': 0,
 'background_opacity': 0,
 'caption': 'nyboria.de',
 'caption_color': 'd0d0ff',
 'caption_font': 'Vera',
 'caption_font_size': 10,
 'click_anytime': False,
 'delay': 300,
 'host': 'imap.1und1.de',
 'icon_height': 48,
 'icon_width': 48,
 'icons': ['error.png', 'noemail.png', 'email.png'],
 'mailspool': '/dev/null',
 'message_color': False,
 'message_font': None,
 'message_font_size': None,
 'messages': ['Fehler', '0 Mails', '1 Mail', 'Mails'],
 'messages_prepend': True,
 'method': 'imap4',
 'program_on_click': '/opt/thunderbird/thunderbird',
 'program_on_new': 'beep.sh',
 'ssl': False,
 'text_padding': 5,
 'user_name': 'myuser',
 'user_password': 'mypassword',
 'window_never_shrink': False}

./mailer.py &