Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shell files aren't usable on Ubuntu/Debian #11

Open
TimonLukas opened this issue Apr 12, 2021 · 2 comments
Open

Shell files aren't usable on Ubuntu/Debian #11

TimonLukas opened this issue Apr 12, 2021 · 2 comments

Comments

@TimonLukas
Copy link
Contributor

Hey! I'm currently trying to deploy parts of this repository for a demo, thank you for the great resource!

Sadly, it's not possible to use the scripts as-is on Ubuntu or Debian machines. Their sh implementation, Dash, is written to be very small, and as such doesn't support many features you'd expect. They already error on the declaration of the out function. If I execute them using bash (as in the shebang line) everything works perfectly.

My solution is now to replace every occurrence of sh [some script] with ./[some script]. On another machine I solved this problem by symlinking /bin/bash to /usr/bin/s, but that's a very dirty solution.

Based on the shebang lines, would anything break on other environments if this change would be implemented?

@nheidloff
Copy link
Member

Hi Timon, I've only tested on Mac so far. I want to create a Docker image with all tools installed so that the scripts can be run on various operating systems.

@TimonLukas
Copy link
Contributor Author

TimonLukas commented Apr 12, 2021

That sounds like a great idea!

For now I'll still try to get it running myself on a Debian installation. I'll update this comment with any steps that were necessary to do so - if it's just a few and it keeps working on Mac (which it should), it could be a nice thing to do until that Docker setup is finished :)

  • Permissions error for /scripts/* and /scripts-docker/* - fixed using chown -R +x scripts*
  • Error because of Dash - fixed using sed -i 's/sh /.\//' scripts*/**

Update: Those commands sufficed, now the script ran through perfectly on Debian :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants