[build] Fix Makefile based build.

[build] Fix Makefile based build.

This fixes the Python include path after the GYP files have been
relocated into another directory.

R=machenbach@chromium.org

Committed: https://crrev.com/ee69b4ff319430fc2799a18fcbd7b1f2f0fb4888
Cr-Commit-Position: refs/heads/master@{#35928}

[Michael Achenbach, 2016-05-02 09:21:51]

lgtm

[Michael Starzinger, 2016-05-02 09:29:28]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-05-02 09:29:47]

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

[commit-bot: I haz the power, 2016-05-02 09:41:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-02 09:42:15]

Description was changed from

==========
[build] Fix Makefile based build.

This fixes the Python include path after the GYP files have been
relocated into another directory.

R=machenbach@chromium.org
==========

to

==========
[build] Fix Makefile based build.

This fixes the Python include path after the GYP files have been
relocated into another directory.

R=machenbach@chromium.org

Committed: https://crrev.com/ee69b4ff319430fc2799a18fcbd7b1f2f0fb4888
Cr-Commit-Position: refs/heads/master@{#35928}
==========

[commit-bot: I haz the power, 2016-05-02 09:42:16]

Patchset 1 (id:??) landed as
https://crrev.com/ee69b4ff319430fc2799a18fcbd7b1f2f0fb4888
Cr-Commit-Position: refs/heads/master@{#35928}

[caitp (gmail), 2016-05-02 12:45:36]

icu.gyp still includes build/shim_headers.gypi on 2 lines, which causes Makefile
builds to fail.

Can you submit a followup to fix that?

[Michael Achenbach, 2016-05-02 12:49:17]

On 2016/05/02 12:45:36, caitp wrote:
> icu.gyp still includes build/shim_headers.gypi on 2 lines, which causes
Makefile
> builds to fail.
> 
> Can you submit a followup to fix that?

But that should work after a gclient sync. It pulls "build" from chromium which
includes that file.

[Michael Achenbach, 2016-05-02 12:50:15]

On 2016/05/02 12:49:17, Michael Achenbach wrote:
> On 2016/05/02 12:45:36, caitp wrote:
> > icu.gyp still includes build/shim_headers.gypi on 2 lines, which causes
> Makefile
> > builds to fail.
> > 
> > Can you submit a followup to fix that?
> 
> But that should work after a gclient sync. It pulls "build" from chromium
which
> includes that file.

Maybe we also need the pythonpath to include the "build" folder?

[caitp (gmail), 2016-05-02 12:52:49]

On 2016/05/02 12:49:17, Michael Achenbach wrote:
> On 2016/05/02 12:45:36, caitp wrote:
> > icu.gyp still includes build/shim_headers.gypi on 2 lines, which causes
> Makefile
> > builds to fail.
> > 
> > Can you submit a followup to fix that?
> 
> But that should work after a gclient sync. It pulls "build" from chromium
which
> includes that file.

ah, I see. `gclient sync` was failing for some unrelated reason