GrowlSafari patch

[Ben]

I'm resending a link to this patch, which resurrects GrowlSafari on
Leopard/Safari 3. I've been using it for a month or so & it works
great for me. [Google's web interface seems to mangle patches, so I
won't paste it here.]

http://willmore.eu/growlsafari.patch

Cheers,

Ben

[The Karl Adam]

Another avenue we could go with for Growl Safari is writing a WebKit
plugin and doing the method patching from there if the host app is
Safari. It's still the same icky injection methods but moved into a
place that's expected to be loaded and is a real plugin. Bonus is if
we ever want to actually support websites growling like growl for
windows has added we can implement it in the same plugin.

_Karl

[Clint Hime]

I still have that .tar you gave me awhile back, but I still never got
it to get up and running, but I do have other input managers running
like safariblock, inquisitor, and safariadblock. do you think those
or other inputmanagers could be stopping your growlsafari workaround?

[Ben]

Hi Clint,

On May 12, 6:19 am, Clint Hime <clinth...@gmail.com> wrote:
> I still have that .tar you gave me awhile back, but I still never got
> it to get up and running, but I do have other input managers running
> like safariblock, inquisitor, and safariadblock. do you think those
> or other inputmanagers could be stopping your growlsafari workaround?

The changes I made to GrowlSafari are pretty trivial, and unlikely to
generate new conflicts with those other programs. Also, I just tried
installing Inquisitor, SafariBlock and Safari AdBlock, and GrowlSafari
works fine for me with all three installed (though I only tested the
Download Completed notifications). So, I'm not sure where your problem
is coming from.

It would be worth running Safari from the Terminal (by typing /
Applications/Safari.app/Contents/MacOS/Safari) and seeing whether any
messages relating to GrowlSafari come up. Also, going through a
process of elimination with other extras (you mentioned an
inputmanager enabler; that could be problematic) should help. Also,
are some notifications are working, and others broken, or all broken?

Just to be sure we're on the same page, the most recent version I have
is at http://willmore.eu/GrowlSafari.zip. To install:

1 Unzip the zip file (Safari might do it for you).
2 Copy the resulting GrowlSafari to /Library/InputManagers/ (you may
need to create the InputManagers directory first).
3 Then, open a Terminal window and type:
sudo chown -R root /Library/InputManagers/
Enter your login password when requested.

Cheers,

Ben

[Ben]

On May 12, 10:43 am, Ben <bwillm...@gmail.com> wrote:
>
> 3 Then, open a Terminal window and type:
> sudo chown -R root /Library/InputManagers/
> Enter your login password when requested.

I can't remember if this matters, but this should read:
sudo chown -R root:admin /Library/InputManagers/

Ben

[Clint Hime]

Hey, you got it working again, the problem is definitely SIMBL i would
say, which is a shame because so many other safari plugin's use
SIMBL. Also, my hardware growler shows the volume mounted
notification before the growlsafari notification says the download is
complete. the compression status and disk image status are faster,
but not the download complete status.

[Ben]

On May 12, 5:02 pm, Clint Hime <clinth...@gmail.com> wrote:
> Hey, you got it working again, the problem is definitely SIMBL i would
> say, which is a shame because so many other safari plugin's use

I just tested, and GrowlSafari works fine for me even with the latest
SIMBL installed.

> SIMBL. Also, my hardware growler shows the volume mounted
> notification before the growlsafari notification says the download is
> complete. the compression status and disk image status are faster,
> but not the download complete status.

As for the late 'Download Complete' notification, that is probably a
bug -- there are a whole load of events on which GrowlSafari might
notify, and I probably just chose the wrong one. I'll look into it.

Ben

[Evan Schoenberg]

Ben,

Great work!  I've reviewed this patch and committed it for inclusion in Growl 1.1.3 as it is a vital bugfix for GrowlSafari ([4835]).   I also fixed the GrowlSafari installer to use the root:admin ownership settings needed for 10.5 to let the input manager run ([4836]).

Thanks :)

Cheers,

Evan