Fix depot_tools\git.bat when depot_tools in not in PATH

Fix depot_tools\git.bat when depot_tools in not in PATH

R=iannucci@chromium.org, dpranke@chromium.org
BUG=

[Dirk Pranke, 2016-05-25 15:46:17]

Why would depot_tools need to be in the path for this? What git command will
invoke a non-git depot_tools command?

[anatoly techtonik, 2016-05-25 16:11:21]

On 2016/05/25 15:46:17, Dirk Pranke (slow) wrote:
> Why would depot_tools need to be in the path for this? What git command will
> invoke a non-git depot_tools command?

git cl

[Dirk Pranke, 2016-05-25 16:12:54]

On 2016/05/25 16:11:21, anatoly techtonik wrote:
> On 2016/05/25 15:46:17, Dirk Pranke (slow) wrote:
> > Why would depot_tools need to be in the path for this? What git command will
> > invoke a non-git depot_tools command?
> 
> git cl

Good example :).

lgtm.

[anatoly techtonik, 2016-05-25 18:09:12]

The CQ bit was checked by techtonik@gmail.com

[commit-bot: I haz the power, 2016-05-25 18:09:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2009173002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2009173002/1

[commit-bot: I haz the power, 2016-05-25 18:13:05]

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

[commit-bot: I haz the power, 2016-05-25 18:13:06]

Try jobs failed on following builders:
  Depot Tools Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Depot%20Tools%20Presubm...)

[anatoly techtonik, 2016-05-25 18:52:17]

I guess it is linefeed problem.

[iannucci, 2016-05-25 20:27:05]

On 2016/05/25 18:52:17, anatoly techtonik wrote:
> I guess it is linefeed problem.

Yes this is a newline issue.

[iannucci, 2016-05-25 20:41:35]

iannucci@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[iannucci, 2016-05-25 20:41:36]

I'm not sure how we can commit this without someone taking over this CL :/

Pawel, do you know of a way that we can CQ this? Otherwise I can land this on
your behalf Anatoly.

[iannucci, 2016-05-25 20:46:00]

On 2016/05/25 20:41:36, iannucci wrote:
> I'm not sure how we can commit this without someone taking over this CL :/
> 
> Pawel, do you know of a way that we can CQ this? Otherwise I can land this on
> your behalf Anatoly.

Sounds like I'll have to land this. I'll do this now.

[anatoly techtonik, 2016-05-25 20:48:02]

On 2016/05/25 20:46:00, iannucci wrote:
> On 2016/05/25 20:41:36, iannucci wrote:
> > I'm not sure how we can commit this without someone taking over this CL :/
> > 
> > Pawel, do you know of a way that we can CQ this? Otherwise I can land this
on
> > your behalf Anatoly.
> 
> Sounds like I'll have to land this. I'll do this now.

Thanks. Probably .bat files need an exception from depot_tools Git config.

[iannucci, 2016-05-25 20:48:57]

closing in favor of https://codereview.chromium.org/2015673002/

[iannucci, 2016-05-25 20:49:33]

I think that this will get better when we switch to gerrit, where there's no
more text patch files.

[anatoly techtonik, 2016-05-26 11:13:58]

On 2016/05/25 20:49:33, iannucci wrote:
> I think that this will get better when we switch to gerrit, where there's no
> more text patch files.

And how does Gerrit handles that? Does it rely on some internal git changeset
object?