Bug #57245 package.path has wrong default on win32

• Submitted: 5 Oct 2010 12:06
• Modified: 12 Oct 2010 8:18
• Reporter: Jan Kneschke
• Status: Closed
• Category: MySQL Proxy: Core
• Severity: S3 (Non-critical)
• Version: 0.8.1
• OS: Windows
• Assigned to: Jan Kneschke
• CPU Architecture: Any

[5 Oct 2010 12:06] Jan Kneschke

Description:
The "package.path" used by lua to find its modules is incorrect on win32:

$ mysql-proxy.exe --version
[...]
  LUA: Lua 5.1.4
    package.path: <somepath>\lib

It should be the same as on Unix:

  LUA: Lua 5.1.4
    package.path: <somepath>/lib/mysql-proxy/lua/?.lua

How to repeat:
Run:

$ mysql-proxy --version

and check that the package.path is correct:

$ mysql-proxy.exe --version
[...]
  LUA: Lua 5.1.4
    package.path: <somepath>\lib\mysql-proxy\lua\?.lua

[11 Oct 2010 10:13] Carsten Segieth

still present with agent build 2.3.0.2032

[11 Oct 2010 11:41] Carsten Segieth

# Windows:

$ bin/mysql-proxy.exe --version
mysql-proxy 0.8.1
  chassis: mysql-proxy 0.8.1
  glib2: 2.20.3
  libevent: 1.4.13-stable
  LUA: Lua 5.1.4
    package.path: D:\Merlin\inst\agent\2.3.0.2032\winxp\IBM\lib
    package.cpath: D:\Merlin\inst\agent\2.3.0.2032\winxp\IBM\bin\lua-?.dll
-- modules
  admin: 0.8.1
  proxy: 0.8.1

# Linux 32bit:

$ bin/mysql-proxy --version
0.8.1.2032-mysql-proxy-0.8.1r1131_20100920_1739.tar.gz
  chassis: 0.8.1.2032-mysql-proxy-0.8.1r1131_20100920_1739.tar.gz
  glib2: 2.16.6
  libevent: 1.4.13-stable
  LUA: Lua 5.1.4
    package.path: /data0/merlin/agent/2.3.0.2032/rhel4-x86-32bit/net-qa1/lib/mysql-proxy/lua/?.lua;
    package.cpath: /data0/merlin/agent/2.3.0.2032/rhel4-x86-32bit/net-qa1/lib/mysql-proxy/lua/?.so;
-- modules
  admin: 0.8.1
  proxy: 0.8.1

[11 Oct 2010 15:41] Jan Kneschke

As the chassis-version says the agent was built against a chassis r1131, the fix is in r1141.

[11 Oct 2010 15:44] Enterprise Tools JIRA Robot

Jan Kneschke writes: 
Pushed to 0.8 as

revno: 1133
fixes bug(s): http://bugs.mysql.com/57245
committer: Jan Kneschke <[e-mail]>
branch nick: rel-0.8
timestamp: Tue 2010-10-05 14:11:34 +0200
message:
  fixed broken default LUA_PATH on win32 (#fixes 57245)
  
    * added a failing test-case
    * join the path parts for LUA_PATH as we do on Unix, only assume one-folder for the lua-cpath

and trunk

revno: 1147
committer: Jan Kneschke <[e-mail]>
branch nick: trunk
timestamp: Tue 2010-10-05 14:16:58 +0200
message:
  merged fix for #57245 from the 0.8 tree