Re: [O] [PATCH] Fix R table import

2015-01-05 Thread Aaron Ecay
Hi Michael, Thanks for following up. It’s concerning that this misbehavior seems to depend on the ESS version. However, since the latest version does indeed work, I don’t think any intervention is warranted. I’m glad the problem is fixed for you and sorry for the inconvenience. -- Aaron Ecay

Re: [O] [PATCH] Fix R table import

2015-01-04 Thread Michael Gauland
My git has so many more commits because I'm sloppy about my local changes. I had been using ESS 5.14 (from ELPA). When I switched to the latest git version, the problem (at least seems to have) went away. For my needs, I don't think this needs any further investigation, though I'm happy to help i

Re: [O] [PATCH] Fix R table import

2015-01-04 Thread Aaron Ecay
Hi Michael, Hmm. I have Org-mode version 8.3beta (release_8.3beta-695-g390015 @ /home/aecay/development/org-mode/lisp/) The git sha 034d00 from your org-version isn’t in my checkout. Yours also thinks it’s 60ish commits ahead of where mine is (758 vs. 695). This is very strange; I don’t know

Re: [O] [PATCH] Fix R table import

2015-01-04 Thread Michael Gauland
I'm using the latest from git, with some unrelated local modifications (though to be sure I did a clean build from origin/master and had the same problem). Versions: Org-mode version 8.3beta (release_8.3beta-758-g034d00.dirty @ /usr/local/share/emacs/site-lisp/org/) GNU Emacs 24.3.1 (x86

Re: [O] [PATCH] Fix R table import

2015-01-04 Thread Andreas Leha
Hi Aaron, Aaron Ecay writes: > Hi Michael, > > Thanks for this patch. Before it is applied, though, I would like to > understand why it is necessary. Vikas said that he could not reproduce > the problem you originally reported. (Here is a web archived version of > Vikas’s message, in case you

Re: [O] [PATCH] Fix R table import

2015-01-03 Thread Aaron Ecay
Hi Michael, Thanks for this patch. Before it is applied, though, I would like to understand why it is necessary. Vikas said that he could not reproduce the problem you originally reported. (Here is a web archived version of Vikas’s message, in case you did not see it at first:

[O] [PATCH] Fix R table import

2015-01-03 Thread Michael Gauland
ob-R.el: Fix R table import * ob-R.el (ob-R-transfer-variable-table-with-header): Change separator from tab to whitespace when reading in a table as a variable. TINYCHANGE lisp/ob-R.el |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lisp/ob-R.el b/lisp/ob-R.el index 2470