Re: Backtrace on hello world

[Arun Isaac <arunisaac@systemreboot.net>]

Hi Simon,

Thanks for trying out ravanan!

> First, I notice that the project on Github [1] has the “Pull Request”
> available but the README mentions: « We do not accept issues or pull
> requests on GitHub. »
>
> Moreover, I have not found the Issue tracker; hence the report here.
> :-)

You found the right place! :-) The working plan for the moment is to do
both issues and patches over email the old fashioned way.

> First thing, the option ’--store’ seems mandatory when ’--help’ does not
> mention it, if I read correctly.

It is documented. See below:
--8<---------------cut here---------------start------------->8---
$ ravanan --help 2>&1 | grep store
Usage: /gnu/store/gddra8pjszmw0bfyhmgipsg12r88195v-ravanan-0.2.0/bin/.ravanan-real [OPTIONS] CWL-WORKFLOW INPUTS
  --store=STORE                  path to shared store to write results into
--8<---------------cut here---------------end--------------->8---

> Second, maybe I’m missing something:
>
> --8<---------------cut here---------------start------------->8---
> $ ravanan doc/hello-world.cwl doc/hello-world-inputs.json --store=/tmp/foo
> Backtrace:
> In ice-9/boot-9.scm:
>   1752:10 17 (with-exception-handler _ _ #:unwind? _ #:unwind-for-type _)
> In unknown file:
>           16 (apply-smob/0 #<thunk 79b46b5132a0>)
> In ice-9/boot-9.scm:
>     724:2 15 (call-with-prompt _ _ #<procedure default-prompt-handler (k proc)>)
> In ice-9/eval.scm:
>     619:8 14 (_ #(#(#<directory (guile-user) 79b46b518c80>)))
> In ice-9/boot-9.scm:
>     152:2 13 (with-fluid* _ _ _)
>     152:2 12 (with-fluid* _ _ _)
> In ice-9/eval.scm:
>     619:8 11 (_ #(#(#(#(#(#(#(#(#<directory (guile-user) 79b46b518c80> ((store . "/tmp/f…") …) …)) …) …) …) …) …) …))
>    293:34 10 (_ #(#(#(#<directory (guile-user) 79b46b518c80> "doc/hello-world.cwl" ((store . "/tmp/foo") …) …) …) …))
> In ice-9/boot-9.scm:
>     724:2  9 (call-with-prompt ("prompt") #<procedure 79b44cb8e9e0 at ice-9/eval.scm:330:13 ()> #<procedure 79b44c…>)
>   1752:10  8 (with-exception-handler _ _ #:unwind? _ #:unwind-for-type _)
> In ravanan/workflow.scm:
>    635:25  7 (run-workflow "hello-world" #f #f (("baseCommand" . #("echo" "Hello")) ("outputs" . #(((# . #) …))) …) …)
> In ice-9/boot-9.scm:
>   1752:10  6 (with-exception-handler _ _ #:unwind? _ #:unwind-for-type _)
> In guix/store.scm:
>    690:37  5 (thunk)
> In ravanan/workflow.scm:
>    204:25  4 (workflow->scheduler-proc "hello-world" (("baseCommand" . #("echo" "Hello")) ("outputs" . #((# …))) …) …)
> In ravanan/command-line-tool.scm:
>    618:49  3 (build-command-line-tool-script "hello-world" #f #f (("baseCommand" . #("echo" "Hello")) ("o…" . #) …) …)
> In rnrs/base.scm:
>    253:18  2 (vector-map #<procedure 79b44cb8f1a0 at ice-9/boot-9.scm:797:8 args> #f)
> In ice-9/boot-9.scm:
>   1685:16  1 (raise-exception _ #:continuable? _)
>   1685:16  0 (raise-exception _ #:continuable? _)
>
> ice-9/boot-9.scm:1685:16: In procedure raise-exception:
> In procedure vector-length: Wrong type argument in position 1 (expecting vector): #f
> --8<---------------cut here---------------end--------------->8---

Oops, this is very embarrassing! I've fixed it now and added tests to
prevent regression.

https://klaus.systemreboot.net/ravanan/commit/c5bccca30b03f6dfef5d6efaaafc20fa9b25f164/
https://klaus.systemreboot.net/ravanan/commit/426201609e13d47960a922456cbbddf61cde3330/

Now, on the latest ravanan (a0b7e7ab6de830182ab30a0e417eefb56a07fd2f),
you can run it like so:

$ ravanan --store=store --guix-manifest=doc/hello-world-manifest.scm
doc/hello-world.cwl doc/hello-world-inputs.yaml

Regards,
Arun