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

Conversation

anoadragon453
Copy link
Member

The building of Synapse v1.2.0's debian packages complained about these dependencies until I added them, so I assume we need them now.

  • libdistro-info-perl seems to be only for debian buster for determining which debian version it was on.
  • libsqlite3-dev ...for sqlite3 stuff I suppose?

@anoadragon453 anoadragon453 requested a review from a team July 25, 2019 14:02
@codecov
Copy link

codecov bot commented Jul 25, 2019

Codecov Report

Merging #5766 into release-v1.3.0 will decrease coverage by 0.14%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           release-v1.3.0    #5766      +/-   ##
==================================================
- Coverage           63.42%   63.28%   -0.15%     
==================================================
  Files                 333      331       -2     
  Lines               36652    36408     -244     
  Branches             6057     6012      -45     
==================================================
- Hits                23246    23039     -207     
+ Misses              11754    11726      -28     
+ Partials             1652     1643       -9

@richvdh richvdh self-requested a review July 25, 2019 14:29
@richvdh
Copy link
Member

richvdh commented Jul 29, 2019

I don't understand why this is needs and want to look into it

@anoadragon453 anoadragon453 requested a review from richvdh July 31, 2019 12:14
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.

can you update the PR title too please?

@anoadragon453 anoadragon453 changed the base branch from develop to release-v1.3.0 August 15, 2019 10:12
@anoadragon453 anoadragon453 changed the title Add libsqlite3-dev and libdistro-info-perl to debian docker build dependencies Remove libsqlite3-dev from required build dependencies. Aug 15, 2019
@anoadragon453 anoadragon453 requested a review from richvdh August 15, 2019 10:15
debian/changelog Outdated
@@ -1,3 +1,8 @@
matrix-synapse-py3 (1.3.0) stable; urgency=medium
Copy link
Member

Choose a reason for hiding this comment

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

I think this needs to be UNRELEASED or something, to avoid confusing the debian release scripts; see https://github.com/matrix-org/synapse/blob/master/CONTRIBUTING.rst#debian-changelog for the right incantation

Co-Authored-By: Richard van der Hoff <[email protected]>
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.

\o/

please squash-merge!

@richvdh richvdh merged commit 8cf7fbb into release-v1.3.0 Aug 15, 2019
@anoadragon453 anoadragon453 deleted the anoa/debian_build_deps branch August 15, 2019 12:02
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