[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