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

fix: use macos compatible sed syntax #63

Merged
merged 1 commit into from
Sep 22, 2024
Merged

fix: use macos compatible sed syntax #63

merged 1 commit into from
Sep 22, 2024

Conversation

aabouzaid
Copy link
Collaborator

Fixes: #62

The sed command used in MacOS is BSD sed which uses some different syntax from GNU sed used in Linux.
So, I use a syntax that works with both.

@aabouzaid aabouzaid self-assigned this Sep 22, 2024
@aabouzaid aabouzaid requested a review from a team as a code owner September 22, 2024 06:10
@aabouzaid aabouzaid force-pushed the 62-fix-sed-syntax branch 15 times, most recently from 42fbdc1 to 203dd1b Compare September 22, 2024 07:34
Signed-off-by: Ahmed AbouZaid <6760103+aabouzaid@users.noreply.github.com>
@aabouzaid aabouzaid added bug Something isn't working os/mac Issues related to macOS labels Sep 22, 2024
@aabouzaid aabouzaid merged commit 4de7c98 into main Sep 22, 2024
5 checks passed
@aabouzaid aabouzaid deleted the 62-fix-sed-syntax branch September 22, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working os/mac Issues related to macOS
Development

Successfully merging this pull request may close these issues.

When running update-all up to date plugins crash the process
1 participant