Skip to content

fix(omemo): add missing includes for libomemo-c builds#2133

Merged
jubalh merged 1 commit intoprofanity-im:masterfrom
GunniBusch:fix/omemo-libary
Mar 27, 2026
Merged

fix(omemo): add missing includes for libomemo-c builds#2133
jubalh merged 1 commit intoprofanity-im:masterfrom
GunniBusch:fix/omemo-libary

Conversation

@GunniBusch
Copy link
Copy Markdown
Contributor

  • I ran valgrind when using my new feature

This pr fixes some missing libomemo-c imports and usages of HAVE_LIBOMEMO_C before import config.h.

The reason for this pr is the failing homebrew build: Homebrew/homebrew-core#274439

Changes by file

  • src/omemo/crypto.h: add missing libomemo-c imports (selection via HAVE_LIBOMEMO_C) and import config.h (for HAVE_LIBOMEMO_C)
  • src/omemo/store.h: add missing libomemo-c imports (selection via HAVE_LIBOMEMO_C)
  • src/omemo/store.c: move config.h before the conditional include so the compile-time flag is defined before header selection.

Signed-off-by: Leon Adomaitis <leon@adomaitis.de>
@jubalh jubalh merged commit 0dbc18c into profanity-im:master Mar 27, 2026
10 checks passed
@jubalh jubalh assigned jubalh and GunniBusch and unassigned jubalh Mar 27, 2026
@jubalh jubalh added this to the next milestone Mar 27, 2026
@jubalh
Copy link
Copy Markdown
Member

jubalh commented Mar 27, 2026

Thank you!

@jubalh
Copy link
Copy Markdown
Member

jubalh commented Mar 27, 2026

@GunniBusch would you be interested in adding a macOS homebrew build to our CI?

@GunniBusch
Copy link
Copy Markdown
Contributor Author

@GunniBusch would you be interested in adding a macOS homebrew build to our CI?

Sure. I can try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants