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

native/libmysqlclient: Fix build by using legacy cmake version #6212

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

th0ma7
Copy link
Contributor

@th0ma7 th0ma7 commented Aug 30, 2024

Description

native/libmysqlclient: Fix build by using legacy cmake version
Ported over code from mk/spksrc.cross-cmake-env.mk to mk/spksrc.native-cmake-env.mk to handle similarly options:

  • USE_NATIVE_CMAKE_LEGACY
  • USE_NATIVE_CMAKE

Fixes cross/mysql-connector-c build which impact ntop and other - Was found out during xz update PR #6210 as a build failure.

Relates to #6200

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

Ported over code from mk/spksrc.cross-cmake-env.mk to
mk/spksrc.native-cmake-env.mk to handle similarly options:
- USE_NATIVE_CMAKE_LEGACY
- USE_NATIVE_CMAKE
@th0ma7 th0ma7 self-assigned this Aug 30, 2024
@th0ma7 th0ma7 mentioned this pull request Aug 30, 2024
10 tasks
@th0ma7 th0ma7 merged commit 5406646 into SynoCommunity:master Aug 30, 2024
2 checks passed
@th0ma7 th0ma7 deleted the misc-update-mysql-connector-c branch August 30, 2024 00:03
@hgy59 hgy59 mentioned this pull request Sep 16, 2024
6 tasks
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.

1 participant