Drop unneeded escaping of - (fixes #3379) #3397
Conversation
| @@ -66,14 +66,14 @@ DB_OTHERS="`ls -1 ${PREPATH}Packages-* | grep -v "$DB_MAIN"`" | |||
| #120903 improve pkg db selection... | |||
| case $DB_MAIN in | |||
| *"-puppy-2-"*|*"-puppy-3-"*|*"-puppy-4-"*|*"-puppy-5-"*) DB_OTHERS="" ;; | |||
| *) DB_OTHERS="`echo "$DB_OTHERS" | grep -v '\\-puppy\\-[2345]\\-'`" ;; #do not look in puppy-2, puppy-3, puppy-4 or puppy-5. | |||
| *) DB_OTHERS="`echo "$DB_OTHERS" | grep -v '\-puppy-[2345]-'`" ;; #do not look in puppy-2, puppy-3, puppy-4 or puppy-5. | |||
There was a problem hiding this comment.
Ha, this one never worked in first place!
~$ (echo abcd; echo a-puppy-4-) | grep -v '\\-puppy\\-[2345]\\-'
abcd
a-puppy-4-
~$ (echo abcd; echo a-puppy-4-) | grep -v '\-puppy-[2345]-'
abcd
Yes - either escape the first |
|
@peabee I'm waiting for you to confirm that this solves your problem. You're probably the only one running woof-CE on Slackware current. |
|
Its not easy to test until it has been merged into testing branch............?? I guess I have to download Woof-CE from https://github.com/dimkr/woof-CE/tree/bugfix/more-grep-warnings |
|
Just done manual edits of 0setup and 1download and they seemed OK - no grep warnings. |
It wasn't easy to fix either 😠 |
|
I'll be building on -current soon enough too. My wayland -current install even prints! 😲 I reckon merge this. In fact Ill do it now, deal with consequences, if any, later. |
|
Test build: Still get a warning during 3builddistro:
|
|
OK after a lot of digging looks like its: |
Alternative to #3395 that should work.