Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: README.md

Issue 29527808: Noissue - Use meson to build the C++ code (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Left Patch Set: Updated from feedback. now require meson 0.40.0, but no default C++ compiler Created Oct. 26, 2017, 4:13 p.m.
Right Patch Set: Removed compile. meson > 0.40.0 Created Dec. 15, 2017, 5:37 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « .hgignore ('k') | compile » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 Adblock Plus core 1 Adblock Plus core
2 ================= 2 =================
3 3
4 This repository contains the generic Adblock Plus code that's shared between 4 This repository contains the generic Adblock Plus code that's shared between
5 platforms. This repository is not designed to be used directly, but instead to 5 platforms. This repository is not designed to be used directly, but instead to
6 serve as a dependency for `adblockplus`, `adblockpluschrome` and 6 serve as a dependency for `adblockplus`, `adblockpluschrome` and
7 `libadblockplus`. 7 `libadblockplus`.
8 8
9 Compiling C++ code 9 Compiling C++ code
10 ------------------ 10 ------------------
11 11
12 ### Purpose 12 ### Purpose
13 13
14 In order to improve performance and memory usage, some of the code (located 14 In order to improve performance and memory usage, some of the code (located
15 inside the `compiled` directory) is written in C++ and compiled to JavaScript 15 inside the `compiled` directory) is written in C++ and compiled to JavaScript
16 via Empscripten. 16 via Empscripten.
17 17
18 ### Requirements 18 ### Requirements
19 19
20 * [Emscripten 1.37.3](https://github.com/kripken/emscripten) 20 * [Emscripten 1.37.3](https://github.com/kripken/emscripten)
21 * [Python 2.7 & 3](https://www.python.org) 21 * [Python 2.7](https://www.python.org)
22 * [meson 0.40.0+](https://www.mesonbuild.com) 22 * [meson 0.40.0+](https://www.mesonbuild.com)
23 * [ninja](https://www.ninja-build.org) 23 * [ninja](https://www.ninja-build.org)
24 24
25 ### Installation 25 Make sure that meson and ninja are in your PATH.
26
27 #### macOS
28
29 Whenever possible, install from brew.
30
31 #### Linux
32
33 Use the distribution package if possible. Some "LTS" might have a too
34 old version of meson.
Wladimir Palant 2017/10/26 19:27:00 Ubuntu 17.04 isn't LTS - it was the latest stable
hub 2017/10/26 19:36:04 I'll remove that sentence. Or if you prefer the wh
Wladimir Palant 2017/11/03 10:50:12 I definitely prefer removing the entire installati
35
36 #### Windows
37
38 TBD
39 26
40 ### Running Emscripten 27 ### Running Emscripten
41 28
42 After installing and configuring Emscripten you can setup the build 29 After installing and configuring Emscripten you can setup the build
43 with the following commands: 30 with the following commands:
44 31
45 meson build 32 meson build
46 33
47 By default it will create a debug build. Pass `--buildtype release` to 34 By default it will create a debug build. Pass `--buildtype release` to
48 create a release build. 35 create a release build.
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 Linting 99 Linting
113 ------- 100 -------
114 101
115 You can lint the code using [ESLint](http://eslint.org). 102 You can lint the code using [ESLint](http://eslint.org).
116 103
117 eslint *.js chrome lib test 104 eslint *.js chrome lib test
118 105
119 You will need to set up ESLint and our configuration first, see 106 You will need to set up ESLint and our configuration first, see
120 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo) 107 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo)
121 for more information. 108 for more information.
LEFTRIGHT

Powered by Google App Engine
This is Rietveld