script: Disable SC1004 check in `contrib/devtools/gen-bitcoin-conf.sh` #25238

pull kristapsk wants to merge 1 commits into bitcoin:master from kristapsk:shellcheck-disable=SC1004 changing 1 files +1 −0

kristapsk commented at 4:35 PM on May 29, 2022: contributor

Otherwise ./test/lint/lint-all.py gives me error:

In contrib/devtools/gen-bitcoin-conf.sh line 58:
    | sed 's,\(^[[:upper:]].*\)\:$,\
                                   ^-- SC1004: This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.

For more information:
  https://www.shellcheck.net/wiki/SC1004 -- This backslash+linefeed is litera...
^---- failure generated from lint-shell.py

Disable SC1004 check 55f0b40419
hebasto commented at 4:41 PM on May 29, 2022: member

If your shellcheck differs from v0.8.0 it could produce errors.
kristapsk commented at 4:43 PM on May 29, 2022: contributor

Just checked, I'm having v0.7.1. Will try to upgrade and test again.
DrahtBot added the label Scripts and tools on May 29, 2022
kristapsk commented at 6:34 PM on May 29, 2022: contributor

After upgrading shellcheck to v0.8.0, linter error disappears, so closing.
kristapsk closed this on May 29, 2022
kristapsk deleted the branch on May 29, 2022
fanquake locked this on May 31, 2022

Contributors

kristapsk

hebasto

Labels

Scripts and tools