Hi Marcel, > please write a proper short subject and a detailed commit message body. > Just squeezing everything in the subject line is not acceptable. > > M5: Git commit message 50/72 formatting > ======================================= > The commit message header should be within 50 characters. And if you > have detailed explanatory text, wrap it to 72 character.
Sorry about this, I will pay attention next time. > > diff --git a/plugins/ofono.c b/plugins/ofono.c > > index 74a5c60..5c5d00b 100644 > > --- a/plugins/ofono.c > > +++ b/plugins/ofono.c > > @@ -688,6 +688,8 @@ static int add_network(struct connman_device *device, > > operator = connman_device_get_string(device, "Operator"); > > if (operator) > > connman_network_set_name(network, operator); > > + else > > + connman_network_set_name(network, ""); > > And this has been tested and is fixing your issue? When I debugged it, I > concluded that it needs to be set way earlier. Yes I have test this. I start oFono and ConnMan, and plug a SpeedUp Dongle. As soon as detected, I am doing some ./test-manager in ConnMan. I could see a service /profile/default/cellular... with an empty Name, and then with the correct network name. Without my fix, there is no /profile/default/cellular. I have to kill ConnMan and restart it to see the service with the Network name. Regards, Bertrand --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters: "Les Montalets"- 2, rue de Paris, 92196 Meudon Cedex, France Registration Number: 302 456 199 R.C.S. NANTERRE Capital: 4,572,000 Euros This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. _______________________________________________ connman mailing list [email protected] http://lists.connman.net/listinfo/connman
