Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Make plugin compatible with macos or restrict it just to linux distro #8

Open
Marek00Malik opened this issue Jun 4, 2019 · 4 comments

Comments

@Marek00Malik
Copy link

Hello,
I ran into a problem with SED when executing the sdk-refresh-completion-files command on MacOs (10.14.5).
This is the following error I'm getting:

➜ ~ sdk-refresh-completion-files sed: illegal option -- r usage: sed script [-Ealn] [-i extension] [file ...]

From what I know sed is a bit different for Mac because of the BSD difference.
Could you change the usage of sed to grep or any other text tool that would be compatible with OSX and Linux ?

@matthieusb
Copy link
Owner

Hello,

I don't have a lot of time right now to take a look at this.
However I'll try as soon as I can (Maybe this week-end). If you come across a fix, don't hesitate to fork and PR !

Thanks for the heads-up.

@matthieusb
Copy link
Owner

HI.

I have released a new version you can use to correct the sed -r error.
However there might be other problems with sed expressions, so i am not sure if this is completely corrected.

Tell me if this fixes you problem. Don't hesitate to give me some more feedback on this issue and I'll try to correct them along the way.

Thanks in advance.

@matthieusb
Copy link
Owner

Has your problem been fixed ?

@matthieusb
Copy link
Owner

Hello, did my update fix your problem ?

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

No branches or pull requests

2 participants