I have been having this problem for the past 2 hours, came to the same
discovery, and looked through all the branches
on http://growl.info/hg/growl/ to make sure I wasn't somehow on a broken
version...
My question is... How has this bug never been fixed back from 2009 where
you were experiencing it? And, how has this worked for anyone in the first
place if they didn't explicitly set defaultNotifications to a list of
indexes (if they didn't actually try to fix the source themselves)?
On Thursday, June 18, 2009 1:12:24 AM UTC-7, Guy Bolton King wrote:
>
> Hi,
>
> Sorry to raise this again: I've already sent this patch in for 1.1.4,
> and I note the problem persists in 1.1.5: am I calling GrowlNotifier
> incorrectly, or is this a bug that the patch fixes?
>
> Here's my original post from 15th April:
>
> It's possible I'm calling GrowlNotifier() the wrong way, however,
> this:
> from Growl import GrowlNotifier
> growl = GrowlNotifier(applicationName='redmine-to-omnifocus',
> notifications=['changed', 'completed'],
> hostname='localhost',
> password='')
> growl.register()
> ...fails at the growl.register() call with the following stacktrace:
> [...]
> File "Growl.py", line 102, in encodeRegistration
> struct.error: required argument is not an integer
> The attached patch appears to fix things.
> Regards,
> Guy.
> ---snip---
> --- Growl.py.orig 2009-04-15 10:07:01.000000000 +0100
> +++ Growl.py 2009-04-15 10:07:05.000000000 +0100
> @@ -99,6 +99,8 @@
> data += struct.pack("!H", len(encoded))
> data += encoded
> for i in defaultNotifications:
> + if type(i) != int:
> + i = notifications.index(i)
> data += struct.pack("B", i)
> return self.encodePassword(data)
>
--
You received this message because you are subscribed to the Google Groups
"Growl Discuss" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/growldiscuss/-/2EcUoyMKW0IJ.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/growldiscuss?hl=en.
