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

Update pynacl dependency to 1.2.1 or higher #2888

Merged
merged 1 commit into from
Feb 19, 2018

Conversation

bachp
Copy link
Contributor

@bachp bachp commented Feb 18, 2018

I'm not sure if there was a reason why pynacl was fixed to 0.3.0. But i tried synapse with this patch on my homeserver and didn't experience any issues so far.

@matrixbot
Copy link
Member

Can one of the admins verify this patch?

1 similar comment
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@richvdh
Copy link
Member

richvdh commented Feb 19, 2018

I'm not sure if there was a reason why pynacl was fixed to 0.3.0.

well, it happened in 0935802. I'm prepared to believe that whatever the problem was has been fixed in the last two years.

@richvdh
Copy link
Member

richvdh commented Feb 19, 2018

@matrixbot test this please

@richvdh
Copy link
Member

richvdh commented Feb 19, 2018

please could you rebase this on develop, and sign off, as per https://github.com/matrix-org/synapse/blob/master/CONTRIBUTING.rst

@bachp bachp changed the base branch from master to develop February 19, 2018 09:49
@bachp
Copy link
Contributor Author

bachp commented Feb 19, 2018

@richvdh Rebased and sign-off added.

@richvdh
Copy link
Member

richvdh commented Feb 19, 2018

@matrixbot test this please

@richvdh
Copy link
Member

richvdh commented Feb 19, 2018

lgtm, thanks!

@richvdh richvdh merged commit 3fc33ba into matrix-org:develop Feb 19, 2018
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