← Back to team overview

ayatana-commits team mailing list archive

Re: [Merge] lp:~ted/dbusmenu/lightweight-proxy into lp:dbusmenu

 

On Thu, 2010-01-07 at 08:45 +0000, David Barth wrote:
> The code looks fine, but I have a hard time seeing how that 
> improves things in the case you're describing. A unit test 
> would actually make things clearer.

If you look at r51, now that test doesn't need the half second timeout
that it did had.  The reason is that dbusmenu now will pick up the
server coming on the bus.

The merge has two fixes.  One is that the changing name_owner function
was broken.  The second is that the proxies watching the dbus
connections weren't being created in the case of the server not being
there like they should have been.

-- 
https://code.launchpad.net/~ted/dbusmenu/lightweight-proxy/+merge/16947
Your team ayatana-commits is subscribed to branch lp:dbusmenu.



References