Re: Re: [Babel] Handling of errors when using Ledger

[Eric S Fraga <ucecesf@ucl.ac.uk>]

Dan <dandavison7@gmail.com> writes:

[...]

> This patch should make ob-ledger use the common org-babel error mechanism. It is
> in branch ledger of the main repo. However, I'm not yet a ledger user. Could
> someone test it?

Unfortunately, it doesn't work at all [1] for a very simple reason: you
have removed the -f option which specifies the file that ledger should
read.  Ledger does not interpret a file argument directly but needs the
-f option, as in:

   ledger -f file.ledger <rest of command line>

Note the second line in the concatenated list of strings below:

[...]

> -    (with-output-to-string
> -      (shell-command (concat "ledger"
> -                            " -f " (org-babel-process-file-name in-file)
> -                            " " cmdline
> -                            " > " (org-babel-process-file-name out-file))))

I am not sure how you pass this option with the much shorter code you
have written:

> +  (org-babel-eval
> +   (concat org-babel-ledger-command " " (cdr (assoc :cmdline params)))
> +   body))

I hope this makes sense?

Footnotes: 
[1]  actually, what does work very nicely indeed is that error messages
     are reported perfectly!!  So thank you very much for this.

-- 
: Eric S Fraga (GnuPG: 0xC89193D8FFFCF67D) in Emacs 23.2.1
: using Org-mode version 7.3 (release_7.3.99.g9db0.dirty)