Re: [dpdk-dev] [PATCH] checkpatches: don't assume bash syntax

2018-08-29 Thread Hunt, David
Hi Stephen, On 13/8/2018 4:47 PM, Stephen Hemminger wrote: The read -d option is a bash extension and not avaiable in other shells. On Debian, /bin/sh is dash and checktpatches would fail with: ./devtools/checkpatches.sh: 52: read: Illegal option -d Fix by using awk -e and adding necessary dou

[dpdk-dev] [PATCH] checkpatches: don't assume bash syntax

2018-08-13 Thread Stephen Hemminger
The read -d option is a bash extension and not avaiable in other shells. On Debian, /bin/sh is dash and checktpatches would fail with: ./devtools/checkpatches.sh: 52: read: Illegal option -d Fix by using awk -e and adding necessary double backslash. Fixes: 7413e7f2aeb3 ("devtools: alert on new ca