Hi Richard, thank you for taking this on.

> (BTW, why does -proposed even exist for the current development
version of Ubuntu? Why not go straight to release?)

Please see https://wiki.ubuntu.com/ProposedMigration. It allows us to
run dep8 tests on reverse test dependencies so updating one package
cannot regress others. phpseclib is currently stuck in yakkety-proposed
for this reason. The error looks quite similar to the problem you're
fixing but may be in a different package.

I'm subscribing Nish to take a look as I think he may know more about a
general solution for this.

This should not block an SRU to Xenial, but for a Xenial fix we want a
minimal diff please. I expect a single changelog entry referencing this
bug and a minimal patch, rather than basing on a newer Debian version.
Just dropping the quilt patch as you have done is fine if that is the
minimal requirement to fix the bug - is it? I appreciate that the newer
Debian version reverted one change, but what isn't clear to me is
whether that reversion is the minimal reversion necessary to fix this
bug.

Additionally we should consider testing other reverse depends of
phpseclib (if any exist?) for regressions, too. I think other reverse
dependencies should be discussed in the Regression Potential section.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1574058

Title:
  php-seclib: Call to undefined method Crypt_Base::Crypt_Base()

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/phpseclib/+bug/1574058/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to