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 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 mi

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

2013-02-25 Thread D M German
Achim> D M German 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 syst

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

2013-02-25 Thread Achim Gratz
D M German 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 multi-char

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 a/lisp/

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