From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Schulte Subject: Re: [PATCH] Use geiser for babel scheme evaluation. Date: Sun, 30 Jun 2013 18:10:21 -0600 Message-ID: <87ppv3jg2a.fsf@gmail.com> References: <18385.1372477848@greg-minshalls-mbp.local> <87ppv3nr8c.fsf@bzg.ath.cx> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:52809) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UtRin-00020H-GH for emacs-orgmode@gnu.org; Sun, 30 Jun 2013 20:11:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UtRim-0004FP-FL for emacs-orgmode@gnu.org; Sun, 30 Jun 2013 20:11:57 -0400 In-Reply-To: <87ppv3nr8c.fsf@bzg.ath.cx> (Bastien's message of "Mon, 01 Jul 2013 00:55:31 +0200") 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-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Bastien Cc: emacs-orgmode@gnu.org, Michael Gauland , Greg Minshall Bastien writes: > Hi Greg, > > Greg Minshall writes: > >> here is the thread: >> http://comments.gmane.org/gmane.emacs.orgmode/64229 > > Thanks. > > I'm reattaching the patch, with a reworked ChangeLog. > > (We could not apply it at the time because Michael didn't > have his copyright assignment.) > > I like geiser very much, but maybe this introduces too much > external dependencies. > > Eric, do you think we can go ahead and apply it? > I've just applied this patch. I don't think external dependencies are a problem if they offload language integration work to a dedicated external package. The more babel can re-use existing packages the better, e.g., common lisp code blocks are just thin wrappers around slime, but they work exceedingly well with almost no babel-side coding or maintenance. Thanks Greg for this patch! > > PS: The patch does not apply, we need to revert > http://orgmode.org/cgit.cgi/org-mode.git/commit/?id=92e327 > first. Thanks for figuring out this conflict. -- Eric Schulte http://cs.unm.edu/~eschulte