TechDebt: simplify list of excluded directories in bashate tox

[Steven Dake]

In tox.ini there is a command:

  bash -c "files=`egrep -rlI --exclude-dir .git --exclude-dir .tox '^[[:punct:]]+!/(bin/|/usr/bin/
env )(ba)?sh' .` && bashate $files"

It would be ideal if tox.ini could instead take something like

--exclude-dir={.git,.tox}

But this results in errors. Escaping with --exclude-dir=\{.git,.tox\} as recommended by the tox documentation also results in errors.

The error is:
    "substitution key %r not found" % key)
tox.ConfigError: ConfigError: substitution key '.git,.tox\\' not found

[Jeff Peeler (jpeeler-z)]

I had difficulty using tox for regex quantifiers, which also uses brackets {}. Surely some form of escaping works though.

[Steven Dake (sdake)]

Agree should be possible. It isn't like I didn't try :) I am in a bit of a rush to finish the bashate gating tho and don't want to block on this detail. Hence the tracking as techdebt ;)

[Steven Dake (sdake)]

this is a low priority techdebt bug. Moving to newton 1.

[Christian Berendt (berendt)]

Closing as invalid. We now have a script called tools/run-bashate.sh. In the tox.ini we use "{toxinidir}/tools/run-bashate.sh".