Skip to content

Conversation

@mcalierno
Copy link
Contributor

@mcalierno mcalierno commented Oct 25, 2025

Add /opt/homebrew/{include,lib} flags for apple silicon Macs.

Homebrew moved from /usr/local to /opt/homebrew for Apple silicon Macs, but /opt/homebrew/{include,lib} are not included as default search paths by Clang/GCC.

As a result, without adding these paths, running 'make' fails with errors complaining that <gcrypt.h>, <signal/key_helper.h> etc cannot be found.


UPDATE:

Use PKG_CHECK_MODULES to check for libgcrypt: previously ./configure --enable-omemo resulted in

configure: gcrypt >= 1.7.0 not found, OMEMO support not enabled
configure: error: OMEMO support requires a library which is missed

despite libgcrypt being installed.

Add /opt/homebrew/{include,lib} flags for apple silicon macs, which are not included as default search paths by Clang/GCC. (Homebrew uses /usr/local/{include,lib} on intel macs)
On macOS running ./configure --enable-omemo was failing to find gcrypt despite being installed.
@mcalierno mcalierno requested a review from sjaeckel October 26, 2025 12:03
Copy link
Member

@sjaeckel sjaeckel left a comment

Choose a reason for hiding this comment

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

LGTM!

Thanks for the PR. @jubalh is sick unfortunately and I don't know when he will be back, so this could take some time to be merged.

Not sure if you have the time & patience, but maybe you want to add CI for Mac?

If you go for it I'd recommend https://github.com/nektos/act for developing that, which should also work on Mac as of their docs.

@mcalierno
Copy link
Contributor Author

I'll definitely look into it!

@jubalh jubalh added this to the next milestone Dec 3, 2025
@jubalh
Copy link
Member

jubalh commented Dec 3, 2025

Thanks!
In case you will contribute the CI, which would be awesome, please open another PR!

@jubalh jubalh merged commit e8e9c29 into profanity-im:master Dec 3, 2025
5 checks passed
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.

3 participants