Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#37172 closed defect (fixed)

mysql55 should link against MacPorts libwrap instead of host version

Reported by: jeremyhu (Jeremy Huddleston Sequoia) Owned by: pixilla (Bradley Giesbrecht)
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: Cc: ryandesign (Ryan Schmidt), jeremyhu (Jeremy Huddleston Sequoia)
Port: mysql5 mysql51 mysql55

Description (last modified by jeremyhu (Jeremy Huddleston Sequoia))

mysql55-5.5.28_1.darwin_11.x86_64.tbz2 from http://packages.macports.org/mysql55

/opt/local/bin $ otool -L /opt/local/lib/mysql55/bin/mysqld
/opt/local/lib/mysql55/bin/mysqld:
	/System/Library/Frameworks/CoreServices.framework/Versions/A/CoreServices (compatibility version 1.0.0, current version 53.0.0)
	/opt/local/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.7)
	/usr/lib/libwrap.7.dylib (compatibility version 7.0.0, current version 7.6.0)
	/usr/lib/libstdc++.6.dylib (compatibility version 7.0.0, current version 52.0.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 159.1.0)

It should be using /opt/local/lib/libwrap.7.dylib from port:tcp_wrappers

Attachments (2)

main.log (177.7 KB) - added by jeremyhu (Jeremy Huddleston Sequoia) 7 years ago.
mail.log
mysql_libwrap.patch (3.9 KB) - added by jeremyhu (Jeremy Huddleston Sequoia) 7 years ago.
patch

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Description: modified (diff)

Deleted my proposed changes since they were insufficient. Even with the package installed, the build picks it up from /usr instead of $prefix

Changed 7 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Attachment: main.log added

mail.log

comment:2 Changed 7 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Weird, the main.log was from the upgrade ... I didn't realize that 'port build' doesn't leave a main.log ... but it doesn't matter... my changes actually did work, I just forgot that I hadn't installed =)

Changed 7 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Attachment: mysql_libwrap.patch added

patch

comment:3 Changed 7 years ago by jeremyhu (Jeremy Huddleston Sequoia)

Resolution: fixed
Status: newclosed

Fixed in r100154 et al

comment:4 Changed 6 years ago by ryandesign (Ryan Schmidt)

It wasn't fixed in percona but I've included the patch to do so in #39500.

Note: See TracTickets for help on using tickets.