Re: [PATCH 06/10] don't run sparse{c,i} tests when sparse-llvm is disabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux