-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix: building with clang 16 #313
Conversation
@reckenrode do you mind updating the flake.lock file within this PR ? |
I pushed another commit with the updated |
c375485
to
0b04a45
Compare
I force-pushed a fix for the evaluation errors that were happening on Linux. |
@reckenrode It's super, it's almost fixed! The Redis extension is causing some issues. Do you think it needs to be updated in the same way other extensions were updated? |
Flake lock file updates: • Updated input 'nixpkgs': 'github:NixOS/nixpkgs/3f21a22b5aafefa1845dec6f4a378a8f53d8681c' (2023-11-20) → 'github:NixOS/nixpkgs/010c7296f3b19a58b206fdf7d68d75a5b0a09e9e' (2023-11-28)
0b04a45
to
d043617
Compare
It did. I added the appropriate warning downgrades and confirmed it builds locally. I also made sure tidy and xdebug built as well. Everything in the checks should pass now. |
Thank you very much for your help!!! |
Downgrade compiler errors and apply a patch to imap to fix building with clang 16.
Closes #312.