Thanks for reviewing it, James. <div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> if have_ofw_tree(): sn = read_ofw('mfg-data/SN') uuid_ = read_ofw('mfg-data/U#') sn = sn or 'SHF00000000' uuid_ = uuid_ or '00000000-0000-0000-0000-000000000000' <div class="im"> > else: > sn = generate_serial_number() > uuid_ = str(uuid.uuid1()) > - setting_name = '/desktop/sugar/collaboration/jabber_server' > - jabber_server = client.get_string(setting_name) > - store_identifiers(sn, uuid_, jabber_server) > + > + setting_name = '/desktop/sugar/collaboration/jabber_server' > + jabber_server = client.get_string(setting_name) > + store_identifiers(sn, uuid_, jabber_server) > + > + if jabber_server: > url = 'http://' + jabber_server + ':8080/' </div>You are effectively repeating the previous if statement but using the output ... seems a bit obscure. </blockquote> <div> ^^ Before applying this : The url (url for registration) was set from gconf only for non-XO devices. I moved that outside the first if..else you have quoted, so that on all devices the url is taken from the gconf property (/desktop/sugar/collaboration/jabber_server) the second 'if jabber_server:' is put there to check whether the jabber_server retrieved from the gcnof property is empty or unset. If jabber_server is a valid one, then change url, other wise keep the url = initialized one. I actually went with the above modification due to the chain of mails here : <a href="http://lists.sugarlabs.org/archive/sugar-devel/2010-July/025265.html">http://lists.sugarlabs.org/archive/sugar-devel/2010-July/025265.html</a> Regards, Dipankar </div></div>