Skip to content
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 detection of odbc and apache2 for Fedora #4461

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

topazus
Copy link
Contributor

@topazus topazus commented Feb 15, 2024

I used this PR as a patch in https://src.fedoraproject.org/rpms/poco/pull-request/5.
It can help the finding of obdc and apache2 system dependencies on Fedora, without manually defining the location of the ODBC_INCLUDE_DIRS and APACHE2_INCLUDE_DIRS variables.

@aleks-f aleks-f merged commit d6dfa25 into pocoproject:devel Feb 15, 2024
31 checks passed
aleks-f added a commit that referenced this pull request Feb 16, 2024
* fix(SQLParser): move to Data dir; add extradirs, remove vs 140,150 build scripts generation

* fix(ActiveRecord): Makefile #4453

* fix(Data): Make SQLParser internal

* fix(build): ODBC Makefile and Data libs CmakeLists #4453

* fix(build): Data libs CMakeLists #4453

* fix(build): regenerate VS projects #4453

* fix(CppUnit): remove Foundation dependency

* fix(SQLParser): remove leftover unnecessary includes

* fix(SQLParser): remove fwd header #4453

* fix(PocoDoc): remove SQLParser from include list

* fix(mkrelease): remove 140,150 vs projects

* fix(SQLParser): Disable SQL parsing by default #4462

* fix(test): Disable SQL parsing by default #4462

* Fix detection of odbc and apache2 for Fedora (#4461)

* Fixed transaction handling in MySQL test when SQL parser is switched off

* chore(ParallelSocketAcceptor): remove unnecessary include and using from header

---------

Co-authored-by: Matej Kenda <[email protected]>
Co-authored-by: topazus <[email protected]>
Co-authored-by: Friedrich Wilckens <[email protected]>
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