From: Josef 'J. S. <je...@jo...> - 2014-08-04 02:16:27
|
Hello, It's been a while and so I tried to recompile Notion (latest tarball) on my OpenIndiana laptop. After the usual tweaking to make the build happy on a SunOS-based system, I attempted to start it via gdm. It failed with the following in .xsession-errors. Any ideas what's wrong? I moved my ~/.notion out of the way and it still breaks. /etc/gdm/Xsession: Setup done, will execute: /usr/bin/ssh-agent -- /usr/bin/notion 2014-08-03 14:44:27 INFO /notion/../ioncore.c:609: ioncore_startup: Starting Notion 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 12-*-*-*-*-*-*-*: 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 14-*-*-*-*-*-*-*: 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 17-*-*-*-*-*-*-*: 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 9 missing charsets for -misc-fixed-medium-r-*-*-13-*-*- *-*-60-*-*: >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) >> Stack trace: 0 [C]: in 'load_module' 1 mod_tiling.lua:17 [Skipping unnamed C functions.] 4 [C]: in 'dopath' 5 /usr/etc/notion/cfg_defaults.lua:11 [Skipping unnamed C functions.] 8 [C]: in 'dopath' 9 /usr/etc/notion/cfg_notion.lua:95 [Skipping unnamed C functions.] >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) >> Stack trace: 0 [C]: in 'load_module' 1 stdin:1 [Skipping unnamed C functions.] 4 [C]: in 'dopath' 5 /usr/etc/notion/cfg_defaults.lua:13 [Skipping unnamed C functions.] 8 [C]: in 'dopath' 9 /usr/etc/notion/cfg_notion.lua:95 [Skipping unnamed C functions.] >> mod_xinerama.lua:179: attempt to call field 'maxn' (a nil value) >> Some bindmaps were empty, loading ioncore_efbb. >> Making the following minimal emergency mappings: F2 -> xterm F11 -> restart F12 -> exit Mod1+C -> close Mod1+K P/N -> WFrame.switch_next/switch_prev Thanks, Jeff. -- Don't drink and derive. Alcohol and algebra don't mix. |
From: Josef 'J. S. <je...@jo...> - 2014-08-04 11:13:15
|
On Sun, Aug 03, 2014 at 09:51:16PM -0400, Josef 'Jeff' Sipek wrote: > Hello, > > It's been a while and so I tried to recompile Notion (latest tarball) on my > OpenIndiana laptop. Ok, more information. Yesterday, I reinstalled my laptop with a newer version of OpenIndiana. This morning, I tried to recompile the old version (2013030200), and it also fails. So, chances are that the issue is somewhere in the system. (FWIW, previously I had Lua 5.1.4, now I have 5.2.2.) Any ideas what changes on the system would cause this sort of failure? Thanks, Jeff. > After the usual tweaking to make the build happy on a > SunOS-based system, I attempted to start it via gdm. It failed with the > following in .xsession-errors. Any ideas what's wrong? I moved my > ~/.notion out of the way and it still breaks. > > /etc/gdm/Xsession: Setup done, will execute: /usr/bin/ssh-agent -- /usr/bin/notion > 2014-08-03 14:44:27 INFO /notion/../ioncore.c:609: ioncore_startup: Starting Notion > 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 12-*-*-*-*-*-*-*: > 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 14-*-*-*-*-*-*-*: > 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 11 missing charsets for -*-helvetica-medium-r-normal-*- 17-*-*-*-*-*-*-*: > 2014-08-03 14:44:27 INFO /notion/../fontset.c:118: de_create_font_in_current_locale: Found a font with 9 missing charsets for -misc-fixed-medium-r-*-*-13-*-*- *-*-60-*-*: > >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) > >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) > >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) > >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) > >> Stack trace: > 0 [C]: in 'load_module' > 1 mod_tiling.lua:17 > [Skipping unnamed C functions.] > 4 [C]: in 'dopath' > 5 /usr/etc/notion/cfg_defaults.lua:11 > [Skipping unnamed C functions.] > 8 [C]: in 'dopath' > 9 /usr/etc/notion/cfg_notion.lua:95 > [Skipping unnamed C functions.] > >> ioncore_bindings.lua:72: attempt to call global 'loadstring' (a nil value) > >> Stack trace: > 0 [C]: in 'load_module' > 1 stdin:1 > [Skipping unnamed C functions.] > 4 [C]: in 'dopath' > 5 /usr/etc/notion/cfg_defaults.lua:13 > [Skipping unnamed C functions.] > 8 [C]: in 'dopath' > 9 /usr/etc/notion/cfg_notion.lua:95 > [Skipping unnamed C functions.] > >> mod_xinerama.lua:179: attempt to call field 'maxn' (a nil value) > >> Some bindmaps were empty, loading ioncore_efbb. > >> Making the following minimal emergency mappings: > F2 -> xterm > F11 -> restart > F12 -> exit > Mod1+C -> close > Mod1+K P/N -> WFrame.switch_next/switch_prev > > Thanks, > > Jeff. > > -- > Don't drink and derive. Alcohol and algebra don't mix. > > ------------------------------------------------------------------------------ > Infragistics Professional > Build stunning WinForms apps today! > Reboot your WinForms applications with our WinForms controls. > Build a bridge from your legacy apps to the future. > http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk > _______________________________________________ > Notion-devel mailing list > Not...@li... > https://lists.sourceforge.net/lists/listinfo/notion-devel -- *NOTE: This message is ROT-13 encrypted twice for extra protection* |
From: Josef 'J. S. <je...@jo...> - 2014-08-04 12:37:22
|
On Mon, Aug 04, 2014 at 07:13:07AM -0400, Josef 'Jeff' Sipek wrote: > On Sun, Aug 03, 2014 at 09:51:16PM -0400, Josef 'Jeff' Sipek wrote: > > Hello, > > > > It's been a while and so I tried to recompile Notion (latest tarball) on my > > OpenIndiana laptop. > > Ok, more information. Yesterday, I reinstalled my laptop with a newer > version of OpenIndiana. This morning, I tried to recompile the old version > (2013030200), and it also fails. So, chances are that the issue is > somewhere in the system. (FWIW, previously I had Lua 5.1.4, now I have 5.2.2.) > > Any ideas what changes on the system would cause this sort of failure? Alright. I got it all figured out. It all stems from incompatibilies in Lua 5.2.2 - especially if it is compiled without legacy support. Here's the patch that works for me. You'll probably want to make it conditional to keep 5.1 happy. Anyway, thanks for listening ;) Sometimes, just having to describe your problems leads to a solution :) Thanks, Jeff. diff -ur notion-3-2014052800.mangled/contrib/scripts/simple_bindings.lua notion-3-2014052800/contrib/scripts/simple_bindings.lua --- notion-3-2014052800.mangled/contrib/scripts/simple_bindings.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/contrib/scripts/simple_bindings.lua 2014-08-04 03:54:53.380883374 -0400 @@ -54,7 +54,7 @@ if not ioncore then ioncore = {get_paths = function() return {userdir="/home/deryni/.ion3"} end} warn = print - loadstring = function(str) return function() print(str) end end + load = function(str) return function() print(str) end end end --}}} Standalone hacks @@ -164,7 +164,7 @@ str = str.."\tkpress(\""..bind.keystr..[[", "]]..bind.bindstr.."\"),\n" end str = str.."})" - loadstring(str)() + load(str)() end -- vim: set expandtab sw=4: diff -ur notion-3-2014052800.mangled/contrib/statusbar/statusbar_external.lua notion-3-2014052800/contrib/statusbar/statusbar_external.lua --- notion-3-2014052800.mangled/contrib/statusbar/statusbar_external.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/contrib/statusbar/statusbar_external.lua 2014-08-04 03:54:53.381313664 -0400 @@ -209,7 +209,7 @@ -- Start all for key in pairs(statusbar_external) do timers[key] = ioncore.create_timer() - callbacks[key] = loadstring("start_execute("..key..")") + callbacks[key] = load("start_execute("..key..")") start_execute(key) end diff -ur notion-3-2014052800.mangled/contrib/statusd/legacy/statusd_sysmon.lua notion-3-2014052800/contrib/statusd/legacy/statusd_sysmon.lua --- notion-3-2014052800.mangled/contrib/statusd/legacy/statusd_sysmon.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/contrib/statusd/legacy/statusd_sysmon.lua 2014-08-04 03:54:53.381656408 -0400 @@ -185,7 +185,7 @@ return (metrics[arg1] or "") end) sysmon_st = string.gsub (sysmon_st, "%$%{(.-)%}", function (arg1) - return loadstring("return "..arg1)() + return load("return "..arg1)() end) -- replacing the '%%' macros with the '%' symbol sysmon_st = string.gsub (sysmon_st, "%%%%", "%%") diff -ur notion-3-2014052800.mangled/ioncore/ioncore_bindings.lua notion-3-2014052800/ioncore/ioncore_bindings.lua --- notion-3-2014052800.mangled/ioncore/ioncore_bindings.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/ioncore/ioncore_bindings.lua 2014-08-04 03:54:53.382014910 -0400 @@ -47,7 +47,7 @@ local gfncode="return function(_, _sub, _chld) "..guardcode.." return true end" local gerr - gfn, gerr=loadstring(gfncode, guardcode) + gfn, gerr=load(gfncode, guardcode) if not gfn then ioncore.warn_traced(TR("Error compiling guard: %s", gerr)) end @@ -69,7 +69,7 @@ if type(cmd)=="string" then local fncode=("return function(_, _sub, _chld) local d = " ..cmd.." end") - local fn, err=loadstring(fncode, cmd) + local fn, err=load(fncode, cmd) if not fn then ioncore.warn_traced(TR("Error in command string: ")..err) return diff -ur notion-3-2014052800.mangled/mod_query/mod_query.lua notion-3-2014052800/mod_query/mod_query.lua --- notion-3-2014052800.mangled/mod_query/mod_query.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/mod_query/mod_query.lua 2014-08-04 03:54:53.382604012 -0400 @@ -1048,7 +1048,7 @@ end err=collect_errors(f) else - local f, err=loadstring(code) + local f, err=load(code) if not f then mod_query.warn(mplex, err) return diff -ur notion-3-2014052800.mangled/mod_xinerama/mod_xinerama.lua notion-3-2014052800/mod_xinerama/mod_xinerama.lua --- notion-3-2014052800.mangled/mod_xinerama/mod_xinerama.lua 2014-05-27 18:00:18.000000000 -0400 +++ notion-3-2014052800/mod_xinerama/mod_xinerama.lua 2014-08-04 04:24:22.371015820 -0400 @@ -138,6 +138,17 @@ return x_in and y_in end +-- table.maxn doesn't exist in lua 5.2 +local function table_maxn(t) + local mn = 0 + for k, v in pairs(t) do + if mn < k then + mn = k + end + end + return mn +end + --DOC -- Merges overlapping screens. I.e. it finds set of smallest rectangles, -- such that these rectangles do not overlap and such that they contain @@ -176,8 +187,8 @@ end end end - if not pos then pos = table.maxn(ret)+1 end - table.insert(ret, pos, newscreen) + if not pos then pos = table_maxn(ret)+1 end + ret[pos] = newscreen end fix_representations(ret) return ret @@ -276,7 +287,7 @@ -- pos keeps index of first set that we merged in this loop, -- we want to insert the product of this merge to pos. - if not pos then pos = table.maxn(screensets)+1 end + if not pos then pos = table_maxn(screensets)+1 end table.insert(screensets, pos, mergedset) end -- Once you have their hardware. Never give it back. (The First Rule of Hardware Acquisition) |
From: Arnout E. <no...@bz...> - 2014-12-16 20:56:11
|
On Mon, Aug 4, 2014 at 2:37 PM, Josef 'Jeff' Sipek <je...@jo...> wrote: > > It all stems from incompatibilies in Lua 5.2.2 - especially if it is > compiled without legacy support. Here's the patch that works for me. > You'll probably want to make it conditional to keep 5.1 happy. Tough one. I really want to avoid adding more compile-time switches, things are too complicated to my taste already. Unfortunately, this means we have the choice: a) dropping support for lua5.1 b) not support lua5.2 unless it was compiled with LUA_COMPAT_ALL Neither are very appealing. Given 'b' is the status quo, I think we should stick with that for now. Kind regards, Arnout |
From: Josef 'J. S. <je...@jo...> - 2014-12-16 21:57:21
|
On Tue, Dec 16, 2014 at 09:48:48PM +0100, Arnout Engelen wrote: > On Mon, Aug 4, 2014 at 2:37 PM, Josef 'Jeff' Sipek <je...@jo...> > wrote: > > > > It all stems from incompatibilies in Lua 5.2.2 - especially if it is > > compiled without legacy support. Here's the patch that works for me. > > You'll probably want to make it conditional to keep 5.1 happy. > > > Tough one. I really want to avoid adding more compile-time switches, things > are too complicated to my taste already. > > Unfortunately, this means we have the choice: > > a) dropping support for lua5.1 > b) not support lua5.2 unless it was compiled with LUA_COMPAT_ALL > > Neither are very appealing. Given 'b' is the status quo, I think we should > stick with that for now. I totally understand. It does look like Lua 5.1 is getting a bit old, while Lua 5.2 is three years old. (It looks like 5.3 will be released in the near future.) http://www.lua.org/versions.html I don't really know how popular the two versions are. Given a quick look at the Debian popcon data, 5.1 is a whole lot more popular than 5.2. (Of course, just because something is installed doesn't mean that it used.) https://qa.debian.org/popcon.php?package=lua5.1 https://qa.debian.org/popcon.php?package=lua5.2 Either way, it's not the end of the world. The patch I have works well enough. :) Hrm, I'm thinking more about this and I wonder of option (c) makes the most sense given that 5.1 won't be viable forever: c) like (b) but start thinking about the future direction Then there are of course more radical options: d) replace the existing build system with something more standard (autotools, or cmake) Jeff. -- Evolution, n.: A hypothetical process whereby infinitely improbable events occur with alarming frequency, order arises from chaos, and no one is given credit. |
From: Arnout E. <no...@bz...> - 2014-12-17 09:30:18
|
On Tue, Dec 16, 2014 at 10:38 PM, Josef 'Jeff' Sipek <je...@jo...> wrote: > > > I really want to avoid adding more compile-time switches, things > > are too complicated to my taste already. > > > > a) dropping support for lua5.1 > > b) not support lua5.2 unless it was compiled with LUA_COMPAT_ALL > > It does look like Lua 5.1 is getting a bit old, while > Lua 5.2 is three years old. (It looks like 5.3 will be released in the > near > future.) http://www.lua.org/versions.html > > Hrm, I'm thinking more about this and I wonder of option (c) > makes the most sense given that 5.1 won't be viable forever: > > c) like (b) but start thinking about the future direction > When 5.3 comes out we should update notion to work with that, and when necessary that could be a good moment to drop support for 5.1. > Then there are of course more radical options: > > d) replace the existing build system with something more standard > (autotools, or cmake) > > I'm not sure how those would help in this case. I personally dislike autotools (esp. with all the different incompatible versions out there). cmake seems pretty nice, I'd really like to see if that could be applied to notion. Kind regards, Arnout |
From: Josef 'J. S. <je...@jo...> - 2014-12-17 13:32:25
|
On Wed, Dec 17, 2014 at 10:04:59AM +0100, Arnout Engelen wrote: > On Tue, Dec 16, 2014 at 10:38 PM, Josef 'Jeff' Sipek <je...@jo...> > wrote: > > > > > I really want to avoid adding more compile-time switches, things > > > are too complicated to my taste already. > > > > > > a) dropping support for lua5.1 > > > b) not support lua5.2 unless it was compiled with LUA_COMPAT_ALL > > > > It does look like Lua 5.1 is getting a bit old, while > > Lua 5.2 is three years old. (It looks like 5.3 will be released in the > > near > > future.) http://www.lua.org/versions.html > > > > Hrm, I'm thinking more about this and I wonder of option (c) > > makes the most sense given that 5.1 won't be viable forever: > > > > c) like (b) but start thinking about the future direction > > > > When 5.3 comes out we should update notion to work with that, and when > necessary that could be a good moment to drop support for 5.1. Fair enough. > > Then there are of course more radical options: > > > > d) replace the existing build system with something more standard > > (autotools, or cmake) > > I'm not sure how those would help in this case. You said you didn't want to add another compile time option. With either autotools or cmake you can make the "configure" step of the build auto-detect which version of lua you have and whether or not you have 5.1 compat features available. > I personally dislike autotools (esp. with all the different incompatible > versions out there). cmake seems pretty nice, I'd really like to see if > that could be applied to notion. Agreed on both points. FWIW, once upon a time, I tried to cmake-ify notion. Sadly, I ran out of time then and then never got back to it. Here's the latest change I have. Maybe you'll find it useful. :) http://repo.or.cz/w/notion/jeffpc.git/commit/4187ec60d46d6a8a62c1abd7202a6a9bc263a7c0 Jeff. -- Linux, n.: Generous programmers from around the world all join forces to help you shoot yourself in the foot for free. |