From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Ecay Subject: Re: [PATCH] New header parameter :show-process for Org-babel-clojure Date: Fri, 18 Nov 2016 15:03:55 +0000 Message-ID: <87k2c0n82s.fsf@gmail.com> References: <036bf007-5e9c-310c-5bf8-0660c1b96a5c@fgiasson.com> <87lgwrgy7z.fsf@nicolasgoaziou.fr> <794c93aa-e2dc-2bfb-537e-0c89d8d34cea@fgiasson.com> <878tslp4ha.fsf@nicolasgoaziou.fr> <27089048-9dab-747f-51dc-20006717866e@fgiasson.com> <87inrlogee.fsf@nicolasgoaziou.fr> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:41687) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c7khv-0007nw-AO for emacs-orgmode@gnu.org; Fri, 18 Nov 2016 10:04:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c7khq-0005Hj-BI for emacs-orgmode@gnu.org; Fri, 18 Nov 2016 10:04:03 -0500 Received: from mail-wm0-x243.google.com ([2a00:1450:400c:c09::243]:34676) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1c7khq-0005Go-4s for emacs-orgmode@gnu.org; Fri, 18 Nov 2016 10:03:58 -0500 Received: by mail-wm0-x243.google.com with SMTP id g23so7077125wme.1 for ; Fri, 18 Nov 2016 07:03:57 -0800 (PST) In-Reply-To: <87inrlogee.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: Nicolas Goaziou , Frederick Giasson Cc: emacs-orgmode@gnu.org Hi Nicolas, hi all, 2016ko azaroak 17an, Nicolas Goaziou-ek idatzi zuen: > > Hello, > > Frederick Giasson writes: > >> I am still waiting for FSF's signature. Should come in soon I guess. > > I applied your patch on a local branch, but compilation issues the > following warnings: > > In toplevel form: > ob-clojure.el:86:1:Warning: Unused lexical variable `nrepl-sync-request-timeout' > > In org-babel-execute:clojure: > ob-clojure.el:149:34:Warning: reference to free variable > `org-babel-clojure-sync-nrepl-timeout' It looks like this warning results from a defcustom for this variable not being included in the patch (and AFAICT it should be). -- Aaron Ecay