Coder Social home page Coder Social logo

Comments (6)

macalinao avatar macalinao commented on July 29, 2024

Found this:

https://lists.nongnu.org/archive/html/emacs-orgmode/2016-06/msg00106.html

from ob-go.

i4ki avatar i4ki commented on July 29, 2024

Strange... I'm using org-mode 8.3.4 (but with emacs 24.5) and symbol org-babel--get-vars exists. We already have a compat function called org-babel-go-get-var to handle compatibility with older org-mode (< 8.3).

https://github.com/pope/ob-go/blob/master/ob-go.el#L187

I had a similar problem, but for 'org-babel-get-header not defined` in 8.3.3 and then I added this compat function to solve the problem. At that time I found this link:
https://lists.gnu.org/archive/html/emacs-orgmode/2016-01/msg00468.html

I don't know what to do here... The release notes of org-mode 8.3.[3,4] doesn't say nothing about org-babel-get-header/org-babel--get-vars, but the master branch (or 9.0 release notes) do say to avoid org-babel-get-header and use org-babel--get-vars instead:
http://orgmode.org/cgit.cgi/org-mode.git/tree/etc/ORG-NEWS#n367

As I'd said before, I think the best way to guarantee this package working across orgmode versions should be submitting it upstream...

from ob-go.

macalinao avatar macalinao commented on July 29, 2024

org-babel-get-header is defined for me.

org-version output:

Org-mode version 8.3.4 (8.3.4-79-gbd4948-elpa @ /usr/local/google/home/igm/.emacs.d/elpa/org-20160606/)

emacs-version output:

GNU Emacs 25.0.95.1 (x86_64-pc-linux-gnu, GTK+ Version 3.10.8), modified by Debian

from ob-go.

i4ki avatar i4ki commented on July 29, 2024

org-babel-get-header not defined here.

org-version output:
Org-mode version 8.3.4 (release_8.3.4-655-g9fb077 @ /home/i4k/.emacs.d/lib/org-mode/lisp/)

emacs-version output:

GNU Emacs 24.5.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.20.6) of 2016-06-04 on juergen

from ob-go.

i4ki avatar i4ki commented on July 29, 2024

Hmm, just figured out that my org is not a released version of orgmode 8.3.4 but a cloned git repo... It says 8.3.4 in (org-version) but it was the master branch of March 14 2016 ...

Sorry about that.

@pope The function org-babel-go-get-var is really wrong... Instead of using org-version to decide between get-header/--get-vars better to only verify if org-babel-get-header is defined and use org-babel--get-vars otherwise...

from ob-go.

 avatar commented on July 29, 2024

Maybe check if function is defined is better idea than checking version number at here.

from ob-go.

Related Issues (15)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.