Skip to content

PXC-4676: Code refresh for - PXC 8.4.5 (Fixed MYSQL_MAINTAINER_MODE=ON build.)#2120

Merged
kamil-holubicki merged 1 commit into
percona:release-8.4.5from
kamil-holubicki:PXC-4676-maint-mode-fix
Jun 10, 2025
Merged

PXC-4676: Code refresh for - PXC 8.4.5 (Fixed MYSQL_MAINTAINER_MODE=ON build.)#2120
kamil-holubicki merged 1 commit into
percona:release-8.4.5from
kamil-holubicki:PXC-4676-maint-mode-fix

Conversation

@kamil-holubicki

@kamil-holubicki kamil-holubicki commented Jun 9, 2025

Copy link
Copy Markdown
Contributor

https://perconadev.atlassian.net/browse/PXC-4676

Fixed MYSQL_MAINTAINER_MODE=ON build.
When mysqlbinlog.cc was compiled, WITH_WSREP was undefined. It caused One Definition Rule violation, because handler.h is included indirectly in several compilation units and it contains legacy_db_type that has values conditionally compiled in (WITH_WSREP).

There is no reason why WITH_WSREP should be undefined for mysqlbinlog.cc

@kamil-holubicki
kamil-holubicki requested a review from aybek June 9, 2025 14:05
https://perconadev.atlassian.net/browse/PXC-4676

Fixed MYSQL_MAINTAINER_MODE=ON build.
When mysqlbinlog.cc was compiled, WITH_WSREP was undefined. It caused
One Definition Rule violation, because handler.h is included indirectly
in several compilation units and it contains legacy_db_type that has
values conditionally compiled in (WITH_WSREP).

There is no reason why WITH_WSREP should be undefined for mysqlbinlog.cc
@kamil-holubicki
kamil-holubicki force-pushed the PXC-4676-maint-mode-fix branch from bc3117b to ae32ec0 Compare June 9, 2025 14:07

@inikep inikep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@kamil-holubicki
kamil-holubicki merged commit 2050d77 into percona:release-8.4.5 Jun 10, 2025
19 of 21 checks passed
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