Bug #48655 Incorrect error message when running configure for MySQL Proxy 0.7.2 ('lua5.1')
Submitted: 10 Nov 2009 5:35 Modified: 28 Jun 2010 11:52
Reporter: Roel Van de Paar (OCA) Email Updates:
Status: Closed Impact on me:
None 
Category:MySQL Proxy: Install/Config Severity:S3 (Non-critical)
Version:0.7.2 OS:Any
Assigned to: Jan Kneschke CPU Architecture:Any
Triage: Needs Triage: D4 (Minor)

[10 Nov 2009 5:35] Roel Van de Paar
Description:
Excerpt from ./configure :
----
checking for LUA... no
checking for LUA... configure: error: Package requirements (lua5.1 >= 5.1) were not met:

No package 'lua5.1' found
----

At this point the 'lua5.1' package was installed.

Installing liblua5.1-0 did not make a difference

Installing liblua5.1-0-dev made it work:

----
checking for LUA... no
checking for LUA... yes
----

How to repeat:
Try to compile without liblua5.1-0-dev present

Suggested fix:
Report 'liblua5.1-0-dev' instead of/together with 'lua5.1'
[10 Nov 2009 5:36] Roel Van de Paar
Verifying as D4
[10 Nov 2009 15:46] Enterprise Tools JIRA Robot
Diego Medina writes: 
Hi Roel,

Thanks for your bug report.

I agree that having a detailed message telling you the exact name of the package you need would be very helpful. 
But we do include that information on the INSTALL file.

We are looking into adding a message that one error, it would direct people to the INSTALL file.

Thanks

Diego
[10 Nov 2009 22:26] Roel Van de Paar
Hi Diego,

OK so once there would be an error, people would be directed to the INSTALL file. 

That would be acceptable if there would be no (incorrect) mention of a 'lua5.1' package.

As such, can we leave this bug open until either:

1. The current error message is removed, and people are directed to the INSTALL file instead.

   or (as per the above)

2. The current error message is changed to list the correct package.
[10 Nov 2009 22:27] Roel Van de Paar
Others running into this:

http://www.google.com.au/search?hl=en&num=50&q=%22No+package+%27lua5.1%27+found%22+%2Bprox...
[10 Nov 2009 22:34] Roel Van de Paar
Also, I could not locate this information in the INSTALL file?

roel@roel-ubuntu-vm:/proxy/mysql-proxy-0.7.2$ cat INSTALL | grep lua
roel@roel-ubuntu-vm:/proxy/mysql-proxy-0.7.2$ cat INSTALL | grep '-' | grep dev
roel@roel-ubuntu-vm:/proxy/mysql-proxy-0.7.2$
[25 Nov 2009 15:58] Enterprise Tools JIRA Robot
Mark Leith writes: 
Documented properly in the 0.8.0 INSTALL file
[13 Jan 2010 18:26] Enterprise Tools JIRA Robot
Keith Russell writes: 
Had to re-open in order to move to AIB. jira bug?
[13 Jan 2010 18:27] Enterprise Tools JIRA Robot
Keith Russell writes: 
Patch available in build.
[25 May 2010 13:47] Enterprise Tools JIRA Robot
Jan Kneschke writes: 
revno: 1090
fixes bug(s): http://bugs.mysql.com/48655
committer: jan@mysql.com
branch nick: trunk
timestamp: Tue 2010-05-25 15:44:43 +0200
message:
  updated the error-msg in the lua.pc isn't found (fixes #48655)
  
    without --with-lua (default):
  
    checking which pkg-config file to use to find Lua... lua.pc and lua5.1.pc
    checking pkg-config is at least version 0.9.0... yes
    checking for LUA... yes
   
    if --with-lua=foobar is given:
   
    checking which pkg-config file to use to find Lua... foobar.pc
    checking for LUA... no
    configure: error: checked for Lua via pkg-config: No package 'foobar' found. Make sure lua and its devel-package, which includes the foobar.pc file, is installed
modified:
  configure.in
[25 May 2010 13:48] Enterprise Tools JIRA Robot
Jan Kneschke writes: 
Added a better check in configure that will lead to a better error-msg. Will land in 0.8.1
[21 Jun 2010 18:47] Enterprise Tools JIRA Robot
Diego Medina writes: 
Verified fixed on 0.8.1 

e
[28 Jun 2010 11:52] MC Brown
A note has been added to the 0.8.1 changelog: 

        When running <command>configure</command> to build, the error                                 
        message relating to the <literal>lua</literal> libraries could                                
        be misleading. The wording and build advice have now been                                     
        updated.