[Merge] lp:~stephane-duchesneau/ubuntu/trusty/pexpect/errno_handling_fix into lp:ubuntu/trusty/pexpect

Barry Warsaw barry at canonical.com
Thu May 8 19:50:12 UTC 2014


Hi Stéphane.  First, thanks very much for your contribution to Ubuntu!

There's a bit of a problem with your merge proposal.  As you'll see in the diff, it is including the .pc directory.  This usually happens when the quilt patch stack for your branch is at a different level than the stack of the target branch.  The fix for this when you're adding a new quilt patch, is to do one `quilt pop` so that the branch you're proposing for merge is at the same level as the branch you're proposing to merge into.

Aside from that, I'll note that the version of pexpect in Utopic (and Debian) is behind upstream's release on PyPI, and in fact this bug is fixed in the latest upstream release.  Here's the upstream bug and diff:

http://pexpect.readthedocs.org/en/latest/history.html#releases

https://github.com/pexpect/pexpect/commit/a97f580d5cdd0f479ea64d36edc164ea769d4084  (this includes tests)

https://github.com/pexpect/pexpect/pull/38/

Since the package is team maintained in Debian, I will update it there, and that should automatically flow into Ubuntu 14.10.  It's probably worth fixing this in Trusty though, so I'll file an SRU and attempt to backport the upstream fix into the Trusty branch.
-- 
https://code.launchpad.net/~stephane-duchesneau/ubuntu/trusty/pexpect/errno_handling_fix/+merge/218813
Your team Ubuntu branches is subscribed to branch lp:ubuntu/trusty/pexpect.



More information about the Ubuntu-reviews mailing list