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

Delta Between Two Patch Sets: README.md

Issue 29458601: Issue 5315 - Add support for Microsoft Edge (Closed)
Left Patch Set: Rebase the README.md, compress images, adress the comments Created June 14, 2017, 3:33 a.m.
Right Patch Set: Cosmetic changes Created July 28, 2017, 11:02 a.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 | « no previous file | ext/background.js » ('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 for Chrome, Opera, Microsoft Edge and Firefox 1 Adblock Plus for Chrome, Opera, Microsoft Edge and Firefox
2 ========================================== 2 ==========================================================
3 3
4 This repository contains the platform-specific Adblock Plus source code for 4 This repository contains the platform-specific Adblock Plus source code for
5 Chrome, Opera, Microsoft Edge and Firefox. It can be used to build 5 Chrome, Opera, Microsoft Edge and Firefox. It can be used to build
6 Adblock Plus for these platforms, generic Adblock Plus code will be extracted 6 Adblock Plus for these platforms, generic Adblock Plus code will be extracted
7 from other repositories automatically (see _dependencies_ file). 7 from other repositories automatically (see _dependencies_ file).
8 8
9 Note that the Firefox extension built from this repository is the new 9 Note that the Firefox extension built from this repository is the new
10 [WebExtension](https://developer.mozilla.org/en-US/Add-ons/WebExtensions). 10 [WebExtension](https://developer.mozilla.org/en-US/Add-ons/WebExtensions).
11 The source code of the legacy Adblock Plus extension 11 The source code of the legacy Adblock Plus extension
12 can be found [here](https://hg.adblockplus.org/adblockplus). 12 can be found [here](https://hg.adblockplus.org/adblockplus).
13 13
14 Building 14 Building
15 --------- 15 ---------
16 16
17 ### Requirements 17 ### Requirements
18 18
19 - [Mercurial](https://www.mercurial-scm.org/) or [Git](https://git-scm.com/) (wh ichever you used to clone this repository) 19 - [Mercurial](https://www.mercurial-scm.org/) or [Git](https://git-scm.com/) (wh ichever you used to clone this repository)
20 - [Python 2.7](https://www.python.org) 20 - [Python 2.7](https://www.python.org)
21 - [The Jinja2 module](http://jinja.pocoo.org/docs) (>= 2.8) 21 - [The Jinja2 module](http://jinja.pocoo.org/docs) (>= 2.8)
22 - [The PIL module](http://www.pythonware.com/products/pil/) 22 - [The PIL module](http://www.pythonware.com/products/pil/)
23 - For signed builds: [PyCrypto module](https://www.dlitz.net/software/pycrypto/) 23 - For signed builds: [PyCrypto module](https://www.dlitz.net/software/pycrypto/)
24 24
25 ### Building the extension 25 ### Building the extension
26 26
27 Run one of the following commands in the project directory, depending on your 27 Run one of the following commands in the project directory, depending on your
28 target platform: 28 target platform:
29 29
30 ./build.py -t chrome build -k adblockpluschrome.pem 30 ./build.py -t chrome build -k adblockpluschrome.pem
31 ./build.py -t edge build
31 ./build.py -t gecko-webext build 32 ./build.py -t gecko-webext build
32 ./build.py -t edge build
Sebastian Noack 2017/06/14 05:13:39 Nit: In the human-readable description we refer to
33 33
34 This will create a build with a name in the form 34 This will create a build with a name in the form
35 _adblockpluschrome-1.2.3.nnnn.crx_ or _adblockplusfirefox-1.2.3.nnnn.xpi_ or 35 _adblockpluschrome-1.2.3.nnnn.crx_, _adblockplusedge-1.2.3.nnnn.appx_ or
Sebastian Noack 2017/06/14 05:13:39 Nit: Don't use multiple "or", use "a, b or c" inst
36 _adblockplusedge-1.2.3.nnnn.appx_. 36 _adblockplusfirefox-1.2.3.nnnn.xpi_.
37 37
38 Note that you don't need an existing signing key for Chrome, a new key 38 Note that you don't need an existing signing key for Chrome, a new key
39 will be created automatically if the file doesn't exist. 39 will be created automatically if the file doesn't exist.
40 40
41 The Microsoft Edge build _adblockplusedge-1.2.3.nnnn.appx_ is unsigned and 41 The Microsoft Edge build _adblockplusedge-1.2.3.nnnn.appx_ is unsigned and
42 is only useful for uploading into Windows Store, where it will be signed. For 42 is only useful for uploading into Windows Store, where it will be signed. For
43 testing use the devenv build. 43 testing use the devenv build.
44 44
45 The Firefox extension will be unsigned, and therefore is mostly only useful for 45 The Firefox extension will be unsigned, and therefore is mostly only useful for
46 upload to Mozilla Add-ons. You can also also load it for testing purposes under 46 upload to Mozilla Add-ons. You can also also load it for testing purposes under
47 _about:debugging_ or by disabling signature enforcement in Firefox Nightly. 47 _about:debugging_ or by disabling signature enforcement in Firefox Nightly.
48 48
49 ### Development environment 49 ### Development environment
50 50
51 To simplify the process of testing your changes you can create an unpacked 51 To simplify the process of testing your changes you can create an unpacked
52 development environment. For that run one of the following commands: 52 development environment. For that run one of the following commands:
53 53
54 ./build.py -t chrome devenv 54 ./build.py -t chrome devenv
55 ./build.py -t edge devenv
55 ./build.py -t gecko-webext devenv 56 ./build.py -t gecko-webext devenv
56 ./build.py -t edge devenv
57 57
58 This will create a _devenv.*_ directory in the repository. You can load the 58 This will create a _devenv.*_ directory in the repository. You can load the
59 directory as an unpacked extension, under _chrome://extensions_ in Chrome, 59 directory as an unpacked extension, under _chrome://extensions_ in Chrome,
60 under _about:debugging_ in Firefox or in Extensions menu in Edge, after enabling 60 under _about:debugging_ in Firefox or in _Extensions_ menu in Microsoft Edge,
Sebastian Noack 2017/06/14 05:13:38 "... in the _Extensions_ menu in Microsoft Edge ..
61 extension development features in _about:flags_. After making changes to the 61 after enabling extension development features in _about:flags_.
62 source code re-run the command to update the development environment. In Chrome 62 After making changes to the source code re-run the command to update the
63 and Firefox the extension should reload automatically after a few seconds. 63 development environment. In Chrome and Firefox the extension should reload
64 automatically after a few seconds.
64 65
65 Edge build does not automatically detect changes, so after rebuilding the 66 Builds for Microsoft Edge do not automatically detect changes, so after
Sebastian Noack 2017/06/14 05:13:39 "Builds for Microsoft Edge do not ..."
66 extension you should manually force reloading it in Edge by hitting the 67 rebuilding the extension you should manually force reloading it in Edge by
67 _Reload Extension_ button. 68 hitting the _Reload Extension_ button.
68 69
69 Running the unit tests 70 Running the unit tests
70 ---------------------- 71 ----------------------
71 72
72 To verify your changes you can use the unit test suite located in the _qunit_ 73 To verify your changes you can use the unit test suite located in the _qunit_
73 directory of the repository. In order to run the unit tests go to the 74 directory of the repository. In order to run the unit tests go to the
74 extension's Options page, open the JavaScript Console and type in: 75 extension's Options page, open the JavaScript Console and type in:
75 76
76 location.href = "qunit/index.html"; 77 location.href = "qunit/index.html";
77 78
78 The unit tests will run automatically once the page loads. 79 The unit tests will run automatically once the page loads.
79 80
80 Linting 81 Linting
81 ------- 82 -------
82 83
83 You can lint the code using [ESLint](http://eslint.org). 84 You can lint the code using [ESLint](http://eslint.org).
84 85
85 eslint *.js lib/ qunit/ ext/ chrome/ 86 eslint *.js lib/ qunit/ ext/ chrome/
86 87
87 You will need to set up ESLint and our configuration first, see 88 You will need to set up ESLint and our configuration first, see
88 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo) 89 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo)
89 for more information. 90 for more information.
LEFTRIGHT
« no previous file | ext/background.js » ('j') | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld