Dan <dandavis...@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) _______________________________________________ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode