Skip to content

Autotools: Fix pdo_firebird shared build - #15371

Merged
petk merged 1 commit into
php:masterfrom
petk:patch-m4-pdo-firebird
Aug 13, 2024
Merged

Autotools: Fix pdo_firebird shared build#15371
petk merged 1 commit into
php:masterfrom
petk:patch-m4-pdo-firebird

Conversation

@petk

@petkpetk commented Aug 13, 2024

Copy link
Copy Markdown
Member

Follow-up of GH-15230:

  • Redundant variables removed

  • Redundant duplicate middle newlines removed

  • PHP_CXX_COMPILE_STDCXX macro arguments quoted

  • When extension is built as shared the PHP_ADD_SOURCES works differently, and PHP_ADD_SOURCES_X needs to be used so this can be used:

    ./configure --with-pdo-firebird=shared

Follow-up of phpGH-15230:
- Redundant variables removed
- Redundant duplicate middle newlines removed
- PHP_CXX_COMPILE_STDCXX macro arguments quoted
- When extension is built as shared the PHP_ADD_SOURCES works
differently, and PHP_ADD_SOURCES_X needs to be used so this can be
used:
./configure --with-pdo-firebird=shared
@petk

petk commented Aug 13, 2024

Copy link
Copy Markdown
MemberAuthor

Merging this as it would be a major issue for the package maintainers causing shared extension not being built...

Sign up for freeto 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.

1 participant

@petk