[xcvb-devel] make test

[Faré]

I would say "then don't use that CL_LAUNCH_FLAGS". Proper quoting
between Make and Shell is HORROR. When you control the environment,
there is no point is having more that one option for the lisps,
anyway.

How would you do it?

[ François-René ÐVB Rideau | Reflection&Cybernethics | http://fare.tunes.org ]
Nature, to be commanded, must be obeyed. — Francis Bacon




On 24 October 2010 08:20, Attila Lendvai <attila.lendvai at gmail.com> wrote:
> in this function in runme.zsh:
>
>
> check_asdf_setup () {
>  CONFIGURED_CL_LAUNCH_FLAGS="$(make show-config $ENV | grep
> "^CL_LAUNCH_FLAGS=" | cut -d= -f2-)"
>  for i in asdf-dependency-grovel cl-launch closer-mop
> command-line-arguments poiu ; do
>    cl-launch $=CONFIGURED_CL_LAUNCH_FLAGS -i \
>    "(cl-launch::quit (if (asdf:find-system :$i nil) 0 1))" ||
>    abort "Couldn't locate ASDF system $i. Make sure your configure.mk has it."
>  done
> }
>
>
> with the value:
>
>
> CL_LAUNCH_FLAGS += --lisp '${LISP} sbcl clisp ccl'
>
>
> i get some zsh quoting headache:
>
>
> +check_asdf_setup:1> CONFIGURED_CL_LAUNCH_FLAGS=+check_asdf_setup:1>
> cut '-d=' -f2-
> +check_asdf_setup:1> CONFIGURED_CL_LAUNCH_FLAGS=+check_asdf_setup:1>
> grep '^CL_LAUNCH_FLAGS='
> +check_asdf_setup:1> CONFIGURED_CL_LAUNCH_FLAGS=' --lisp '\''sbcl sbcl
> clisp ccl'\'' --source-registry
> /home/alendvai/workspace//:/usr/share/common-lisp/source// --verbose'
> +check_asdf_setup:2> i=asdf-dependency-grovel
> +check_asdf_setup:3> cl-launch --lisp \''sbcl' sbcl clisp 'ccl'\'
> --source-registry
> /home/alendvai/workspace//:/usr/share/common-lisp/source// --verbose
> -i '(cl-launch::quit (if (asdf:find-system :asdf-dependency-grovel
> nil) 0 1))'
> Unrecognized command line argument 'sbcl'
> cl-launch.sh 3.004 -- shell wrapper generator for Common Lisp software
>
>
> wasted an hour trying to get rid of those extra quotes, but i gave up
> and simply changed my config to kludge the issue:
>
> CL_LAUNCH_FLAGS += --lisp '${LISP}'
>
>
> $ zsh --version
> zsh 4.3.10 (x86_64-unknown-linux-gnu)
>
> on Ubuntu 10.10
>