[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure
From: |
Aaron Ecay |
Subject: |
Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure |
Date: |
Fri, 18 Nov 2016 15:03:55 +0000 |
User-agent: |
Notmuch/0.22+21~g7e6e23c (https://notmuchmail.org) Emacs/25.1.50.2 (x86_64-unknown-linux-gnu) |
Hi Nicolas, hi all,
2016ko azaroak 17an, Nicolas Goaziou-ek idatzi zuen:
>
> Hello,
>
> Frederick Giasson <address@hidden> 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
- [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/07
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/10
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/10
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/14
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/15
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/17
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/17
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure,
Aaron Ecay <=
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/18
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/18
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/18
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/19
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/18
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Frederick Giasson, 2016/11/18
- Re: [O] [PATCH] New header parameter :show-process for Org-babel-clojure, Nicolas Goaziou, 2016/11/18