Support array spread in exec

[dovahcrow]

Feature Description

This is similar to #133 but came from cargo-make and is for array:
When using cargo-make, additional parameters are stored in ${@}, like other runners.
However, in duckscript, ${@} is an array, making it impossible to pass to exec.

Describe The Solution You'd Like

Maybe add a new syntax to spread the array?

Code Sample

/// paste code here
exec cargo run --release -- ${@}

Or any new syntax makes the above work.

[sagiegurari]

why not do array_join and than spread?

[dovahcrow]

Yeah array_join then spread is an option I haven't thought of (still thinking in strong typing lol)

Anyway, you can close the issue if you feel a new syntax is unnecessary.

EDIT: does not work for empty args unless you add the special case handling in duckscript as well.

[sagiegurari]

just saw the edit, can you tell me what you tried and what you saw?

[dovahcrow]

@sagiegurari I tried:

[tasks.release]
script = """
files = array_join ${@} " "
exec --fail-on-error tar -cf release.tar must_have.sh %{files}
"""
script_runner = "@duckscript"

cargo make release abc worked, but cargo make release won't.

I can add if to the script but that requires every such usage to add an if.

[sagiegurari]

thanks i will check it out

[sagiegurari]

@dovahcrow this is now fixed for spread binding and being released.
i'll package and relase a new cargo-make version contain this fix as well soon

[dovahcrow]

This is great!