Re: pidgin: f7d26d95: Add a purple_account_get_name_for_displa...

View: New views
2 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: pidgin: f7d26d95: Add a purple_account_get_name_for_displa...

by Daniel Atallah-2 :: Rate this Message:

Reply (Restricted by the Administrator) | Reply to Author | View Threaded | Show Only this Message

On Tue, Sep 8, 2009 at 08:52, <deryni@...> wrote:

> -----------------------------------------------------------------
> Revision: f7d26d95395d5013710b12cfdfcf131aa1033e0c
> Ancestor: 7648401c87cc8f2913749424f6b22c79e3d1a345
> Author: deryni@...
> Date: 2009-08-15T05:10:07
> Branch: im.pidgin.pidgin
> URL: http://d.pidgin.im/viewmtn/revision/info/f7d26d95395d5013710b12cfdfcf131aa1033e0c
>
> Modified files:
>        ChangeLog.API libpurple/account.c libpurple/account.h
>
> ChangeLog:
>
> Add a purple_account_get_name_for_display function (I'm not a huge fan of that
> name but didn't want to use get_display_name as that means something else for
> connections).
>
> This wants to be used in places where we need to display an identifier for the
> account to the user and honor the appropriate aliases/etc.
>
> Refs #8391


Shouldn't this commit have gone into im.pidgin.next.minor (and the
2.7.0 changelog)?

-D

_______________________________________________
Devel mailing list
Devel@...
http://pidgin.im/cgi-bin/mailman/listinfo/devel

Re: pidgin: f7d26d95: Add a purple_account_get_name_for_displa...

by Etan Reisner-2 :: Rate this Message:

Reply (Restricted by the Administrator) | Reply to Author | View Threaded | Show Only this Message

On Wed, Sep 09, 2009 at 03:01:30PM -0400, Daniel Atallah wrote:

> On Tue, Sep 8, 2009 at 08:52, <deryni@...> wrote:
> > -----------------------------------------------------------------
> > Revision: f7d26d95395d5013710b12cfdfcf131aa1033e0c
> > Ancestor: 7648401c87cc8f2913749424f6b22c79e3d1a345
> > Author: deryni@...
> > Date: 2009-08-15T05:10:07
> > Branch: im.pidgin.pidgin
> > URL: http://d.pidgin.im/viewmtn/revision/info/f7d26d95395d5013710b12cfdfcf131aa1033e0c
> >
> > Modified files:
> >        ChangeLog.API libpurple/account.c libpurple/account.h
> >
> > ChangeLog:
> >
> > Add a purple_account_get_name_for_display function (I'm not a huge fan of that
> > name but didn't want to use get_display_name as that means something else for
> > connections).
> >
> > This wants to be used in places where we need to display an identifier for the
> > account to the user and honor the appropriate aliases/etc.
> >
> > Refs #8391
>
>
> Shouldn't this commit have gone into im.pidgin.next.minor (and the
> 2.7.0 changelog)?
>
> -D

Yes, I messed that up. Mark fixed the @since tag for me already. There was
some talk in devel@... about whether we wanted to merge ipp.next.minor
already or wait for a while to have more 2.6.x releases but I don't know
if we decided on anything. I was going to fix my ChangeLog.API entry the
other night but figured I'd find out what the story with merging is so I
don't give myself extra work (by adding an extra revision I need to move
from ipp).

So what do we think, merge soon or hold off?

    -Etan

_______________________________________________
Devel mailing list
Devel@...
http://pidgin.im/cgi-bin/mailman/listinfo/devel