ericmckean / mozc

Automatically exported from code.google.com/p/mozc
0 stars 0 forks source link

'build_mozc.py gyp' fails when gyp r1667 or higher is used #198

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
If you are using gclient to check out Mozc and its dependent libraries, this 
issue never affects you.

As of Mozc r171, build_mozc.py has internally called gyp as 'python 
/usr/bin/gyp'.
This was OK because /usr/bin/gyp was a python script for a long time.
However, gyp r1667 replaced /usr/bin/gyp with a bash script.
https://code.google.com/p/gyp/source/detail?r=1667
As a result, 'python /usr/bin/gyp' no longer works well and 'build_mozc.py gyp' 
results in a syntax like this if you build Mozc r171 or prior with gyp r1667 or 
higher. (Note that Mozc r171 is designed and tested to be combined only with 
gyp r1656. See our DEPS file as described below)

Here is the sample error log.
=====
  File "/home/user_name/work/oss_mozc/src/third_party/gyp/gyp", line 6
    BASE=`dirname $0`
                  ^
SyntaxError: invalid syntax
Traceback (most recent call last):
  File "build_mozc.py", line 1480, in <module>
    main()
  File "build_mozc.py", line 1462, in main
    GypMain(cmd_opts, cmd_args)
  File "build_mozc.py", line 926, in GypMain
    RunOrDie(command_line)
  File "/home/user_name/work/oss_mozc/src/build_tools/util.py", line 97, in RunOrDie
    '==========']))
build_tools.util.RunOrDieError: 
=====

If you want to build Mozc r171 with gyp r1667 or higher, you can use the 
attached patch (except for Windows).

A bit detailed background:
Our DEPS file describes what versions of external libraries are used to build 
the corresponding release of Google Japanese Input.
https://code.google.com/p/mozc/source/browse/trunk/src/DEPS

Of course it is OK for a user and/or distributor to use OSS Mozc with different 
versions of external libraries as long as the source code license is satisfied, 
but we will put very low priority on fixing issues that occurs only in such a 
trial. Basically our primary goal is to release a maximum subset of Google 
Japanese Input under OSS license in a sustainable manner.

We appreciate your understanding of our situation.

Original issue reported on code.google.com by yukawa@google.com on 16 Sep 2013 at 5:50

Attachments:

GoogleCodeExporter commented 9 years ago
Fixed in r178.

Original comment by yukawa@google.com on 31 Oct 2013 at 7:04