try to run in Lua 5.2

[supipd]

Hi,
I tried run lua-inspect in Lua 5.2.0 (compiled with LUA_COMPAT_ALL) and had errors (getfenv,setfenv).

I made some changes, and it seems, that goes -> C:\JS_LUA\lua-5.2.0\Debug\lua52.exe test.lua successfully produces examples.html
I'm sure (Lua newbie), that my changes are not fully correct, if somebody know, how do things correctly, please report.

...\metalualib\metalua\base.lua (only 1 row chaged):

....
row 9:   metalua={} -- if not metalua then rawset(getfenv(), 'metalua', { }) end
....

...\luainspect (version dependant empty getfenv/setfenv function added):

#!/usr/bin/env lua

-- Set Lua library paths.
-- based on findbin -- https://gist.github.com/1342365
-- and lib -- https://gist.github.com/1342319
local function findbin()
  local script = arg and arg[0] or ''
  local bin = script:gsub('[/\\]?[^/\\]+$', '') -- remove file name
  if bin == '' then bin = '.' end
  return bin
end
local bin = findbin()
package.path = package.path..';'..bin..'/metalualib/?.lua'
package.path = package.path..';'..bin..'/luainspectlib/?.lua'

-- added for dummy (in Lua 5.2 not exists) functions
print (_VERSION)
if _VERSION=="Lua 5.2" then
    function getfenv()
    end
    function setfenv() 
    end
    function debug.getfenv() 
    end
    function debug.setfenv() 
    end
end

-- print (getfenv,setfenv,debug.getfenv,debug.setfenv)
require 'luainspect.command'

Test output:

C:\JS_LUA\davidm-lua-inspect>C:\JS_LUA\lua-5.2.0\Debug\lua52.exe test.lua
Lua 5.2
warning: [string " context.apply_value('shape$', {color='?', si..."]:1: attempt to index global 'context' (a nil value) :   context.apply_value('shape$', {color
='?', size={x=1,y=1}, f=function() return "?" end})
warning: [string " context.apply_value('^n.*', number)"]:1: attempt to index global 'context' (a nil value) :   context.apply_value('^n.*', number)
warning: [string " context.apply_value('^s.*', string)"]:1: attempt to index global 'context' (a nil value) :   context.apply_value('^s.*', string)
warning: [string " context.apply_value('^b.*', boolean)"]:1: attempt to index global 'context' (a nil value) :   context.apply_value('^b.*', boolean)
status: loading:math
warning: module not found: math
output written to examples.html

C:\JS_LUA\davidm-lua-inspect>lua test.lua
Lua 5.1
status: loading:math
warning: module not found: math
output written to examples.html

[davidm]

Thanks, getfenv has been removed as requested.

I'll have to consider further how best to support mixing 5.1 and 5.2 (e.g. editing 5.1 code using a 5.2 interpreter and vice versa). Your mock functions are a quick workaround though but could confuse other libraries that may attempt to use these functions. Work needs to be done in signatures.lua to add 5.2 signatures, and we need to incorporated the latest metalua, which has a 5.2 lexer.

[felixdae]

do you plan to support lua 5.3?