Andy asked me to have a look, so here's my initial feedback.

General
=======

I'm not sure if these changes should be backported to Artful too, to
maintain the upgrade path for users transitioning off Xenial. Could you
please ask the SRU team?

Otherwise, the changes are risky but I think they should be in principle
okay.

The test cases and regression potential in the description here could do
with fleshing out a bit - could you do that please? For example,
snapcraft depends on pymacaroons - probably want to make sure that keeps
working. And it is supposed to be compatible, but that doesn't mean
there is no bug in the newer version - it is a regression potential IMO.

httmock
=======

Since this is completely new and a no-change backport, I think it's
okay. Uploaded.

protobuf
========

I'm a bit worried about bugs coming from the use of 2to3 which is best
used as a starting point for a manual port, not for an automatic
conversion - how about taking the upstream patches instead?

https://github.com/google/protobuf/commit/f336d4b7a5c1d369ed508e513d482c885705e939

https://github.com/google/protobuf/commit/fe7d9379df3ce7c951bc0652a451413cff02382a

https://github.com/google/protobuf/commit/47ee4d37c17db8e97fe5b15cf918ab56ff93bb18

I tried the testsuite with this and it works after resolving some minor
conflicts.

pymacaroons
===========

Seems okay if it's compatible - can you check with Colin please? I'm
happy to handle the actual upload.

py-macaroon-bakery
==================

I'm not qualified to review this - maybe also check with Colin as the
maintainer? The patch says it's editing a generated file - is that
right?

** Changed in: httmock (Ubuntu)
       Status: New => In Progress

** Changed in: httmock (Ubuntu)
     Assignee: (unassigned) => Andrea Azzarone (azzar1)

** Also affects: protobuf (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Also affects: python-nacl (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Also affects: pymacaroons (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Also affects: httmock (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Changed in: httmock (Ubuntu)
       Status: In Progress => Fix Released

** Changed in: httmock (Ubuntu Xenial)
       Status: New => In Progress

** Changed in: httmock (Ubuntu Xenial)
     Assignee: (unassigned) => Andrea Azzarone (azzar1)

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

Title:
  [SRU] Please backport python3-macaroonbakery 0.0.6-1 [universe] from
  bionic

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

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

Reply via email to