Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Optional dependencies specifically for building the debs #7210

Closed
wants to merge 1 commit into from

Conversation

anoadragon453
Copy link
Member

Only applies the workaround for #7208 to building debian packages, not regular Synapse users.

Prevents people with Pillow v7.1.0 from receiving an error trying to start Synapse. They could downgrade, but it'd be a massive faff.

@@ -100,6 +98,9 @@
"sentry": ["sentry-sdk>=0.7.2"],
"opentracing": ["jaeger-client>=4.0.0", "opentracing>=2.2.0"],
"jwt": ["pyjwt>=1.6.4"],
# Pillow 7.1.0 causes the following issue on debian buster:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More than just buster was affected, it was just the first to report an error.

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure I'm a fan of this solution, but on mobile right now

@richvdh
Copy link
Member

richvdh commented Apr 2, 2020

to enlarge on the above, I'd like to see us doing this with patches in the debian directory rather than defining a new extras section, which is somewhat opaque imho. In particular I think that --preinstall="pillow==7.0.0" as an argument to dh_virtualenv will do what you want.

In any case, I think --install-suffix defines the path we install to in /opt/venvs, which isn't what you're after anyway.

@richvdh
Copy link
Member

richvdh commented Apr 2, 2020

I think #7212 is the correct solution here.

@richvdh richvdh closed this Apr 3, 2020
@anoadragon453 anoadragon453 deleted the anoa/debian_optional_deps branch April 3, 2020 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants