Skip to content

hotfix-for-234: use POSIX dot command for Digital Ocean compatibility. #235

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

Merged
merged 1 commit into from
Jul 27, 2024

Conversation

dsriseah
Copy link

@dsriseah dsriseah commented Jul 26, 2024

This PR changes the way that npm run dev works, as the system shell on Ubuntu is different than MacOs. Specifically, it replaces the source command with the "dot" command which is more universal.

Previously, we had assumed that /bin/sh implemented the same interface so if it worked on MacOs then it would work elsewhere. In actuality, the system shell is merely a "default shell" and can vary quite a lot.

See issue #234 for details.

TESTING

  • pull this branch on Digital Ocean
  • try npm run dev

You should be greeted with something like this:
image
instead of
image

@dsriseah dsriseah requested review from benloh and jdanish July 26, 2024 17:43
@dsriseah dsriseah self-assigned this Jul 27, 2024
@dsriseah
Copy link
Author

I had already confirmed this on my Digital Ocean instance that I use for URSYS testing, so I'll just accept it :-)

@dsriseah dsriseah removed request for benloh and jdanish July 27, 2024 09:43
@dsriseah dsriseah merged commit 01a6775 into dev Jul 27, 2024
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

Successfully merging this pull request may close these issues.

1 participant