Re: ob-lua

[Nicolas Goaziou <mail@nicolasgoaziou.fr>]

Hello,

Thibault Marin <thibault.marin@gmx.com> 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