Opened 11 years ago

Closed 9 years ago

#40465 closed defect (wontfix)

9.9.3-P2_1 +dlz_mysql fails to build

Reported by: erik@… Owned by: danielluke (Daniel J. Luke)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: pixilla (Bradley Giesbrecht)
Port: bind9

Description

Attached main.log...

Attachments (1)

main.log (314.1 KB) - added by erik@… 11 years ago.

Download all attachments as: .zip

Change History (17)

comment:1 Changed 11 years ago by danielluke (Daniel J. Luke)

Owner: changed from macports-tickets@… to dluke@…
Status: newassigned
Version: 2.2.0

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

comment:2 Changed 11 years ago by danielluke (Daniel J. Luke)

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

comment:3 in reply to:  1 Changed 11 years ago by 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

comment:4 in reply to:  2 Changed 11 years ago by 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...

Last edited 11 years ago by erik@… (previous) (diff)

Changed 11 years ago by erik@…

Attachment: main.log added

comment:5 Changed 11 years ago by 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 ?

comment:6 in reply to:  5 Changed 11 years ago by 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

comment:7 Changed 11 years ago by 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).

comment:8 Changed 11 years ago by 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.

comment:9 Changed 11 years ago by 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?

comment:10 Changed 11 years ago by 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.

comment:11 in reply to:  10 Changed 11 years ago by 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...

comment:12 Changed 11 years ago by danielluke (Daniel J. Luke)

Summary: 9.9.3-P2_0 fails to build9.9.3-P2_1 +dlz_mysql fails to build

comment:13 Changed 11 years ago by danielluke (Daniel J. Luke)

Cc: pixilla@… added

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

comment:14 Changed 10 years ago by 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.

comment:15 Changed 10 years ago by danielluke (Daniel J. Luke)

Last edited 10 years ago by danielluke (Daniel J. Luke) (previous) (diff)

comment:16 Changed 9 years ago by danielluke (Daniel J. Luke)

Resolution: wontfix
Status: assignedclosed
Note: See TracTickets for help on using tickets.