From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Goaziou Subject: Re: ob-lua Date: Sat, 20 Aug 2016 18:02:13 +0200 Message-ID: <87h9af1la2.fsf@saiph.selenimh> References: <87bn0of273.fsf@dell-desktop.WORKGROUP> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:47638) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bb8iw-0007aE-Pc for emacs-orgmode@gnu.org; Sat, 20 Aug 2016 12:02:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bb8is-0003hU-Lh for emacs-orgmode@gnu.org; Sat, 20 Aug 2016 12:02:17 -0400 Received: from relay4-d.mail.gandi.net ([217.70.183.196]:35718) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bb8is-0003hC-EY for emacs-orgmode@gnu.org; Sat, 20 Aug 2016 12:02:14 -0400 In-Reply-To: <87bn0of273.fsf@dell-desktop.WORKGROUP> (Thibault Marin's message of "Sat, 20 Aug 2016 00:18:24 -0500") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: Thibault Marin Cc: "emacs-orgmode@gnu.org" Hello, Thibault Marin writes: > I was looking for lua support in source blocks and I came across this > discussion: > https://lists.gnu.org/archive/html/emacs-orgmode/2014-05/msg01149.html. It > seems that this was never merged, but I am not sure why. It looks like Bastien was waiting for a following patch, which never happened. > Trying to use the ob-lua.el file linked, it appears to be working for my task (I > just need to be able to pass variables to a lua block). I only had to make a > small change to the file to get it to work, as `org-babel-get-header' is no > longer available: > > 132c132 > < (mapcar #'cdr (org-babel-get-header params :var)))) > --- >> (remove nil (mapcar (lambda (x) (when (eq (car x) :var) (cdr x))) params)))) > > There may be a better way to do it, but it seems to work. In this case, `org-babel-get-header' should be replaced with `org-babel--get-vars', per ORG-NEWS. > So my question is: could this be considered for a merge? The code does > not seem to support sessions, I am not sure if that should be > a blocker. It isn't a blocker, indeed. Could you send an updated patch? A set of tests would be nice, too. Thank you. Regards, -- Nicolas Goaziou