9.9.3-P2_1 +dlz_mysql fails to build

[erik@…]

Attached main.log...

[danielluke (Daniel J. Luke)]

That's not a clean build, can you clean, attempt to build again, and attach the main.log if it fails?

[danielluke (Daniel J. Luke)]

also - can you verify if you're building revision 0 or revision 1 (the one I just committed)?

[erik@…]

Replying to dluke@…:

That's not a clean build, can you clean, attempt to build again, and attach the main.log if it fails?

Ok, I've done: port clean bind9

[erik@…]

Replying to dluke@…:

also - can you verify if you're building revision 0 or revision 1 (the one I just committed)?

#port outdated
bind9                          9.9.1-P1_0 < 9.9.3-P2_1
#port upgrade outdated
.
.
.
Error: org.macports.build for port bind9 returned: command execution failed

Uploading new log...

[danielluke (Daniel J. Luke)]

Ok, looks like this is the +dlz_mysql5 variant that is causing the issue. I don't use it, so it's been entirely supported by people who use it and who generate patches when it doesn't work correctly.

From your log, it looks like ${mysql_lib} just isn't set (for some reason) resulting in the bad link line.

What's the output of /opt/local/bin/mysql_config5 --libs ?

[erik@…]

Replying to dluke@…:

Ok, looks like this is the +dlz_mysql5 variant that is causing the issue. I don't use it, so it's been entirely supported by people who use it and who generate patches when it doesn't work correctly.

From your log, it looks like ${mysql_lib} just isn't set (for some reason) resulting in the bad link line.

What's the output of /opt/local/bin/mysql_config5 --libs ?

-L/opt/local/lib -Wl,-headerpad_max_install_names   -L/opt/local/lib/mysql5/mysql -lmysqlclient -L/opt/local/lib -lz -lm  -L/opt/local/lib/ -lssl -lcrypto

[danielluke (Daniel J. Luke)]

Ok, can you edit your local mysql-link.patch to change the awk 'print substr($2,3)' to 'print substr($1,3)', clean and build and let me know if it works (if so, I'll update the patch).

[danielluke (Daniel J. Luke)]

otherwise, the mysql-link.patch probably just needs some other change to make it work. ... and we should get someone who uses it/who is interested in it to fix it up.

[erik@…]

I found two instances of the string you mentioned. I edited both. Then:

port clean bind9
port build bind9
port install bind9
shutdown -r now

Unfortunately DLZ is no longer functional:

12/09/13 20.05.27	named[391]	starting BIND 9.9.3-rl.13214.22-P2 -f
12/09/13 20.05.27	named[391]	built with '--prefix=/opt/local' '--mandir=/opt/local/share/man' '--with-openssl=/opt/local' '--with-libxml2=/opt/local' '--enable-threads' '--enable-ipv6' 'CC=/usr/bin/gcc-4.2' 'CFLAGS=-pipe -Os -arch x86_64' 'LDFLAGS=-L/opt/local/lib -Wl,-headerpad_max_install_names -arch x86_64' 'CPPFLAGS=-I/opt/local/include'
12/09/13 20.05.27	named[391]	----------------------------------------------------
12/09/13 20.05.27	named[391]	BIND 9 is maintained by Internet Systems Consortium,
12/09/13 20.05.27	named[391]	Inc. (ISC), a non-profit 501(c)(3) public-benefit 
12/09/13 20.05.27	named[391]	corporation.  Support and training for BIND 9 are 
12/09/13 20.05.27	named[391]	available at https://www.isc.org/support
12/09/13 20.05.27	named[391]	----------------------------------------------------
12/09/13 20.05.27	named[391]	unsupported DLZ database driver 'mysql'.  Mysql zone not loaded.
12/09/13 20.05.27	named[391]	loading configuration: not found
12/09/13 20.05.27	named[391]	exiting (due to fatal error)
12/09/13 20.05.27	com.apple.launchd[1]	(org.macports.bind9) Throttling respawn: Will start in 10 seconds

You may be right that someone with a better understanding of the DLZ module is needed. I have no clue who to ask. Do you have contact information?

[danielluke (Daniel J. Luke)]

ok, I think that that patch change I had you do made it select just the first of the link lines (and we want at least the middle one). How about changing that awk (which is now '{print substr($1,3)}' to '{ if (/mysql5/) print substr ( $1,3 ) }'

just in the two places where it's after $use_dlz_mysql --libs | awk

... then clean and rebuild ... otherwise I we can try to CC the last person who fixed up the dlz patch and ask if he/she is still using it.

[erik@…]

Replying to dluke@…:

ok, I think that that patch change I had you do made it select just the first of the link lines (and we want at least the middle one). How about changing that awk (which is now '{print substr($1,3)}' to '{ if (/mysql5/) print substr ( $1,3 ) }'

just in the two places where it's after $use_dlz_mysql --libs | awk

... then clean and rebuild ... otherwise I we can try to CC the last person who fixed up the dlz patch and ask if he/she is still using it.

Done, same result...

[danielluke (Daniel J. Luke)]

Maybe pixilla can help since I think he provided the last functional change to the patch?

[danielluke (Daniel J. Luke)]

I'm going to remove the broken variant (for now), if/when someone who uses it can provide a working patch I'll be happy to add it back to the port.

[danielluke (Daniel J. Luke)]

r112492