Merge lp:~penguin359/ubuntu/precise/network-manager/fix-for-1005091 into lp:~network-manager/network-manager/ubuntu
| Status: | Merged |
|---|---|
| Approved by: | Mathieu Trudel-Lapierre on 2012-06-04 |
| Approved revision: | 518 |
| Merge reported by: | Mathieu Trudel-Lapierre |
| Merged at revision: | not available |
| Proposed branch: | lp:~penguin359/ubuntu/precise/network-manager/fix-for-1005091 |
| Merge into: | lp:~network-manager/network-manager/ubuntu |
| Diff against target: |
12 lines (+1/-1) 1 file modified
debian/ifblacklist_migrate.sh (+1/-1) |
| To merge this branch: | bzr merge lp:~penguin359/ubuntu/precise/network-manager/fix-for-1005091 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Mathieu Trudel-Lapierre | 2012-05-27 | Disapprove on 2012-05-29 | |
|
Review via email:
|
|||
Description of the Change
Fixed issue with NetworkManager failing to disable all lines related to network interface when taking over management of the device. Prior to this fix, if IPv6 was employed on the network, it would not disable the inet6 lines for the interface thereby preventing NetworkManager from taking over control of the interface, but still disabling dhcp in the /etc/network/
| Loren M. Lang (penguin359) wrote : | # |
I've recreated this merge proposal at:
https:/
And using lp:~network-manager/network-manager/ubuntu as the parent branch, but that appears to be what I used the first time so I might be misunderstanding things. My second MP listed above also includes an additional related fix so it should be used instead. According to Bzr-Vcs: in the Deb, https:/
| Mathieu Trudel-Lapierre (cyphermox) wrote : | # |
Rejected: superseded by another MP.
| Mathieu Trudel-Lapierre (cyphermox) wrote : | # |
This is getting messy, but this is indeed the branch I want to merge; based on lp:~network-maanger/network-manager/ubuntu.


Thanks!
Could you please re-do your changes against the packaging branch for network-manager:
lp:~network-manager/network-manager/ubuntu
Disapprove, since this is the wrong branch; but otherwise it looks good on paper (I'll test on my end too).