Skip to content

[omemo-backend-signal] updated to 0.2.3 #1760

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

goffi-contrib
Copy link
Contributor

"wireformat.patch" has been removed because it is not needed anymore since the recent update of protobuf recipe.

@goffi-contrib goffi-contrib force-pushed the omemo-backend-signal_update_0.2.3 branch from 470d774 to fe7879a Compare March 16, 2019 15:15
@AndreMiras
Copy link
Member

Thanks for the PR, the build is currently failing because of the DoubleRatchet download as you were mentioning. I'd like us to spend some time to investigate this too before we can merge

@AndreMiras AndreMiras self-assigned this Mar 16, 2019
@goffi-contrib goffi-contrib force-pushed the omemo-backend-signal_update_0.2.3 branch from fe7879a to 7259dcb Compare March 17, 2019 16:21
@AndreMiras
Copy link
Member

Now it's failing because the xeddsa recipe error /bin/cp: cannot stat '_crypto_sign.so': No such file or directory.
This probably needs to be addressed in #1756

Copy link
Member

@AndreMiras AndreMiras left a comment

Choose a reason for hiding this comment

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

Hi @goffi-contrib since #1756 got merged, could you rebase your PR to see if the build is now passing?

@goffi-contrib goffi-contrib force-pushed the omemo-backend-signal_update_0.2.3 branch from 7259dcb to 70987e6 Compare March 30, 2019 09:50
"wireformat.patch" has been removed because it is not needed anymore since the recent update of protobuf recipe.
@goffi-contrib goffi-contrib force-pushed the omemo-backend-signal_update_0.2.3 branch from 70987e6 to d85e988 Compare March 30, 2019 09:50
@AndreMiras
Copy link
Member

Build is now passing, merging, thanks for the contrib 😄

@AndreMiras AndreMiras merged commit 926e612 into kivy:master Mar 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants