pidgin: f15338f5: Use CLIENT_WEBSITE rather than PURPLE_WEBSITE...

Evan Schoenberg evan.s at dreskin.net
Tue Jul 1 17:41:17 EDT 2008


On Jul 1, 2008, at 3:22 PM, Stu Tomlinson wrote:

> I wonder what happened to our commit emails?...
>
> <CIA-40> evands at pidgin.im im.pidgin.pidgin *  
> f15338f57c0920b685e5164d2f53e50a6d0f64b7 /libpurple/ (5 files in 5  
> dirs):
> <CIA-40> Use CLIENT_WEBSITE rather than PURPLE_WEBSITE in libpurple.  
> If the build
> <CIA-40> process doesn't #define CLIENT_WEBSITE, internal.h will  
> define it as
> <CIA-40> PURPLE_WEBSITE. No change for Pidgin or Finch, but this  
> will let other clients
> <CIA-40> such as Adium present their own site in error messages as  
> needed.
>
> A compile-time #define is not much use to clients that want to use
> libpurple as a real library instead of shipping their own copy, so I
> doubt this is really a good idea.
>
> Why not instead add a "website" member to the ui_info hashtable, and
> reference that in the prpls ? (like jabber does for name and version
> already)

That's an even better idea, Stu, thanks.  I'll do that as soon as I  
get a chance.  My only concern: Will this be an okay change to make  
for 2.5.0?

-Evan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://pidgin.im/pipermail/devel/attachments/20080701/427a707c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part
URL: <http://pidgin.im/pipermail/devel/attachments/20080701/427a707c/attachment.sig>


More information about the Devel mailing list