Warn on uses of rctx.execute which discard the return value

[alexeagle]

This is nearly always wrong, because the return value has a return_code property which should be checked to provide error messaging back to the user. Otherwise failures of subprocesses in repository rules are just ignored.