evan.s at dreskin.net
Mon Jan 5 23:12:47 UTC 2009
On Jan 5, 2009, at 3:25 PM, Eion Robb wrote:
> Is it worth giving the libpurple Facebook plugin a shot, even as just
> a seperate plugin -- similar to how you guys gave the msn-pecan a go?
>> From what I'm told, there are a lot of issues/feature requests on
> bug tracker that could be closed just by switching over.
> I completely agree that the Adium plugin code is a lot more elegant,
> but the libpurple plugin has had a lot of TLC by Mark Doliner and
> Casey Ho and is on-par with a lot of the other protocols now (If you
> consider that a good thing ;) ).
> It might make more sense to consolidate our efforts instead of working
> on two seperate protocol implementations... both of our sets of users
> would probably benefit from the merge.
> Anyway, that's my thoughts. I'm happy to work with you guys in making
> an AdiumLibpurplePlugin wrapper if you like?
I think we should consolidate efforts. The only reason the Adium code
is more elegant is because it's written in Cocoa and is benefitting
from most of the low-level work being taken care of automatically by
NSConnection and WebView.... which are also turning out to be a
problem on some unclear level.
An AdiumLibpurplePlugin for it would be a solid way to go from here...
I think that'd be great, personally. We'd need to take a look at use
'in the field' in a beta to make sure that we're not gaining more
problems than we're fixing in the switch.... but my suspicion is that
it'd be well worth it, as our #1 issue with the Adium FB plugin is
definitely the shared cookie jar between Safari and Adium.
I've checked out the current pidgin-facebook source but have found it
doesn't compile on OS X yet. Making a Makefile.am for it would let
the automake tools do their thing - probably a lot easier than
hardcoding the settings for building a universal binary.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 194 bytes
Desc: This is a digitally signed message part
More information about the Adium-devl