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

sed -z is not available on MacOS #7

Open
elidhu opened this issue Feb 3, 2023 · 1 comment
Open

sed -z is not available on MacOS #7

elidhu opened this issue Feb 3, 2023 · 1 comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@elidhu
Copy link

elidhu commented Feb 3, 2023

I would suggest either:

  • changing to sed s/$/ /
  • conditional compilation
  • not using sed!

Note that this affects your unit tests and the regressing test for shell, including the regression test checker

@FauconFan
Copy link
Owner

This asks a bigger problem, such as testing in a MacOS like environment, currently only ubuntu is tested. See #5 for Windows. I'll mark this as Help wanted if anyone wants to extend the CI pipeline to MacOS.

Ty for reporting this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants