Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Dwight Engen
On Thu, 5 Sep 2013 08:45:33 +0200 Natanael Copa wrote: > Adjust code for Lua 5.2 and keep compatibility with Lua 5.1. > > We also fix a bug in the parsing of /proc/mounts while here. After some discussion on IRC, we decided the bug fix should be split out. There is also one more change needed

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > On Thu, 5 Sep 2013 08:45:33 +0200 > Natanael Copa wrote: > > > Adjust code for Lua 5.2 and keep compatibility with Lua 5.1. > > > > We also fix a bug in the parsing of /proc/mounts while here. > > After some discussion on IRC, we decided the bu

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Dwight Engen
On Thu, 5 Sep 2013 12:52:34 -0500 Serge Hallyn wrote: > Quoting Dwight Engen (dwight.en...@oracle.com): > > On Thu, 5 Sep 2013 08:45:33 +0200 > > Natanael Copa wrote: > > > > > Adjust code for Lua 5.2 and keep compatibility with Lua 5.1. > > > > > > We also fix a bug in the parsing of /proc/m

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Dwight Engen
On Thu, 5 Sep 2013 15:55:16 +0200 Natanael Copa wrote: > On Thu, 5 Sep 2013 09:34:14 -0400 > Dwight Engen wrote: > > > On Thu, 5 Sep 2013 08:11:57 -0500 > > Serge Hallyn wrote: > > > > > Hi Dwight, > > > > > > do you have time to review/ack these (3, so far) lua patches? > > > > > > (I prob

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Natanael Copa
On Thu, 5 Sep 2013 09:34:14 -0400 Dwight Engen wrote: > On Thu, 5 Sep 2013 08:11:57 -0500 > Serge Hallyn wrote: > > > Hi Dwight, > > > > do you have time to review/ack these (3, so far) lua patches? > > > > (I probably don't know enough to do so) > > Yeah I've taken a look. I had already don

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Natanael Copa
On Thu, 5 Sep 2013 08:11:57 -0500 Serge Hallyn wrote: > Hi Dwight, > > do you have time to review/ack these (3, so far) lua patches? I send 2 more patches, to get rid of fork/exec. (os.execute and io.popen) They should all be available from https://github.com/ncopa/lxc Thanks! -nc -

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Dwight Engen
On Thu, 5 Sep 2013 08:11:57 -0500 Serge Hallyn wrote: > Hi Dwight, > > do you have time to review/ack these (3, so far) lua patches? > > (I probably don't know enough to do so) Yeah I've taken a look. I had already done an update similar to the first two and was just getting ready to post afte

Re: [lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-05 Thread Serge Hallyn
Hi Dwight, do you have time to review/ack these (3, so far) lua patches? (I probably don't know enough to do so) thanks, -serge Quoting Natanael Copa (nc...@alpinelinux.org): > Adjust code for Lua 5.2 and keep compatibility with Lua 5.1. > > We also fix a bug in the parsing of /proc/mounts whi

[lxc-devel] [PATCH] lua: prepare for Lua 5.2

2013-09-04 Thread Natanael Copa
Adjust code for Lua 5.2 and keep compatibility with Lua 5.1. We also fix a bug in the parsing of /proc/mounts while here. Signed-off-by: Natanael Copa --- src/lua-lxc/core.c | 10 -- src/lua-lxc/lxc.lua | 18 ++ 2 files changed, 22 insertions(+), 6 deletions(-) diff --