Skip to content

Conversation

@def-
Copy link

@def- def- commented Nov 4, 2025

No description provided.

@def- def- requested a review from a team as a code owner November 4, 2025 07:13
@confluent-cla-assistant
Copy link

confluent-cla-assistant bot commented Nov 4, 2025

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ def-
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@def- def- force-pushed the pr-fix-oauthbearer-oidc branch from 09315bd to c71a10f Compare November 4, 2025 09:37
@def- def- changed the title Fix compilation with WITH_OAUTHBEARER_OIDC=0 Fix build with WITH_OAUTHBEARER_OIDC=0 Nov 4, 2025
@def- def- force-pushed the pr-fix-oauthbearer-oidc branch from f1dbc9a to c71a10f Compare November 4, 2025 14:11
@gkozar
Copy link

gkozar commented Nov 6, 2025

Could this be merged? It's causing a problem for me as well.

More detailed explanation:
WITH_OAUTHBEARER_OIDC is always defined, but it set to 0 if disabled, and 1 if enabled.
This is because packaging/cmake/config.h.in has: #cmakedefine01 WITH_OAUTHBEARER_OIDC

Therefore, #ifdef WITH_OAUTHBEARER_OIDC always runs.

@gkozar gkozar mentioned this pull request Nov 6, 2025
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