Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#15435 closed enhancement (fixed)

mysql5 readline support

Reported by: n0ts (Naoya Nakazawa) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Low Milestone:
Component: ports Version: 1.6.0
Keywords: Cc: jyrkiwahlstedt, MarcusCalhoun-Lopez (Marcus Calhoun-Lopez), n0ts (Naoya Nakazawa)
Port: mysql5, mysql5-devel

Description (last modified by ryandesign (Ryan Carsten Schmidt))

Mysql5 is not support readline.

I found below the patch. This patch is added variant readline.

--- /opt/local/var/macports/sources/rsync.macports.org/release/ports/databases/mysql5/Portfile.orig     2008-05-05 13:56:36.000000000 +0900
+++ /opt/local/var/macports/sources/rsync.macports.org/release/ports/databases/mysql5/Portfile  2008-05-26 15:53:07.000000000 +0900
@@ -68,6 +68,12 @@
     startupitem.stop    "${prefix}/share/${mysql}/mysql/mysql.server stop"
 }
 
+variant readline {
+    # enable readline
+    depends_lib-append  port:readline
+    configure.args-append  --with-readline
+}
+
 pre-destroot {
     # Some directories we must have in all cases
     xinstall -m 755 -d ${destroot}${sysconfdir}

Change History (10)

comment:1 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: jwa@… added; ryandesign@… removed
Milestone: MacPorts 1.6.1Port Enhancements
Owner: changed from macports-tickets@… to ryandesign@…
Status: newassigned
Summary: Mysql5 is not support readlinemysql5 readline support
Type: defectenhancement

readline functionality is probably useful enough to enable always, not in a variant. I'll look into this.

comment:2 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

After a little investigation ("./configure --help"), it looks to me as though "--with-readline" is the default, for mysql3, mysql4 and mysql5, and means "use the readline library included with MySQL". Alternately, you can use "--without-readline" which means "don't use the included readline but use the system readline" which in our case would mean "use the MacPorts readline". We could do that, but would that provide any advantage? I'm also curious how you arrived at the conclusion that "Mysql5 is not support readline" and whether the +readline variant you supplied actually does anything.

comment:3 in reply to:  2 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Dave Feldman sent me this reply via email:

In fact, I did experience problems with mysql5 installed from macports. While emacs keybindings appeared to work fine, vi keybindings did not work at all despite being enabled in my .inputrc. When I modified my Portfile to add --with-readline and reinstalled mysql5, vi keybindings worked perfectly fine.

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

Description: modified (diff)
Port: mysql5 mysql5-devel added

comment:5 Changed 16 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Cc: mcalhoun@… added

Cc Me!

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

Cc: naoya.n@… added

Has duplicate #19030, in which the reporter states that adding --without-readline (i.e. without the built-in readline but with the system's (i.e. MacPorts') readline) made readline work, as I had supposed above.

comment:7 Changed 16 years ago by n0ts (Naoya Nakazawa)

I'm sorry for post duplicate ticket and thank you for your comment. My previous patch is adding variants, but I think that adding --without-readline configuration option is good way.

regards

comment:8 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Oh, I didn't even realize you were the same person reporting both issues. Well it's easy to forget after 10 months. My apologies for letting this ticket sit so long. I think the reason is that I don't know how to use readline, so I don't know how to verify that the fix works.

comment:9 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: assignedclosed

Should be fixed in r48916. I was at least able to verify that MySQL's parts were not linked with the readline libraries before the change, and now they are.

comment:10 Changed 16 years ago by (none)

Milestone: Port Enhancements

Milestone Port Enhancements deleted

Note: See TracTickets for help on using tickets.