Move cr build configuration outside output folders

Move cr build configuration outside output folders

This patch moves the cr config files from:

  chromium_checkout/
    .cr/
      config.py (global client config)
    src/
      out/
        Release/
          .cr/
            config.py (build config for out/Release)

To:

  chromium_checkout/
    .cr/
      client/
        config.py (global client config)
      builds/
        out/
          Release/
            config.py (build config for out/Release)

By doing this, we will avoid the situation when clobber deletes the
build config (see the associated tracking bug below).

Note that cr will automatically migrate existing config files to the new
directory structure.

BUG=493023
Committed: https://crrev.com/3fb5c2d9a5922fd6bc4a0905f1eea9713fb2d0c6
Cr-Commit-Position: refs/heads/master@{#332613}

[petrcermak, 2015-06-03 10:47:43]

Patchset #1 (id:1) has been deleted

[petrcermak, 2015-06-03 10:49:08]

petrcermak@chromium.org changed reviewers:
+ skyostil@chromium.org

[petrcermak, 2015-06-03 10:49:08]

Hi, please review this patch.

Thanks,
Petr

[Sami, 2015-06-03 12:49:14]

Thanks, lgtm with a nit.

https://codereview.chromium.org/1157293004/diff/20001/tools/cr/cr/base/client.py
File tools/cr/cr/base/client.py (right):

https://codereview.chromium.org/1157293004/diff/20001/tools/cr/cr/base/client...
tools/cr/cr/base/client.py:89: def _GetOldConfigDir(use_build_dir):
nit: s/Old/Deprecated/ to make it a little more obvious what this is for.

https://codereview.chromium.org/1157293004/diff/20001/tools/cr/cr/base/client...
tools/cr/cr/base/client.py:252: print 'Build config file is',
_GetConfigFile(_GetConfigDir(
Nice, this is useful.

[petrcermak, 2015-06-03 13:21:19]

Thanks for your comments.

Petr

https://codereview.chromium.org/1157293004/diff/20001/tools/cr/cr/base/client.py
File tools/cr/cr/base/client.py (right):

https://codereview.chromium.org/1157293004/diff/20001/tools/cr/cr/base/client...
tools/cr/cr/base/client.py:89: def _GetOldConfigDir(use_build_dir):
On 2015/06/03 12:49:14, Sami wrote:
> nit: s/Old/Deprecated/ to make it a little more obvious what this is for.

Done.

[petrcermak, 2015-06-03 13:24:34]

The CQ bit was checked by petrcermak@chromium.org

[petrcermak, 2015-06-03 13:24:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/1157293004/#ps40001
(title: "Address skyostil's comments")

[commit-bot: I haz the power, 2015-06-03 13:25:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1157293004/40001

[commit-bot: I haz the power, 2015-06-03 14:28:13]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-06-03 14:28:56]

Patchset 2 (id:??) landed as
https://crrev.com/3fb5c2d9a5922fd6bc4a0905f1eea9713fb2d0c6
Cr-Commit-Position: refs/heads/master@{#332613}