emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [babel] sreen : unable to run org-babel-screen-test
@ 2009-11-30  8:13 bluedian
  2009-11-30 11:32 ` Benjamin Andresen
  0 siblings, 1 reply; 6+ messages in thread
From: bluedian @ 2009-11-30  8:13 UTC (permalink / raw)
  To: emacs-orgmode

Hi,

When running "org-babel-screen-test", a xterm appear with its prompt, but no
message appear. Emacs hangs and my cpu is à 100%.

When hitting c-G (whit debug mode activated), I get this message :
Debugger entered--Lisp error: (quit)
  (not (file-readable-p tmpfile))
  (while (not (file-readable-p tmpfile)) (format "org-babel-screen: File not
readable yet."))
  (let* ((session "org-babel-testing") (random-string ...) (tmpfile
"/tmp/org-babel-screen.test") (body ...) process tmp-string)
(org-babel-execute:screen body org-babel-default-header-args:screen) (while (not
...) (format "org-babel-screen: File not readable yet.")) (setq tmp-string
(with-temp-buffer ... ...)) (delete-file tmpfile) (message (concat
"org-babel-screen: Setup " ...)))
  org-babel-screen-test()
  call-interactively(org-babel-screen-test)
  (unwind-protect (call-interactively chosen) (smex-rank chosen)
(smex-show-key-advice chosen))
  (let ((chosen ...)) (unwind-protect (call-interactively chosen) (smex-rank
chosen) (smex-show-key-advice chosen)))
  smex()
  call-interactively(smex nil nil)

In the * message * buffer, I get this :
Sending source code block to interactive terminal session...
Warning: Tried to connect to session manager, None of the authentication
protocols specified are supported
Entering debugger...

the last sentence "None of the authentication protocols specified are supported"
is surely the key, but I didn't understand it as I have xterm, sh and screen
installed and correctly configurated.

I'm on Ubuntu 9.04 - Jaunty Jackalope, Emacs 23, the last (this morning)
org-mode version on the git repository (I use org-track-update).

Other parts of babel are correctly working (shell and emacs mode without the
screen part)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: sreen : unable to run org-babel-screen-test
  2009-11-30  8:13 [babel] sreen : unable to run org-babel-screen-test bluedian
@ 2009-11-30 11:32 ` Benjamin Andresen
  2009-11-30 11:59   ` Benjamin Andresen
  0 siblings, 1 reply; 6+ messages in thread
From: Benjamin Andresen @ 2009-11-30 11:32 UTC (permalink / raw)
  To: emacs-orgmode

Hey,

bluedian <bluedian@gmail.com> writes:

> Hi,
>
> When running "org-babel-screen-test", a xterm appear with its prompt, but no
> message appear. Emacs hangs and my cpu is à 100%.

I just tried it myself, and apparently it's broken with the current git.

I will have a look and report back with my findings!

br,
benny

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: sreen : unable to run org-babel-screen-test
  2009-11-30 11:32 ` Benjamin Andresen
@ 2009-11-30 11:59   ` Benjamin Andresen
  2009-11-30 14:26     ` Eric Schulte
  0 siblings, 1 reply; 6+ messages in thread
From: Benjamin Andresen @ 2009-11-30 11:59 UTC (permalink / raw)
  To: emacs-orgmode

[-- Attachment #1: Type: text/plain, Size: 566 bytes --]

Hey,

Benjamin Andresen <benny@in-ulm.de> writes:

> I will have a look and report back with my findings!

I was relying on a variable which used to contain the value it needed,
but doesn't do any more. It was a foolish shortcut and now it uses the
official route by processing the parameters that are passed to it.

As I don't have access to the official org-mode git repo, can someone
please apply this patch for me?

br,
benny

P.S. I'm not committing this patch to the org-babel repo, out of fear of
merge issues later on. I guess this route is better? Thanks!


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: org-babel-screen-session.patch --]
[-- Type: text/x-patch, Size: 875 bytes --]

diff --git a/contrib/babel/lisp/langs/org-babel-screen.el b/contrib/babel/lisp/langs/org-babel-screen.el
index efeab23..1529ac0 100644
--- a/contrib/babel/lisp/langs/org-babel-screen.el
+++ b/contrib/babel/lisp/langs/org-babel-screen.el
@@ -55,7 +55,9 @@ In case you want to use a different screen than one selected by your $PATH")
 \"default\" session is be used when none is specified."
   (message "Sending source code block to interactive terminal session...")
   (save-window-excursion
-    (let ((socket (org-babel-screen-session-socketname session)))
+    (let* ((processed-params (org-babel-process-params params))
+           (session (first processed-params))
+           (socket (org-babel-screen-session-socketname session)))
       (unless socket (org-babel-prep-session:screen session params))
       (org-babel-screen-session-execute-string session body))))
 

[-- Attachment #3: Type: text/plain, Size: 201 bytes --]

_______________________________________________
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

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: Re: sreen : unable to run org-babel-screen-test
  2009-11-30 11:59   ` Benjamin Andresen
@ 2009-11-30 14:26     ` Eric Schulte
  2009-12-01  7:50       ` bluedian
  0 siblings, 1 reply; 6+ messages in thread
From: Eric Schulte @ 2009-11-30 14:26 UTC (permalink / raw)
  To: Benjamin Andresen; +Cc: emacs-orgmode

Hi Benny,

We should have caught this usage when we changed the variable passing
behavior in org-babel.  Thanks for the patch, it is now applied.

Best -- Eric

Benjamin Andresen <benny@in-ulm.de> writes:

> Hey,
>
> Benjamin Andresen <benny@in-ulm.de> writes:
>
>> I will have a look and report back with my findings!
>
> I was relying on a variable which used to contain the value it needed,
> but doesn't do any more. It was a foolish shortcut and now it uses the
> official route by processing the parameters that are passed to it.
>
> As I don't have access to the official org-mode git repo, can someone
> please apply this patch for me?
>
> br,
> benny
>
> P.S. I'm not committing this patch to the org-babel repo, out of fear of
> merge issues later on. I guess this route is better? Thanks!
>
> diff --git a/contrib/babel/lisp/langs/org-babel-screen.el b/contrib/babel/lisp/langs/org-babel-screen.el
> index efeab23..1529ac0 100644
> --- a/contrib/babel/lisp/langs/org-babel-screen.el
> +++ b/contrib/babel/lisp/langs/org-babel-screen.el
> @@ -55,7 +55,9 @@ In case you want to use a different screen than one selected by your $PATH")
>  \"default\" session is be used when none is specified."
>    (message "Sending source code block to interactive terminal session...")
>    (save-window-excursion
> -    (let ((socket (org-babel-screen-session-socketname session)))
> +    (let* ((processed-params (org-babel-process-params params))
> +           (session (first processed-params))
> +           (socket (org-babel-screen-session-socketname session)))
>        (unless socket (org-babel-prep-session:screen session params))
>        (org-babel-screen-session-execute-string session body))))
>  
> _______________________________________________
> 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: sreen : unable to run org-babel-screen-test
  2009-11-30 14:26     ` Eric Schulte
@ 2009-12-01  7:50       ` bluedian
  2009-12-01 12:20         ` Benjamin Andresen
  0 siblings, 1 reply; 6+ messages in thread
From: bluedian @ 2009-12-01  7:50 UTC (permalink / raw)
  To: emacs-orgmode

Hi,

With the last git version (updated by org-track 5 five minutes ago), I have
still the same problem and the same error message.

Regards,
BlueDian

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: sreen : unable to run org-babel-screen-test
  2009-12-01  7:50       ` bluedian
@ 2009-12-01 12:20         ` Benjamin Andresen
  0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Andresen @ 2009-12-01 12:20 UTC (permalink / raw)
  To: emacs-orgmode

Hey,

bluedian <bluedian@gmail.com> writes:

> Hi,
>
> With the last git version (updated by org-track 5 five minutes ago), I have
> still the same problem and the same error message.

I'm a bit at a loss with debugging this thing non-interactively. (Yeah I
know, after the first one was unsuccessful that's early to give up.)

But because it uses both the filesystem and GNU screen the error could
lie anywhere and not just in the emacs lisp code.

I will try and see if I can improve the test code by testing checking at
several stages if something has gone wrong, so that I will get a better
idea why and when things start to go awry.

> Regards,
> BlueDian

br,
benny

P.S. Could anyone that comes by this with GNU/Linux and GNU screen try
it out as well? I'm using this line to test the setup:
% emacs --batch -l orgloadpath_org-babel-screen-test.el

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2009-12-01 12:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-30  8:13 [babel] sreen : unable to run org-babel-screen-test bluedian
2009-11-30 11:32 ` Benjamin Andresen
2009-11-30 11:59   ` Benjamin Andresen
2009-11-30 14:26     ` Eric Schulte
2009-12-01  7:50       ` bluedian
2009-12-01 12:20         ` Benjamin Andresen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).