Ensure buildbot knows that the checkout was patched when patching.

Ensure buildbot knows that the checkout was patched when patching.

R=rogerta@chromium.org
BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155506

[M-A Ruel, 2012-09-07 20:47:48]

https://chromiumcodereview.appspot.com/10905160/diff/1/apply_issue.py
File apply_issue.py (right):

https://chromiumcodereview.appspot.com/10905160/diff/1/apply_issue.py#newcode102
apply_issue.py:102: open('.buildbot-patched', 'w').close()
In this case, the current working directory is the right one.

[Roger Tawa OOO till Jul 10th, 2012-09-07 22:13:42]

Lgtm
On Sep 7, 2012 4:47 PM, <maruel@chromium.org> wrote:

> Reviewers: Roger Tawa,
>
>
>
https://chromiumcodereview.**appspot.com/10905160/diff/1/**apply_issue.py<htt...
> File apply_issue.py (right):
>
> https://chromiumcodereview.**appspot.com/10905160/diff/1/**
>
apply_issue.py#newcode102<https://chromiumcodereview.appspot.com/10905160/diff/1/apply_issue.py#newcode102>
> apply_issue.py:102: open('.buildbot-patched', 'w').close()
> In this case, the current working directory is the right one.
>
> Description:
> Ensure buildbot knows that the checkout was patched when patching.
>
> TBR=rogerta@chromium.org
> BUG=
>
>
> Please review this at
https://chromiumcodereview.**appspot.com/10905160/<https://chromiumcodereview...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/tools/depot_tools<http://svn.chromium.org/chrome/trunk/tools/depot_tools>
>
> Affected files:
>   M apply_issue.py
>
>
> Index: apply_issue.py
> diff --git a/apply_issue.py b/apply_issue.py
> index 22018742d45a860fa51a3eac9f4cad**72da06fe08..**
> a69a726153bd034da02c59497c11dc**a9052a2278 100755
> --- a/apply_issue.py
> +++ b/apply_issue.py
> @@ -6,6 +6,7 @@
>  """Applies an issue from Rietveld.
>  """
>
> +import getpass
>  import logging
>  import optparse
>  import os
> @@ -93,6 +94,13 @@ def main():
>    else:
>      parser.error('Couldn\'t determine the scm')
>
> +  # TODO(maruel): HACK, remove me.
> +  if options.root_dir == 'src' and getpass.getuser() == 'chrome-bot':
> +    # See sourcedirIsPatched() in:
> +    # http://src.chromium.org/**viewvc/chrome/trunk/tools/**
>
build/scripts/slave/<http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/>
> +    #    chromium_commands.py?view=**markup
> +    open('.buildbot-patched', 'w').close()
> +
>    # Apply the patch.
>    try:
>      scm_obj.apply_patch(patchset)
>
>
>

[commit-bot: I haz the power, 2012-09-07 22:20:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/10905160/3

[commit-bot: I haz the power, 2012-09-07 22:23:26]

Change committed as 155506