#7480 BLOC 8.2.0 (: Need to 'reset' the network configurations - short term fix

Zarro Boogs per Child bugtracker at laptop.org
Wed Aug 13 16:52:50 EDT 2008


#7480: Need to 'reset' the network configurations - short term fix
------------------------+---------------------------------------------------
   Reporter:  kimquirk  |       Owner:  erikg                  
       Type:  defect    |      Status:  assigned               
   Priority:  blocker   |   Milestone:  8.2.0 (was Update.2)   
  Component:  sugar     |     Version:  Update.1               
 Resolution:            |    Keywords:  8.2.0:? blocks:8.2.0 r-
Next_action:  code      |    Verified:  0                      
  Blockedby:            |    Blocking:                         
------------------------+---------------------------------------------------

Comment(by cscott):

 From my reading of http://svn.gnome.org/viewvc/network-manager-
 applet/trunk/src/applet.c?view=annotate it seems the sleep/wake is what
 the "enable networking" checkbox in the GNOME applet does.  Another option
 for OHM to try is nm_client_wireless_set_enabled...

 BUT for this bug, all the saving and loading the configuration file is
 actually done by sugar in:
 http://dev.laptop.org/git?p=sugar;a=blob;f=src/hardware/nminfo.py;h=a703ff64ea78374236ab45092e9cd8f85a237da0;hb=HEAD#l407

 So it seems like all you need to do is send the appropriate sugar service
 a "reload configuration" signal of some sort and/or prevent the running
 NMInfo from writing out its config (overwriting the cleared one).

 The cleanest solution that I can see is to have the
 NMInfoDBusServiceHelper implement an additional one-method interface
 (org.laptop.sugar.nm?) with a "ReloadConfig" method that does:
 {{{
 self._allowed_networks = self._read_config()
 }}}

 Then the control panel can send the org.freedesktop.NetworkManagerInfo
 object the ReloadConfig message after we've punted the config file.

 A more forward-thinking approach might be to add a removeNetworkInfo
 method which could be used with the getNetworks method to remove all the
 remembered networks to implement a 'reset' button.  In the future, when
 this control panel lets you edit the various network properties, you can
 use 'removeNetworkInfo' and 'updateNetworkInfo' to implement the edits.

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


More information about the Bugs mailing list