Page MenuHomeFreeBSD

databases/mysql56-server: Fix build with LibreSSL
ClosedPublic

Authored by brnrd on Jan 21 2017, 11:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 9, 11:22 AM
Unknown Object (File)
Sun, Feb 8, 2:09 PM
Unknown Object (File)
Sat, Jan 31, 5:06 PM
Unknown Object (File)
Sat, Jan 31, 1:12 PM
Unknown Object (File)
Fri, Jan 30, 10:10 PM
Unknown Object (File)
Mon, Jan 19, 11:08 AM
Unknown Object (File)
Dec 25 2025, 1:08 PM
Unknown Object (File)
Dec 23 2025, 2:33 AM
Subscribers

Details

Summary
databases/mysql56-server: Fix build with LibreSSL

  - Fix CMake SSL detection
  - Always set WITH_SSL=${OPENSSLBASE}

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

brnrd retitled this revision from to databases/mysql56-server: Fix build with LibreSSL.
brnrd updated this object.
brnrd edited the test plan for this revision. (Show Details)
brnrd added a reviewer: mmokhi.

I already worked on a patch and tested it,
Would you please check it too?

databases/mysql56-client/files/patch-cmake_ssl.cmake
1 ↗(On Diff #24272)

The patch you have in 216311 should actually look for LIBRESSL_VERSION_NUMBER, not OPENSSL_MAJOR_VERSION. Not sure how to do that.

8 ↗(On Diff #24272)

This is actually incorrect. With cmake 3.7 it should be VERSION_GREATER_EQUAL

databases/mysql56-client/files/patch-cmake_ssl.cmake
1 ↗(On Diff #24272)

Using new variable (LIBRESSL_VERSION_NUMBER instead of OPENSSL_MAJOR_VERSION) is not necessary i guess, because the script will fill OPENSSL_MAJOR_VERSION with any found info inside $OPENSSL_BASE.

Changing the name makes script much more readable though ;-)
But currently it works.

brnrd edited edge metadata.

Update diff to VERSION_GREATER_EQUAL

mmokhi edited edge metadata.

Okay, Looks good to me.
Thanks :-)

This revision is now accepted and ready to land.Jan 21 2017, 12:59 PM
This revision was automatically updated to reflect the committed changes.