mojo: Fix dependency for building python bindings.

mojo: Fix dependency for building python bindings.

R=sdefresne@chromium.org

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

[sdefresne, 2014-09-18 08:19:21]

Code LGTM but I may miss the big picture there. I guess that the file you
removed was added just to handle those binary modules, and using cp.py is now
enough. Correct?

[qsr, 2014-09-18 08:34:41]

On 2014/09/18 08:19:21, sdefresne wrote:
> Code LGTM but I may miss the big picture there. I guess that the file you
> removed was added just to handle those binary modules, and using cp.py is now
> enough. Correct?

 It is a little more complicated than that.

 I build mojo module with the gyp infrastructure, which mean that the system
module ends up in a .so called {{PRODUCT_DIR}}/libmojo_python_system.so

 I need this file to be copied to {{PRODUCT_DIR}}/python/mojo/system.so

 The previous build did that automatically, using the
third_party/cython/cp_python_binary_modules.py by being given an argument that
was build by the cython code generation. The issue is that the target didn't
have the right dependencies, so a change to system.pyx would rebuild
{{PRODUCT_DIR}}/libmojo_python_system.so but would not copy it to its final
destination. Which cause the breakage on the build because the wrong module was
tested.

 This CL change this to hardcode the needed copy, so that they have the right
input/output, and so are correctly updated.

[qsr, 2014-09-18 08:42:59]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-09-18 08:43:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/578893002/20001

[commit-bot: I haz the power, 2014-09-18 10:07:11]

Committed patchset #2 (id:20001) as b4a4154e75179b92cf66d90208abbf36c7d04703

[commit-bot: I haz the power, 2014-09-18 10:56:07]

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