From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Schulte Subject: Re: babel perl issue Date: Fri, 14 Dec 2012 11:13:14 -0700 Message-ID: <87k3skh58l.fsf@gmail.com> References: <87obi193mn.fsf@gmail.com> <87zk1lu53s.fsf@Rainer.invalid> <87zk1kmpyo.fsf@gmail.com> <87r4mwph61.fsf@bzg.ath.cx> <87zk1kf2bw.fsf@gmail.com> <8762472rg8.fsf@bzg.ath.cx> <874njrdzdg.fsf@gmail.com> <87ip871bog.fsf@bzg.ath.cx> <87obhzcjrs.fsf@gmail.com> <87mwxjyzor.fsf@bzg.ath.cx> <87fw3bfb2i.fsf@Rainer.invalid> <87a9tjyyi3.fsf@bzg.ath.cx> <87sj7bb1sh.fsf@gmail.com> <87r4mvxhxe.fsf@bzg.ath.cx> <87txrr9lc8.fsf@gmail.com> <876245ttw5.fsf@bzg.ath.cx> <87hanpa2ps.fsf@gmail.com> <87vcc5ufdp.fsf@bzg.ath.cx> <871uesiq3x.fsf@gmail.com> <87hanole91.fsf@bzg.ath.cx> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([208.118.235.92]:52136) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TjZlK-0000NG-Cl for emacs-orgmode@gnu.org; Fri, 14 Dec 2012 13:13:31 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TjZlF-0001D0-IE for emacs-orgmode@gnu.org; Fri, 14 Dec 2012 13:13:30 -0500 Received: from mail-ia0-f169.google.com ([209.85.210.169]:63208) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TjZlF-0001Ck-Dy for emacs-orgmode@gnu.org; Fri, 14 Dec 2012 13:13:25 -0500 Received: by mail-ia0-f169.google.com with SMTP id r4so3462253iaj.0 for ; Fri, 14 Dec 2012 10:13:24 -0800 (PST) In-Reply-To: <87hanole91.fsf@bzg.ath.cx> (Bastien's message of "Fri, 14 Dec 2012 18:44:58 +0100") 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: Achim Gratz , emacs-orgmode@gnu.org Bastien writes: > Eric Schulte writes: > >> This patch is still broken > > (I don't know why ~$ git format-patch master did not produce an > applicable patch.) > > These three ones are applicable against master/68d4de2. > > Bare in mind this is just meant as a proof it "works", we can > rework the patches later on. These applied and I now see what they are doing. This is equivalent to moving all of the core Babel libraries into a single massive ob.el. I explicitly like the separation of functional units of Babel into separate files, and this sort of refactoring would make Babel harder to maintain in the future. What do you find so unappealing about the existence of ob-core.el to justify such drastic proposals? Perhaps it would be possible to take all of those functions which are defined in ob-core *and* are required by other components of Babel, and move them to an ob-util.el file. Then we could move the remaining public facing functions of Babel which are not used in any of it's components (e.g., org-babel-execute-src-block) and move them into ob.el. Would that feel cleaner to you? Thanks, -- Eric Schulte http://cs.unm.edu/~eschulte