You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
One of the core tasks of snowsaw is its capability of running shell commands by using a shell field in the snowblock.json file. In the python implementation of snowsaw, user can use complex shell commands such as echo a || echo b in the command field and it will be handled correctly. However, in the go implementation of snowsaw, such command cannot be executed correctly. I believe this is because the code will take the first word -- echo in the example -- as the command name, and interpret everything else -- a || echo b in the example -- as the parameters of the command.
This breaks the backward compatibility of the snowsaw tool.
The text was updated successfully, but these errors were encountered:
xulongwu4
changed the title
The re-implementation of snowsaw in go cannot handle complex shell command
The re-implementation of snowsaw in go cannot handle complex shell commands
Oct 5, 2021
One of the core tasks of
snowsaw
is its capability of running shell commands by using ashell
field in thesnowblock.json
file. In the python implementation ofsnowsaw
, user can use complex shell commands such asecho a || echo b
in thecommand
field and it will be handled correctly. However, in the go implementation ofsnowsaw
, such command cannot be executed correctly. I believe this is because the code will take the first word --echo
in the example -- as the command name, and interpret everything else --a || echo b
in the example -- as the parameters of the command.This breaks the backward compatibility of the
snowsaw
tool.The text was updated successfully, but these errors were encountered: