#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.1 → Port Enhancements |
Owner: | changed from macports-tickets@… to ryandesign@… |
Status: | new → assigned |
Summary: | Mysql5 is not support readline → mysql5 readline support |
Type: | defect → enhancement |
comment:2 follow-up: 3 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 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: | assigned → closed |
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
readline functionality is probably useful enough to enable always, not in a variant. I'll look into this.