[Merge] lp:~phablet-team/network-manager/lp1445080-wily into lp:~network-manager/network-manager/ubuntu

Mathieu Trudel-Lapierre mathieu.tl at gmail.com
Wed Jul 15 18:51:42 UTC 2015


Review: Needs Information

Have the code bits been discussed with upstream? What is the likely impact on non-ofono modems of the changes to nm_device_modem to not return that connections are unavailable when the modem is still in NM_MODEM_STATE_INITIALIZING state, for example?

As per the other patches, I think the ofono code changes should be merged in to the main patch for the modem support (since we're at a development release, this makes it just one large patch to fixup when there is a need to refresh the patch, rather than conflicting, interdepending change sets), and any core changes should be in a separate, isolated patch so that we can send them upstream ASAP.

Since I don't know whether there has been careful testing of the impact of the NMDeviceModem changes on non-ofono modem -> Needs Information.
-- 
https://code.launchpad.net/~phablet-team/network-manager/lp1445080-wily/+merge/264895
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/network-manager/lp1461593-wily.



More information about the Ubuntu-reviews mailing list