Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-03-02 Thread Achim Gratz
Achim Gratz writes: Here are two patches that fix this and implement (partly) some of your suggestions. I don't think Org should pollute the global Perl namespace by default, so I've left the definition of org-babel-perl-preface to the user for now. The second patch has the debugging aid

Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-02-26 Thread Achim Gratz
D M German dmg at uvic.ca writes: print $BO join($/, @r), $/ ; Sorry, this should really be: print $BO join(qq($/), @r), qq($/); Anyway, I think I'll have to rework the wrapper to be an anonymous subroutine so that the (reasonably expactable) return @foo; at the end of the program (or in the

Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-02-25 Thread D M German
Achim Gratz twisted the bytes to say: Hi Achim, thanks for taking the time to do this. I applied the patch, one of the hunks didn't apply due to Eric's changes, but that is not an issue, since they do the same: -- diff

Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-02-25 Thread Achim Gratz
D M German dmg at uvic.ca writes: I think the issue is that, at least in my computer the variable $\ returns empty (the record separator). Thinko on my side, what I wanted was the input record separator $/ (to avoid specifying a literal newline for those systems where this is actually

Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-02-25 Thread D M German
Achim D M German dmg at uvic.ca writes: I think the issue is that, at least in my computer the variable $\ returns empty (the record separator). Achim Thinko on my side, what I wanted was the input record separator $/ Achim (to avoid Achim specifying a literal newline for those systems

Re: [O] [PATCH] bug in expansion of variables in babel Perl

2013-02-24 Thread Achim Gratz
D M German writes: I found a bug in the Babel perl code. When a table is used as input, the values of the table are not escaped. Here are two patches that fix this and implement (partly) some of your suggestions. I don't think Org should pollute the global Perl namespace by default, so I've