Run scripts which don't have any compile targets if any source files change.

Run scripts which don't have any compile targets if any source files change.

This fixes the problem that checkdeps could be broken by changes that don't impact Linux, which is the only trybot that runs these tests.

BUG=462308

[luqui, 2015-03-02 23:21:52]

luqui@chromium.org changed reviewers:
+ luqui@chromium.org

[luqui, 2015-03-02 23:21:53]

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium_trybot.py (right):

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium_trybot.py:389: return
You could either add a test for it in GenTests (get affected_files to be empty
somehow) or add # pragma: no cover.  The former is preferable, of course, if you
can figure out how.

[jam, 2015-03-04 15:16:42]

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium_trybot.py (right):

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium_trybot.py:389: return
On 2015/03/02 23:21:53, luqui wrote:
> You could either add a test for it in GenTests (get affected_files to be empty
> somehow) or add # pragma: no cover.  The former is preferable, of course, if
you
> can figure out how.

right, i didn't know how to do the "somehow" part :)

[luqui, 2015-03-04 21:51:04]

luqui@chromium.org changed reviewers:
+ iannucci@chromium.org

[luqui, 2015-03-04 21:51:04]

+iannucci in case I'm wrong.

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
File scripts/slave/recipes/chromium_trybot.py (right):

https://codereview.chromium.org/963263002/diff/1/scripts/slave/recipes/chromi...
scripts/slave/recipes/chromium_trybot.py:389: return
On 2015/03/04 15:16:41, jam wrote:
> On 2015/03/02 23:21:53, luqui wrote:
> > You could either add a test for it in GenTests (get affected_files to be
empty
> > somehow) or add # pragma: no cover.  The former is preferable, of course, if
> you
> > can figure out how.
> 
> right, i didn't know how to do the "somehow" part :)

You could try 

   yield (api.test('...') + 
          ... +
          api.override_step_data('git diff to analyze patch',
api.raw_io.stream_output('')))

I'm not super familiar with this, but that's my best guess.

[luqui, 2015-08-11 22:32:04]

I took this over at https://codereview.chromium.org/1287723002

[luqui, 2015-08-11 22:32:21]

I took this over at https://codereview.chromium.org/1287723002