search

cheusov / dictd

Client/server software, human language dictionary databases, and tools supporting the DICT protocol (RFC 2229)
71 stars 10 forks source link

Potential bashisms #13

Closed mcepl closed 2 years ago

mcepl commented 2 years ago 
stitny~/b/E/d/dictd-1.13.1$ checkbashisms dictfmt_index2suffix.in
possible bashism in dictfmt_index2suffix.in line 41 ($PIPESTATUS):
    exit_="echo \${PIPESTATUS[@]} | egrep '^0( 0)*$' >/dev/null"
possible bashism in dictfmt_index2suffix.in line 41 (bash arrays, ${name[0|*|@]}):
    exit_="echo \${PIPESTATUS[@]} | egrep '^0( 0)*$' >/dev/null"
stitny~/b/E/d/dictd-1.13.1$ checkbashisms dictfmt_index2word.in
possible bashism in dictfmt_index2word.in line 36 ($PIPESTATUS):
    exit_="echo \${PIPESTATUS[@]} | egrep '^0( 0)*$' >/dev/null"
possible bashism in dictfmt_index2word.in line 36 (bash arrays, ${name[0|*|@]}):
    exit_="echo \${PIPESTATUS[@]} | egrep '^0( 0)*$' >/dev/null"
stitny~/b/E/d/dictd-1.13.1$

Should I care about this? I know you develop on *BSD, so I guess it shouldn’t be a problem?

cheusov commented 2 years ago

This code is surrounded by "if test $BASH; then". So, it should not be a problem.