On Tue, Aug 5, 2014 at 2:38 AM, Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxx> wrote: > +if [ ! -x "$default_path/sparse-llvm" ]; then > + disabled_cmds="sparsec sparsei sparse-llvm" > +fi I think this can be simplified as if the test command was not found, then that test will be disabled. It doesn't need to be sparse llvm, it can be other test program. We don't need to set an disable_cmds list then check against that list. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html