From mboxrd@z Thu Jan 1 00:00:00 1970 From: Satoru KURASHIKI Subject: Re: Two ob-lua versions Date: Wed, 14 Feb 2018 10:28:12 +0900 Message-ID: References: <87eflsduo9.fsf@dell-desktop.WORKGROUP> <877erjdgom.fsf@nicolasgoaziou.fr> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:32881) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ellrs-0001Yb-75 for emacs-orgmode@gnu.org; Tue, 13 Feb 2018 20:28:17 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ellrq-00023i-JK for emacs-orgmode@gnu.org; Tue, 13 Feb 2018 20:28:15 -0500 Received: from mail-yw0-x22d.google.com ([2607:f8b0:4002:c05::22d]:41016) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ellrq-00022r-Du for emacs-orgmode@gnu.org; Tue, 13 Feb 2018 20:28:14 -0500 Received: by mail-yw0-x22d.google.com with SMTP id f12so4894072ywb.8 for ; Tue, 13 Feb 2018 17:28:14 -0800 (PST) In-Reply-To: <877erjdgom.fsf@nicolasgoaziou.fr> 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: emacs-org list hi, On Sun, Feb 11, 2018 at 6:22 PM, Nicolas Goaziou wrote: > Hello, > > Thibault Marin writes: > >> I noticed that a recent change added an ob-lua.el file to contrib/lisp/. >> Since there is already an ob-lua.el in lisp/, I was wondering if the new >> version was adding functionality not supported by the one in lisp/ (it >> does not seem to be the case on casual inspection) and if so, whether >> they could/should be merged. > > I just realized that, too. It seems wrong. I'm Cc'ing stardiviner so we > can have some insight about it. Anyway this breaks installation with package.el in org-plus-contrib case. because of conflicting "ob-lua.el" filename. > stardiviner, would you mind explaining why you need to create an > "ob-lua.el" library in "contrib/" ? It has to be fixed, so please revert that merge, until stardiviner will notice this thread and retry PR... regards, -- KURASHIKI Satoru