Skip to content

Commit

Permalink
Pin dependencies for py36 compatibility
Browse files Browse the repository at this point in the history
macaroonbakery in version 1.3.4 started depending on protobuf>3.20 which
it's not compatible with python 3.6, so this change pins macaroonbakery
to 1.3.2 which was the last release compatible with older versions of
protobuf.

For more details see:

- go-macaroon-bakery/py-macaroon-bakery#92
- protocolbuffers/protobuf#10076

It's worth to mention that there is no protobuf-4.0, it goes from 3.20
to 4.21 - https://pypi.org/project/protobuf/#history

(cherry picked from commit a8e4de5)
(cherry picked from commit 300c5bb)
  • Loading branch information
freyes committed Dec 14, 2023
1 parent 8d58b5f commit 6e972ae
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 5 deletions.
17 changes: 15 additions & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,19 @@ python-libmaas
# Documentation requirements
sphinx
sphinxcontrib-asyncio
# https://github.com/go-macaroon-bakery/py-macaroon-bakery/issues/94
macaroonbakery!=1.3.3
openstacksdk<=1.5.0; # Pin to a lower version of openstacksdk due to introduced libraries
# The following pinning is to keep zaza compatible with python 3.6
#
# protobuf dropped support for python-3.6 in v3.20.0[0], although it wasn't
# until 4.21.0 that it became truly incompatible. The pinning used is `<4.21.0`
#
# [0] https://github.com/protocolbuffers/protobuf/commit/301d315dc4674d1bc799446644e88eff0af1ac86
# [1] https://github.com/protocolbuffers/protobuf/issues/10076
protobuf<4.21.0

# macaroonbakery in v1.3.4 added a constraint of protobuf>=3.20.0[0] which makes it incompatible with python 3.6
# while v1.3.3 was released in a broken state[1]
#
# [0] https://github.com/go-macaroon-bakery/py-macaroon-bakery/commit/7f1fe6a2adb2f80db12bccfb81f629d66d106e03
# [1] https://github.com/go-macaroon-bakery/py-macaroon-bakery/pull/92
macaroonbakery < 1.3.3
17 changes: 14 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,20 @@
'PyYAML',
'tenacity',
'python-libmaas',

# https://github.com/go-macaroon-bakery/py-macaroon-bakery/issues/94
'macaroonbakery != 1.3.3',
# protobuf dropped support for python-3.6 in v3.20.0[0], although it wasn't
# until 4.21.0 that it became truly incompatible. The pinning used is
# `<4.21.0`
#
# [0] https://github.com/protocolbuffers/protobuf/commit/301d315dc4674d1bc799446644e88eff0af1ac86 # noqa
# [1] https://github.com/protocolbuffers/protobuf/issues/10076
'protobuf < 4.21.0',
# macaroonbakery in v1.3.4 added a constraint of protobuf>=3.20.0[0] which
# makes it incompatible with python 3.6 while v1.3.3 was released in a
# broken state[1]
#
# [0] https://github.com/go-macaroon-bakery/py-macaroon-bakery/commit/7f1fe6a2adb2f80db12bccfb81f629d66d106e03 # noqa
# [1] https://github.com/go-macaroon-bakery/py-macaroon-bakery/pull/92
'macaroonbakery < 1.3.3',
]

tests_require = [
Expand Down

0 comments on commit 6e972ae

Please sign in to comment.