Update build instructions for Linux...

cztomczak · cztomczak · commit 6e0dd2a592eb · 2016-06-21T22:27:40.000+02:00
Minor fix to automate.py
diff --git a/docs/Build-instructions.md b/docs/Build-instructions.md
@@ -50,6 +50,15 @@ __Windows__
 
 __Linux__
 
+* Install required packages using apt-get or similar:
+```
+sudo apt-get install cmake g++ pkg-config gtk+-2.0 libgtkglext1-dev
+```
+* Download [ninja](http://martine.github.io/ninja/) 1.7.1 or later
+  and copy it to /usr/bin and chmod 755.
+* When building CEF from sources you will need to install many more packages
+  using the install-build-deps.sh script - instructions are provided
+  further down on this page.
 * Building on Ubuntu 12.04 is supported up to branch 2526 (Chrome 47).
   For branches 2623 (Chrome 49) or later Ubuntu 14.04+ is required.
 * To build on Debian 7 see
@@ -83,11 +92,27 @@ need to customize the build then use the --help flag to see more.
 
 Run the automate.py tool using the --build-cef flag. Specify cef branch
 and optionally how many parallel ninja jobs to run (by default cores/2).
+
 ```
 cd tools/
 python automate.py --build-cef --cef-branch 2526 --ninja-jobs 6
 ```
 
+__IMPORTANT__: On Linux it will fail on first run. After the chromium
+sources are downloaded, it will try to build cef projects and fail
+due to missing packages. You will need to run the install-build-deps.sh
+script (intended for Ubuntu systems). When the "ttf-mscorefonts-installer"
+graphical installer pops up you can deny EULA and not install these fonts
+(might deteriorate UX on some systems, experienced on Ubuntu 12.04 32-bit).
+
+```
+cd build/chromium/src/build/
+chmod 755 install-build-deps.sh
+sudo ./install-build-deps.sh --no-arm --no-chromeos-fonts --no-nacl
+```
+
+After dependencies are satisifed re-run automate.py.
+
 You should be fine by running it with the default options, but if you
 need to customize the build then use the --help flag to see more.
 
diff --git a/tools/automate.py b/tools/automate.py
@@ -392,7 +392,8 @@ def getenv():
     env = os.environ
     env["PATH"] = Options.depot_tools_dir + os.pathsep + env["PATH"]
     env["GYP_GENERATORS"] = Options.gyp_generators
-    env["GYP_MSVS_VERSION"] = Options.gyp_msvs_version
+    if platform.system() == "Windows":
+        env["GYP_MSVS_VERSION"] = Options.gyp_msvs_version
     # Issue73 patch applied
     env["GYP_DEFINES"] = "use_allocator=none"
     # To perform an official build set GYP_DEFINES=buildtype=Official.
