Remove recipes.py and friends from chromium/src (for now).

Remove recipes.py and friends from chromium/src (for now).

Until we have a better testing/rolling strategy, we're removing recipes from
chromium/src.

R=dnj@chromium.org, dpranke@chromium.org
BUG=chromium:584079
Review-Url: https://codereview.chromium.org/2585583007
Cr-Commit-Position: refs/heads/master@{#442172}
Committed: https://chromium.googlesource.com/chromium/src/+/c5560a91769c4495c6669d147e3bc024c7d28019

[Dirk Pranke, 2016-12-18 02:58:25]

lgtm

[dnj, 2016-12-18 15:53:33]

lgtm. FWIW I intentionally left it in there when I cleared the actual recipes
themselves both as a placeholder and in case Chromium project wanted to add its
own recipes in the future. Is there any real need to remove it?

[Dirk Pranke, 2016-12-18 17:40:54]

On 2016/12/18 15:53:33, dnj wrote:
> lgtm. FWIW I intentionally left it in there when I cleared the actual recipes
> themselves both as a placeholder and in case Chromium project wanted to add
its
> own recipes in the future. Is there any real need to remove it?

If we're not using code, it's better to delete it to keep things clean and avoid
confusion.

[iannucci, 2016-12-18 19:09:52]

On 2016/12/18 17:40:54, Dirk Pranke wrote:
> On 2016/12/18 15:53:33, dnj wrote:
> > lgtm. FWIW I intentionally left it in there when I cleared the actual
recipes
> > themselves both as a placeholder and in case Chromium project wanted to add
> its
> > own recipes in the future. Is there any real need to remove it?
> 
> If we're not using code, it's better to delete it to keep things clean and
avoid
> confusion.

yep, that.

[iannucci, 2016-12-18 19:09:56]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2016-12-18 19:10:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-18 19:16:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-18 19:16:11]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[iannucci, 2016-12-19 03:41:13]

ptal again. Looks like I missed the test stuff too. I suspect it's better to
remove the PRESUBMIT file entirely rather than just having one that does
nothing?

[Dirk Pranke, 2016-12-19 05:51:04]

agreed. still lgtm.

[iannucci, 2017-01-07 21:23:19]

The CQ bit was checked by iannucci@chromium.org

[iannucci, 2017-01-07 21:23:19]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@chromium.org
Link to the patchset: https://codereview.chromium.org/2585583007/#ps20001
(title: "Remove presubmit too")

[commit-bot: I haz the power, 2017-01-07 21:23:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-07 21:32:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483824199287020,
"parent_rev": "07e10fd70e44126d20a19b639fb18deee52cd906", "commit_rev":
"c5560a91769c4495c6669d147e3bc024c7d28019"}

[commit-bot: I haz the power, 2017-01-07 21:33:19]

Description was changed from

==========
Remove recipes.py and friends from chromium/src (for now).

Until we have a better testing/rolling strategy, we're removing recipes from
chromium/src.

R=dnj@chromium.org, dpranke@chromium.org
BUG=chromium:584079
==========

to

==========
Remove recipes.py and friends from chromium/src (for now).

Until we have a better testing/rolling strategy, we're removing recipes from
chromium/src.

R=dnj@chromium.org, dpranke@chromium.org
BUG=chromium:584079

Review-Url: https://codereview.chromium.org/2585583007
Cr-Commit-Position: refs/heads/master@{#442172}
Committed:
https://chromium.googlesource.com/chromium/src/+/c5560a91769c4495c6669d147e3b...
==========

[commit-bot: I haz the power, 2017-01-07 21:33:20]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c5560a91769c4495c6669d147e3b...