Re: Babel ':session ' bug?

From: "Eric Schulte" <schulte.eric@gmail.com>
To: "Charles C. Berry" <cberry@tajo.ucsd.edu>
Cc: emacs-orgmode@gnu.org
Subject: Re: Babel ':session ' bug?
Date: Tue, 23 Nov 2010 09:40:39 -0700	[thread overview]
Message-ID: <87tyj856yg.fsf@gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.1011221615170.18978@tajo.ucsd.edu> (Charles C. Berry's message of "Mon, 22 Nov 2010 16:27:33 -0800")

Hi chuck,

I just pushed up a change (and an exercising unit test) which should now
finally actually fix this issue.  Thanks for persistently pointing out
the problem.

Best -- Eric

"Charles C. Berry" <cberry@tajo.ucsd.edu> writes:

> On Mon, 22 Nov 2010, Eric Schulte wrote:
>
>> Hi,
>>
>> I get the following when calling (org-babel-get-src-block-info) on the
>> following code block
>>
>> #+BEGIN_SRC R :session :results output
>> xyz
>> #+END_SRC
>>
>> ("R" "xyz
>> " ((:colname-names) (:rowname-names) (:result-params "output" "replace")
>> (:result-type . output) (:comments . "") (:shebang . "") (:cache . "no")
>> (:noweb . "no") (:tangle . "no") (:exports . "code") (:results . "output
>> replace") (:session) ...) "" nil 0)
>>
>> note that I'm getting "(:session)" for the session header argument, which
>> is correct.  Could you send me a copy of the offending code block?
>>
>
> To my eye, this seemed to be exactly what you have. But using hexl-mode, I 
> see I have three blanks between ':session' and ':results', and two of 
> them show up in '(:session\ \ )'. Using:
>
>
> #+BEGIN_SRC R :session   :results output
> xyz
> #+END_SRC
>
>
>   I get:
>
> ("R" "xyz
> " ((:colname-names) (:rowname-names) (:result-params "output" "replace") 
> (:result-type . output) (:comments . "") (:shebang . "") (:cache . "no") 
> (:noweb . "no") (:tangle . "no") (:exports . "code") (:results . "output 
> replace") (:session\ \ ) ...) "" nil 0)
>
> Deleting the extra blanks gives the same result you have.
>
> Chuck
>
>
>> Thanks -- Eric
>>
>> "Charles C. Berry" <cberry@tajo.ucsd.edu> writes:
>>
>>> On Mon, 22 Nov 2010, Eric Schulte wrote:
>>>
>>>> Hi Charles,
>>>>
>>>> I've just pushed up a change which should fix this issue.
>>>>
>>>
>>> Thanks.
>>>
>>> ':session' has the right number of colons, but is followed by '\ \ ',
>>> and no session was started.
>>>
>>> i.e.,
>>>
>>> 	M-x eval-expression RET (org-babel-get-src-block-info) RET
>>>
>>> gives:
>>>
>>> ("R" "xyz
>>> " ((:colname-names) (:rowname-names) (:result-params "output"
>>> "replace") (:result-type . output) (:comments . "") (:shebang . "")
>>> (:cache . "no") (:noweb . "no") (:tangle . "no") (:exports . "code")
>>> (:results . "output replace") (:session\ \ ) ...) "" nil 0)
>>>
>>>
>>> Chuck
>>>
>>>
>>>> Thanks for the report -- Eric
>>>>
>>>> "Charles C. Berry" <cberry@tajo.ucsd.edu> writes:
>>>>
>>>>> Correction in line below. CCB
>>>>>
>>>>> On Sat, 20 Nov 2010, Charles C. Berry wrote:
>>>>>
>>>>>> I think :session without passing a string used to start a session, but
>>>>>> now it does not. I think this is a bug. Details:
>>>>>>
>>>>>> Just did git pull and now with this:
>>>>>>
>>>>>> ,----
>>>>>> |  #+BEGIN_SRC R :session   :results output xyz
>>>>>> |  #+END_SRC
>>>>>> | |  #+results:
>>>>>> | |  #+BEGIN_SRC R :session *R*  :results output xyz
>>>>>> |  #+END_SRC
>>>>>> `----
>>>>>>
>>>>>
>>>>> Oops! That should have been
>>>>>
>>>>> ,----
>>>>> |  #+BEGIN_SRC R :session   :results output |  xyz
>>>>> |  #+END_SRC
>>>>> |
>>>>> |  #+results:
>>>>> |
>>>>> |  #+BEGIN_SRC R :session *R*  :results output |  xyz
>>>>> |  #+END_SRC
>>>>> `----
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>>> If I put point after 'xyz'  in the first block and do
>>>>>>
>>>>>>   M-x eval-expression RET (org-babel-get-src-block-info) RET
>>>>>>
>>>>>> I get
>>>>>>
>>>>>> ("R" "xyz
>>>>>> " ((:colname-names) (:rowname-names) (:result-params "output"
>>>>>> "replace") (:result-type . output) (:comments . "") (:shebang . "")
>>>>>> (:cache . "no") (:noweb . "no") (:tangle . "no") (:exports . "code")
>>>>>> (:results . "output replace") (::session\ \ ) ...) "" nil 0)
>>>>>>
>>>>>> Note '(::session\ \ ).
>>>>>>
>>>>>> Whilst putting point in the second block leads to
>>>>>>
>>>>>> ("R" "xyz
>>>>>> " ((:colname-names) (:rowname-names) (:result-params "output"
>>>>>> "replace") (:result-type . output) (:comments . "") (:shebang . "")
>>>>>> (:cache . "no") (:noweb . "no") (:tangle . "no") (:exports . "code")
>>>>>> (:results . "output replace") (:session . "*R*") ...) "" nil 0)
>>>>>>
>>>>>>
>>>>>> Chuck
>>>>>>
>>>>>>
>>>>>> Charles C. Berry                            Dept of
>>>>>> Family/Preventive Medicine
>>>>>> cberry@tajo.ucsd.edu			    UC San Diego
>>>>>> http://famprevmed.ucsd.edu/faculty/cberry/  La Jolla, San Diego 92093-0901
>>>>>>
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> 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
>>>>>>
>>>>>>
>>>>>
>>>>> Charles C. Berry Dept of Family/Preventive Medicine
>>>>> cberry@tajo.ucsd.edu			    UC San Diego
>>>>> http://famprevmed.ucsd.edu/faculty/cberry/  La Jolla, San Diego 92093-0901
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> 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
>>>>
>>>> _______________________________________________
>>>> 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
>>>>
>>>
>>> Charles C. Berry                            Dept of Family/Preventive Medicine
>>> cberry@tajo.ucsd.edu			    UC San Diego
>>> http://famprevmed.ucsd.edu/faculty/cberry/  La Jolla, San Diego 92093-0901
>>
>
> Charles C. Berry                            Dept of Family/Preventive Medicine
> cberry@tajo.ucsd.edu			    UC San Diego
> http://famprevmed.ucsd.edu/faculty/cberry/  La Jolla, San Diego 92093-0901