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

Silence sed warning #12

Closed
mklxd opened this issue Sep 11, 2018 · 5 comments
Closed

Silence sed warning #12

mklxd opened this issue Sep 11, 2018 · 5 comments

Comments

@mklxd
Copy link

mklxd commented Sep 11, 2018

To silence the following error message from sed, apply the patch below:
sed: character class syntax is [[:space:]], not [:space:]

Patch:

diff --git a/updBlackList.sh b/updBlackList.sh
index d04caca..246b9d0 100755
--- a/updBlackList.sh
+++ b/updBlackList.sh
@@ -475,7 +475,7 @@ doProcess4()
     cat ${flBlockList} | \
         sed -e '/^[;#]/d' \
             -e '/ERROR/d' \
-            -e '/[:\::]/d' \
+            -e '/[\:\::]/d' \
             -e 's/ .*//g' \
             -e 's#//.*##g' \
             -e 's/[^0-9,.,/]*//g' \
@WaterByWind
Copy link
Owner

I do not receive this error. Are you running this on your edgerouter or elsewhere? On edgeos 'sed' is actually the busybox implementation.

@mklxd
Copy link
Author

mklxd commented Sep 13, 2018

Yes. I get this error running on ER4 under 2.0 beta-3

@WaterByWind
Copy link
Owner

Ah - You probably mean 2.0.0-alpha.3. FYI that is actually not a beta and is actually a still in-development alpha that is currently still undergoing significant changes. I do not see the error on alpha-2, for instance.

Once that version moves into a beta (it has not yet) I will be testing there but until then it is still far too early to try to chase as the platform is still in too great of a state of change.

I'll keep this open to track until then.

@mklxd
Copy link
Author

mklxd commented Sep 14, 2018 via email

@WaterByWind
Copy link
Owner

I've pushed this minor update (finally). It has been working on both 1.x and 2.x releases for me and it just took me finding the 30 seconds to publish it. . .

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