Surbhi, here is a quick review:
1) "This merge should solve the bug: LP: #603582" doesn't say anything. 
Something like "Merge from Debian unstable" is more descriptive.
2) debian/changelog: It's not obvious what change was a "remaining change" and 
what was a new change introduces by you.
3) You have to check lintian to not introduce more warnings with your merge. 
For example syntax-error-in-debian-changelog and debian-changelog-line-too-long 
needs to get fixed.
4) debian/changelog: "debian/rules: kept the commit 
388049ed6f5f76ac8e484dda8de80831ab66a8ab from debian git." doesn't describe the 
change.
5) Please give debian/patches/debian-changes-3.1.2-2ubuntu1 a new name and a 
proper DEP-3 header.
6) Please describe which Ubuntu changes can be dropped (either here or in 
debian/changelog).

Please resubscribe ubuntu-sponsors once you have addressed the points.

** Changed in: mdadm (Ubuntu)
       Status: In Progress => Incomplete

-- 
Please merge mdadm 2.7.1.1 (main) from Debian unstable (main)
https://bugs.launchpad.net/bugs/603582
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to