Script fails when fish is the default shell on remote machine

[ghostbuster91]

No description provided.

[artem-zinnatullin]

Any logs?

[arturdryomov]

[arturdryomov]

I heard that fish is not POSIX compliant (LOL). Probably shabang in the script is not enough and we must execute bash mainframer.sh explicitly.

[artem-zinnatullin]

I believe rsync will solve this as well, @ghostbuster91 please try 1.1.0 (will be released soon) and report back.

[ghostbuster91]

Thanks for quick replay.
Here is the error msg:

Unsupported use of '&&'. In fish, please use 'COMMAND; and COMMAND'.
fish: set -xe && cd ~/el-peon-android/ && ./gradlew compileDebugUnitTestSources mockableAndroidJar

There are two problems with this command.

1. Fish does not support && for concatenation. Instead to be both fish and bash compatible you might want to use ; character to separate commands.
2. The set command looks different in fish.

I think that the best solution would be to explicitly invoke these commands in bash shell, because supporting many shells might be cumbersome.
E.g. like this:

# Build project on a remote machine.
ssh $REMOTE_BUILD_MACHINE "echo 'set -xe && cd ~/$PROJECT_DIR_NAME/ && $BUILD_COMMAND' | bash"

[artem-zinnatullin]

Yup, sounds reasonable

…

On Thu, Dec 29, 2016, 17:38 Kasper Kondzielski ***@***.***> wrote: Thanks for quick replay. Here is the error msg: fish: set -xe && cd ~/el-peon-android/ && ./gradlew compileDebugUnitTestSources mockableAndroidJar There are two problems with this command. 1. Fish does not support && for concatenation. Instead to be both fish and bash compatible you might want to use ; character to separate commands. 2. The set command looks different in fish. I think that the best solution would be to explicitly invoke these commands in bash shell, because supporting many shells might be cumbersome. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#43 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA7B3AtjuJ2FBABPBot_XLoiyvHVqk-nks5rM8XYgaJpZM4LXfps> .

[artem-zinnatullin]

Closed by #44.