profanity icon indicating copy to clipboard operation
profanity copied to clipboard

Replace libsignal-protocol-c with libomemo-c

Open jubalh opened this issue 10 months ago • 5 comments

The library libsignal-protocol-c is not maintained. This commit will replace the dependency libsignal-protocol-c with libomemo-c.

On 2025-02-07 a new version of protobuf-c has been uploaded to Debian. This new version induced a bug for libsignal-protocol-c and libomemo-c:

  • Bug#1095352: libomemo-c: FTBFS [1]
  • Bug#1095353: libsignal-protocol-c: FTBFS [2]

(Thanks to Dino team and Debian Maintainer it's solved)

libomemo-c [3] is a fork of libsignal-protocol-c [4] maintained by the Dino team.

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1095352 [2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1095353 [3] https://github.com/dino/libomemo-c [4] https://github.com/signalapp/libsignal-protocol-c

Note: Commits squashed and commit message edited by @jubalh.

jubalh avatar Feb 21 '25 18:02 jubalh

Received those patches via PM and mailinglist from Stefan.

Comments (to Stefan): I squashed the two commits into 1. If you use 2 commits then the configure change should have been done with the same commit as changing the source files. Since otherwise it wouldn't build ;) Then the CI changes could be a separate step.

Don't forget that we use present tense in commit messages (Replace not Replaced).

I also added the (quite valuable) info that you added to the email but not to the patch files themselves.

And CI sais no :)

jubalh avatar Feb 21 '25 18:02 jubalh

@jubalh: Nice! :)

Can you help to update libomemo-c to latest XEP-0384 version?

  • https://xmpp.org/extensions/xep-0384.html

At this time: 0.8.3 (2022-01-18).

Ticket here:

  • https://github.com/dino/libomemo-c/issues/11

Note: @Ri0n from @psi-im team has done a PR here:

  • https://github.com/dino/libomemo-c/pull/10

Neustradamus avatar Feb 21 '25 18:02 Neustradamus

@sjaeckel do we want an ifdef or just switch to libomemo-c?

jubalh avatar Mar 12 '25 14:03 jubalh

https://github.com/dino/libomemo-c/releases/tag/v0.5.1 has been released. It contains https://github.com/dino/libomemo-c/commit/bf9daf2a11083413874f915a0249f6b4ce761d54.

jubalh avatar Apr 08 '25 05:04 jubalh

@jubalh: Thanks for 0.5.1:

  • https://github.com/dino/libomemo-c/releases/tag/v0.5.1

cc: @Ri0n (@psi-im)

Neustradamus avatar Apr 08 '25 10:04 Neustradamus