#7740 NORM 8.2.0 (: react gracefully to dbus services being restarted

[Zarro Boogs per Child]

#7740: react gracefully to dbus services being restarted
----------------------+-----------------------------------------------------
   Reporter:  mtd     |       Owner:  mtd                 
       Type:  defect  |      Status:  new                 
   Priority:  normal  |   Milestone:  8.2.0 (was Update.2)
  Component:  sugar   |     Version:  Git as of bug date  
 Resolution:          |    Keywords:  8.2.0:? r+          
Next_action:  code    |    Verified:  0                   
  Blockedby:          |    Blocking:  6995, 7690          
----------------------+-----------------------------------------------------
Changes (by marco):

  * keywords:  8.2.0:? r? => 8.2.0:? r+


Comment:

 Replying to [comment:4 mtd]:
 > Replying to [comment:3 marco]:
 > > Did you check that all the services you are changing are stateless (or
 cleanup themself correctly)?
 >
 > Yes, I checked that.  I worry that you're asking because you know of
 some subtle gotcha I've missed, but all three things I changed seem
 obviously stateless:
 >
 > - nmclient.NMClient._nm_obj is only used to setActiveDevice() - the rest
 of the code already follows name changes via a signal and doesn't care
 about that member variable

 I was suspicious of this one. For some reason I thought it was a complex
 service... (haven't see that code in a while.

 > I don't want this cleanup/change to detract from the patch, so if you're
 uneasy, I will a patch without the cleanup.

 r+ to the patch without the cleanup.

 Let's discuss the cleanup in other ticket, it seem like we should not land
 cleanups on 8.2 anyway at this time (I'm unsure myself about what's the
 best way to handle it, let me read your comments and think about it more
 carefully).

-- 
Ticket URL: <http://dev.laptop.org/ticket/7740#comment:6>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system