Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

To install an assistant, sam runs bash -c 'sh setup.sh' - which breaks venv #185

Open
psiniemi opened this issue Aug 6, 2019 · 0 comments
Assignees
Labels
sub: sam Related to SAM type: bug Not working as expected

Comments

@psiniemi
Copy link

psiniemi commented Aug 6, 2019


What's the issue?

Venv activation requires to be run in bash to work, but sam runs bash -c 'sh setup.py' and sh is a symlink to dash on a lot of systems (ubuntu and newer raspbians at least) and that breaks venv activation.

Use Cases Impacted

Assistant install and update

How to reproduce

Run sam install assistant to a system with dash as sh

Expected behavior

Should install the assistant without errors

Version and Environment
0.63.3 / Lubuntu 19.04

Additional information

Workaround

Symlinking sh -> bash fixes this or just editing /usr/local/lib/node_modules/snips-sam/lib/session/ssh.js to simply run setup with bash directly.

@cpoisson cpoisson added type: bug Not working as expected sub: sam Related to SAM labels Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
sub: sam Related to SAM type: bug Not working as expected
Projects
None yet
Development

No branches or pull requests

3 participants